broulik created this revision.
broulik added reviewers: Plasma, graesslin.
broulik set the repository for this revision to R120 Plasma Workspace.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
Otherwise it fails to lock
GENERAL INFO
BUILD UNSTABLE
Build URL:
https://build.kde.org/job/plasma-tests%20Plasma-5.9%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/3/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Tue, 07 Feb 2017 10:00:03 +
Build duration: 1 min 5 sec
CHANGE SET
Revision
subdiff created this revision.
subdiff added reviewers: Plasma, mart, hein.
subdiff added a subscriber: hein.
subdiff set the repository for this revision to R242 Plasma Framework (Library).
subdiff added a project: Plasma.
Restricted Application added subscribers: Frameworks, plasma-devel.
GENERAL INFO
BUILD UNSTABLE
Build URL:
https://build.kde.org/job/plasma-desktop%20Plasma-5.9%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/53/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Tue, 07 Feb 2017 10:11:58 +
Build duration: 6 min 12 sec
CHANGE SET
Revision
anthonyfieroni added a comment.
Ah, the bug stays over an year opened. Ok i will investigate to re-simulate
move start.
REPOSITORY
R31 Breeze
REVISION DETAIL
https://phabricator.kde.org/D3578
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: drosca,
hpereiradacosta added a comment.
In https://phabricator.kde.org/D3210#83622, @anthonyfieroni wrote:
> We ends with no configurable scrollbar size at all, it has plan for this
*really* ?
Indeed. This was deemed unnecessary (see discussion above), with respect to
the added code
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129917/
---
(Updated fév. 7, 2017, 10:20 matin)
Review request for kde-workspace and
mart accepted this revision.
mart added a reviewer: mart.
This revision is now accepted and ready to land.
REPOSITORY
R265 KConfigWidgets
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D4471
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To:
broulik updated this revision to Diff 10993.
broulik added a comment.
- Disable instead of hiding check box
- if not always show checkbox uncheked
Verified that enabling and disabling the checkbox still enables and disables
the setting. If plasma-pa is uninstalled the checkbox stays
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129932/#review102440
---
not sure is a good idea.
the tooltips not dismissing
davidedmundson accepted this revision.
davidedmundson added a reviewer: davidedmundson.
This revision is now accepted and ready to land.
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D4472
EMAIL PREFERENCES
hein created this revision.
hein added reviewers: Plasma, davidedmundson.
hein added a subscriber: plasma-devel.
Restricted Application added a project: Plasma.
REVISION SUMMARY
Alphabetical sorting currently compares a concatenation of
"AppName" (determined by heuristic) and Qt::DisplayRole
davidedmundson created this revision.
davidedmundson added a reviewer: Plasma.
Restricted Application added projects: Plasma, Frameworks.
Restricted Application added subscribers: Frameworks, plasma-devel.
REVISION SUMMARY
The current code checks if the existing action is valid rather the
This revision was automatically updated to reflect the committed changes.
Closed by commit R289:80c5aa01d207: Send desktopfilename as part of
notifyByPopup hints (authored by davidedmundson).
REPOSITORY
R289 KNotifications
CHANGES SINCE LAST UPDATE
GENERAL INFO
BUILD UNSTABLE
Build URL:
https://build.kde.org/job/plasma-workspace%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/725/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Tue, 07 Feb 2017 09:50:41 +
Build duration: 7 min 32 sec
CHANGE SET
Revision
GENERAL INFO
BUILD UNSTABLE
Build URL:
https://build.kde.org/job/plasma-workspace%20Plasma-5.9%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/48/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Tue, 07 Feb 2017 09:48:16 +
Build duration: 7 min 46 sec
CHANGE SET
Revision
hein accepted this revision.
hein added a comment.
This revision is now accepted and ready to land.
+1
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D4293
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: broulik, hein,
This revision was automatically updated to reflect the committed changes.
Closed by commit R119:d798319fee1d: [Task Manager] Enable "Mark applications
that play audio" option only if plasma… (authored by broulik).
REPOSITORY
R119 Plasma Desktop
CHANGES SINCE LAST UPDATE
hpereiradacosta added a comment.
> Did you actually try to revert it and see if anything changes?
>
> The problem is reproducible only with Qt 5.8, and it doesn't just block
second drag - it breaks all mouse events until you click in the window to
"reset" it. Because the wm move is
davidedmundson accepted this revision.
davidedmundson added a reviewer: davidedmundson.
This revision is now accepted and ready to land.
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D4470
EMAIL PREFERENCES
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129917/#review102439
---
applets/systemloadviewer/package/contents/config/main.xml
This revision was automatically updated to reflect the committed changes.
Closed by commit R120:8bf479e35834: [User Switcher] Ungrab keyboard before
trying to lock the screen (authored by broulik).
REPOSITORY
R120 Plasma Workspace
CHANGES SINCE LAST UPDATE
drosca added a comment.
@hpereiradacosta Yes, happens with both QQC and QWidgets.
REPOSITORY
R31 Breeze
REVISION DETAIL
https://phabricator.kde.org/D3578
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: drosca, #plasma, hpereiradacosta
Cc:
broulik created this revision.
broulik added a reviewer: Plasma.
broulik set the repository for this revision to R120 Plasma Workspace.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
QtScript is deprecated and at least
davidedmundson accepted this revision.
davidedmundson added a reviewer: davidedmundson.
This revision is now accepted and ready to land.
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D4474
EMAIL PREFERENCES
This revision was automatically updated to reflect the committed changes.
Closed by commit R119:acae3816b7ae: [Solid Device Actions KCM] Encode action
file name (authored by broulik).
REPOSITORY
R119 Plasma Desktop
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D4474?vs=11001=11002
GENERAL INFO
BUILD UNSTABLE
Build URL:
https://build.kde.org/job/plasma-desktop%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/598/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Tue, 07 Feb 2017 11:27:46 +
Build duration: 10 min
CHANGE SET
Revision
hein abandoned this revision.
hein added a comment.
I'll abandon, I don't really like this approach either.
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D4475
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: hein,
mart added inline comments.
INLINE COMMENTS
> davidedmundson wrote in framesvg.cpp:136
> I'm lost on why we have the pendingEnabledBorders
>
> you have some bugs if you do:
>
> setRepaintBlocked(false);
> setEnabledBorders(Left)
> enabledBorders() /// returns All not Left.
>
> it'll get
GENERAL INFO
BUILD UNSTABLE
Build URL:
https://build.kde.org/job/plasma-desktop%20Plasma-5.9%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/54/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Tue, 07 Feb 2017 11:11:42 +
Build duration: 6 min 21 sec
CHANGE SET
Revision
guoyunhe closed this revision.
REPOSITORY
R242 Plasma Framework (Library)
REVISION DETAIL
https://phabricator.kde.org/D4288
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: guoyunhe, #plasma, mart
Cc: plasma-devel, guoyunhe, #frameworks, lesliezhai,
GENERAL INFO
BUILD UNSTABLE
Build URL:
https://build.kde.org/job/plasma-tests%20Plasma-5.9%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/4/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Tue, 07 Feb 2017 11:12:48 +
Build duration: 59 sec
CHANGE SET
Revision
GENERAL INFO
BUILD UNSTABLE
Build URL:
https://build.kde.org/job/plasma-desktop%20Plasma-5.9%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/55/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Tue, 07 Feb 2017 11:29:42 +
Build duration: 18 min
CHANGE SET
Revision
GENERAL INFO
BUILD UNSTABLE
Build URL:
https://build.kde.org/job/plasma-desktop%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/600/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Tue, 07 Feb 2017 12:17:52 +
Build duration: 16 min
CHANGE SET
Revision
This revision was automatically updated to reflect the committed changes.
Closed by commit R242:d8a1a9eb084b: don't regenerate frames when setting every
property (authored by mart).
REPOSITORY
R242 Plasma Framework (Library)
CHANGES SINCE LAST UPDATE
subdiff added a comment.
In https://phabricator.kde.org/D4473#83779, @hein wrote:
> Aye. I think that's weird
I agree.
> Does defaulting the frames to on mean the bottom flash is back too or is
that still fixed?
Still fixed.
> Can't comment on the ramifications of
GENERAL INFO
BUILD UNSTABLE
Build URL:
https://build.kde.org/job/plasma-desktop%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/597/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Tue, 07 Feb 2017 10:12:21 +
Build duration: 6 min 26 sec
CHANGE SET
Revision
GENERAL INFO
BUILD SUCCESS
Build URL:
https://build.kde.org/job/kde-gtk-config%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/44/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Tue, 07 Feb 2017 10:13:48 +
Build duration: 1 min 45 sec
CHANGE SET
Revision
broulik created this revision.
broulik added reviewers: Plasma, dfaure.
broulik set the repository for this revision to R119 Plasma Desktop.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
Allows to put slashes in there.
GENERAL INFO
BUILD UNSTABLE
Build URL:
https://build.kde.org/job/plasma-workspace%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/726/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Tue, 07 Feb 2017 10:41:19 +
Build duration: 5 min 38 sec
CHANGE SET
Revision
hein accepted this revision.
hein added a reviewer: hein.
This revision is now accepted and ready to land.
REPOSITORY
R119 Plasma Desktop
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D4401
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To:
mart added a comment.
what about changing the default to true but still have the binding?
REPOSITORY
R242 Plasma Framework (Library)
REVISION DETAIL
https://phabricator.kde.org/D4473
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: subdiff, #plasma,
davidedmundson added inline comments.
INLINE COMMENTS
> framesvg.cpp:136
>
> -FrameData *fd = d->frames[d->prefix];
> -
> -const QString oldKey = d->cacheId(fd, d->prefix);
> -const EnabledBorders oldBorders = fd->enabledBorders;
> -fd->enabledBorders = borders;
> -const
davidedmundson added inline comments.
INLINE COMMENTS
> mart wrote in framesvg.cpp:136
> it has pendingEnabledBorders because right now the borders are saved only n
> the frame, that we don't know if we can keep it or we'll have to throw it
> away ( or just dereference because some other
mart updated this revision to Diff 11006.
mart added a comment.
- just use enabledBorders to return as a property
REPOSITORY
R242 Plasma Framework (Library)
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D4414?vs=10861=11006
BRANCH
arcpatch-D4414
REVISION DETAIL
mart accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R242 Plasma Framework (Library)
REVISION DETAIL
https://phabricator.kde.org/D4473
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: subdiff, #plasma, hein, mart
Cc:
This revision was automatically updated to reflect the committed changes.
Closed by commit R120:605fb9acd867: [Calculator Runner] Port non-Qalculate
codepath from QtScript to QtQml (authored by broulik).
REPOSITORY
R120 Plasma Workspace
CHANGES SINCE LAST UPDATE
Hey,
One of the topics that were put in the Etherpad for the Plasma sprint is the
following:
- get new look: remove it from 5.8? users complain that breaks their
system as it doesn't support dependencies in 5.8 (deps can't be backported
either,as would bump frameworks requirement); now we
GENERAL INFO
BUILD UNSTABLE
Build URL:
https://build.kde.org/job/plasma-workspace%20Plasma-5.9%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/49/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Tue, 07 Feb 2017 11:13:08 +
Build duration: 6 min 5 sec
CHANGE SET
Revision
This revision was automatically updated to reflect the committed changes.
Closed by commit R119:5df635d80464: Make it possible to change icon themes from
elsewhere in kns (authored by apol).
REPOSITORY
R119 Plasma Desktop
CHANGES SINCE LAST UPDATE
hein created this revision.
hein added reviewers: Plasma, broulik, davidedmundson.
hein added a subscriber: plasma-devel.
Restricted Application added a project: Plasma.
REVISION SUMMARY
This is an attempt at a lazy fix for bug 375871, fixing a regression
from
broulik added a comment.
Also, shouldn't any kind of drag raise windows, not just URL drops? I could
for instance want to drag a color from one Inkscape palette to another
INLINE COMMENTS
> MouseHandler.qml:69
>
> +<<< Updated upstream
> +===
conflict markers
>
subdiff added a comment.
The scroll indicators **are** the frame component in the Breeze style. So
they get replaced in any other style by the frame specified there. If you take
a look at ScrollViewStyle.qml, the indicators are inside the frame component
and the frame sides just change
davidedmundson added a comment.
In https://phabricator.kde.org/D4416#83780, @cfeck wrote:
> QApplication::desktopFileName() was only added in Qt 5.7, while frameworks
still support Qt 5.6. Please add a Qt version check around the new code.
Done, thanks
REPOSITORY
R289
GENERAL INFO
BUILD SUCCESS
Build URL:
https://build.kde.org/job/kde-gtk-config%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/44/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Tue, 07 Feb 2017 10:13:48 +
Build duration: 1 min 45 sec
CHANGE SET
Revision
hein added a comment.
IIUI this means scroll indicators are now disabled by default unless a UI
sets frameVisible: true? Marco, is this what you +1'd?
REPOSITORY
R242 Plasma Framework (Library)
REVISION DETAIL
https://phabricator.kde.org/D4473
EMAIL PREFERENCES
broulik added a comment.
-1
I don't get the overflow indicator mark anywhere anymore since frameVisible
is `false` by default for Plasma ScrollView.
REPOSITORY
R242 Plasma Framework (Library)
REVISION DETAIL
https://phabricator.kde.org/D4473
EMAIL PREFERENCES
subdiff added a comment.
Note: This patch changes the default. Since until now the frameVisible
property wasn't evaluated the frames were always shown.
But since QtQuickControls ScrollView (i.e. Plasma Extra Components ScrollArea
aswell) has set frameVisible to false by default, they
subdiff updated this revision to Diff 11003.
subdiff added a comment.
Set ScrollArea's frameVisible default to true additional in order to not
change current implementations.
REPOSITORY
R242 Plasma Framework (Library)
CHANGES SINCE LAST UPDATE
elvisangelaccio added a comment.
Another possible fix would be to add an empty QUrl to the ark mimeData, i.e.
fixing it from the ark side.
Question is: are we sure ark is the only application affected by the
regression?
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129932/#review102441
---
I agree with Marco.
Furthermore, introducing special
guoyunhe closed this revision.
REVISION DETAIL
https://phabricator.kde.org/D3009
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: guoyunhe, #plasma_workspaces, mart
Cc: davidedmundson, mart, plasma-devel, lesliezhai, ali-mohamed,
jensreuterberg, abetts,
This revision was automatically updated to reflect the committed changes.
Closed by commit R242:eb39b3514130: [ScrollViewStyle] Evaluate frameVisible
property (authored by subdiff).
REPOSITORY
R242 Plasma Framework (Library)
CHANGES SINCE LAST UPDATE
davidedmundson updated this revision to Diff 11013.
davidedmundson added a comment.
Trying again to make phab show the same thing
REPOSITORY
R119 Plasma Desktop
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D4476?vs=11012=11013
BRANCH
master
REVISION DETAIL
broulik added inline comments.
INLINE COMMENTS
> broulik wrote in MouseHandler.qml:135
> For `onDrop` the URL check is fine. There we would launch the respective
> application and send the URL to it
Nevermind. You did the right thing, and I was already wondering why my comment
showed up on
broulik added a comment.
+1
INLINE COMMENTS
> MouseHandler.qml:125
> onDrop: {
> // Reject internal drops.
> +if (event.mimeData.formats.indexOf("text/x-plasmoidservicename")
> >= 0) {
// Reject plasmoid drops, so the containment can handle them.
davidedmundson updated this revision to Diff 11012.
davidedmundson added a comment.
Fix stupid phab getting itself confused
REPOSITORY
R119 Plasma Desktop
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D4476?vs=11011=11012
BRANCH
master
REVISION DETAIL
broulik created this revision.
broulik added a reviewer: Plasma.
broulik set the repository for this revision to R120 Plasma Workspace.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
Take into account the label's height
davidedmundson updated this revision to Diff 11011.
davidedmundson added a comment.
update
REPOSITORY
R119 Plasma Desktop
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D4476?vs=11008=11011
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D4476
AFFECTED FILES
davidedmundson abandoned this revision.
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D4476
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: davidedmundson, #plasma
Cc: broulik, plasma-devel, lesliezhai, ali-mohamed,
Plasma 5.9.1 is out now with a week's worth of updates
https://www.kde.org/announcements/plasma-5.9.1.php
broulik added a comment.
> doesn't this rather suggests more than iconsizes.small
Yes, it needs to, since below the icon you always get a label which we didn't
take into account previously.
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D4477
EMAIL
subdiff added a comment.
You mean in Plasma's ScrollArea? Would be ok for me. It's a rather random
deviation from upstream's ScrollView though.
Is this ok for anyone else aswell?
REPOSITORY
R242 Plasma Framework (Library)
REVISION DETAIL
https://phabricator.kde.org/D4473
EMAIL
hein added a comment.
Is there some danger in this when using the component with a different theme
that has visible frames?
Should the scroll indicators be tied to frame visibility at all? That seems
dubious to me.
REPOSITORY
R242 Plasma Framework (Library)
REVISION DETAIL
GENERAL INFO
BUILD UNSTABLE
Build URL:
https://build.kde.org/job/plasma-desktop%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/599/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Tue, 07 Feb 2017 11:48:44 +
Build duration: 18 min
CHANGE SET
Revision
cfeck added a comment.
QApplication::desktopFileName() was only added in Qt 5.7, while frameworks
still support Qt 5.6. Please add a Qt version check around the new code.
REPOSITORY
R289 KNotifications
REVISION DETAIL
https://phabricator.kde.org/D4416
EMAIL PREFERENCES
hein added a comment.
Aye. I think that's weird, but I'm guessing the Breeze style had no choice
there.
Does defaulting the frames to on mean the bottom flash is back too or is that
still fixed?
Can't comment on the ramifications of defaulting it to on + other themes ...
davidedmundson created this revision.
davidedmundson added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
A previous patch made the task manager only accept drag events which
contained URL lists or
mart added a comment.
not sure about this..
doesn't this rather suggests more than iconsizes.small (that semantically
should be the smallest icon that makes sense) isn't scaled correctly for the
high dpi?
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
> On Feb. 7, 2017, 9:26 a.m., Marco Martin wrote:
> > not sure is a good idea.
> > the tooltips not dismissing anymore if hovered may become quite annoying,
> > which is usually the main complaint on our tooltips.
Well, the tooltip shows the load and frequency on all CPUs and memory
broulik added inline comments.
INLINE COMMENTS
> MouseHandler.qml:135
> -
> -if (event.mimeData.hasUrls) {
> -parent.urlsDropped(event.mimeData.urls);
For `onDrop` the URL check is fine. There we would launch the respective
application and send the URL to it
GENERAL INFO
BUILD UNSTABLE
Build URL:
https://build.kde.org/job/plasma-workspace%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/727/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Tue, 07 Feb 2017 16:07:28 +
Build duration: 6 min 3 sec
CHANGE SET
Revision
GENERAL INFO
BUILD UNSTABLE
Build URL:
https://build.kde.org/job/plasma-workspace%20Plasma-5.9%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/50/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Tue, 07 Feb 2017 16:07:02 +
Build duration: 14 min
CHANGE SET
Revision
This revision was automatically updated to reflect the committed changes.
Closed by commit R120:7b1dc9a4bb03: make services disqualification much
stricter (authored by sitter).
REPOSITORY
R120 Plasma Workspace
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D4415?vs=10865=11015
hein accepted this revision.
hein added a reviewer: hein.
This revision is now accepted and ready to land.
REPOSITORY
R119 Plasma Desktop
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D4476
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To:
broulik created this revision.
broulik added reviewers: Plasma, apol.
broulik set the repository for this revision to R224 KDE Connect.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
Also ensure that it wraps instead of
davidedmundson accepted this revision.
davidedmundson added a reviewer: davidedmundson.
This revision is now accepted and ready to land.
REPOSITORY
R114 Plasma Addons
REVISION DETAIL
https://phabricator.kde.org/D4479
EMAIL PREFERENCES
elvisangelaccio updated this revision to Diff 11019.
elvisangelaccio added a comment.
- Rebased
REPOSITORY
R120 Plasma Workspace
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D4480?vs=11018=11019
BRANCH
bug/376131
REVISION DETAIL
https://phabricator.kde.org/D4480
AFFECTED
broulik created this revision.
broulik added a reviewer: Plasma.
broulik set the repository for this revision to R114 Plasma Addons.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
Allows to cancel picking with right
This revision was automatically updated to reflect the committed changes.
Closed by commit R114:55b6011d3b02: [Color Picker] Pick color only on left
click (authored by broulik).
REPOSITORY
R114 Plasma Addons
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D4479?vs=11016=11017
elvisangelaccio created this revision.
elvisangelaccio added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
Commit
https://phabricator.kde.org/R120:72e39f083a103e0887624482d6a0ce8b0556dd5e broke
the
graesslin added inline comments.
INLINE COMMENTS
> switchuserdialog.cpp:171
> +{
> +setKeyboardGrabEnabled(false);
> +}
I highly suggest to explicitly xcb_flush here. Otherwise your request to ungrab
the keyboard is racing with your DBus call to screenlocker. There is a chance
that the
mart added a comment.
with this on top of https://codereview.qt-project.org/#/c/184278/ the
situation with Qt 5.8 is definitely improved
REPOSITORY
R108 KWin
REVISION DETAIL
https://phabricator.kde.org/D4482
EMAIL PREFERENCES
broulik updated the summary for this revision.
broulik updated this revision to Diff 11026.
broulik added a comment.
- For launchers show all places. There the menu is less likely to become
overly large as there's no window controls in this case.
I looked into KActivitiesStats or
This revision was automatically updated to reflect the committed changes.
Closed by commit R120:f0eb59ed9314: [Icon Applet] More sensible minimum height
(authored by broulik).
REPOSITORY
R120 Plasma Workspace
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D4477?vs=11010=11031
anthonyfieroni added a comment.
Workaround cannot be made, Qt 5.8 counts internal messages e.g.
MouseEventPress internal wants MouseEventRelease intenal, our message can't
mess counting. Will be good when Qt bug is closed :)
REPOSITORY
R31 Breeze
REVISION DETAIL
graesslin created this revision.
graesslin added reviewers: KWin, Plasma.
Restricted Application added a project: KWin.
Restricted Application added subscribers: kwin, plasma-devel.
REVISION SUMMARY
Increases minimum Qt version to 5.7. This allows to drop the pre-5.7
virtual keyboard and
bshah accepted this revision.
bshah added a reviewer: bshah.
This revision is now accepted and ready to land.
REPOSITORY
R108 KWin
BRANCH
require-qt-5.7
REVISION DETAIL
https://phabricator.kde.org/D4485
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To:
davidedmundson accepted this revision.
davidedmundson added a reviewer: davidedmundson.
This revision is now accepted and ready to land.
REPOSITORY
R108 KWin
BRANCH
plasmasurface-before-shell-surface-5.9
REVISION DETAIL
https://phabricator.kde.org/D4482
EMAIL PREFERENCES
This revision was automatically updated to reflect the committed changes.
Closed by commit R108:8edd0336e67d: Support creation of PlasmaShellSurface
prior to ShellSurface (authored by graesslin).
REPOSITORY
R108 KWin
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D4482?vs=11022=11032
1 - 100 of 119 matches
Mail list logo