[Differential] [Request, 18 lines] D4470: [User Switcher] Ungrab keyboard before trying to lock the screen

2017-02-07 Thread Kai Uwe Broulik
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

Jenkins-kde-ci: plasma-tests Plasma-5.9 stable-kf5-qt5 » Linux,gcc - Build # 3 - Still Unstable!

2017-02-07 Thread no-reply
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

[Differential] [Request, 2 lines] D4473: [ScrollViewStyle] Evaluate frameVisible property

2017-02-07 Thread Roman Gilg
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.

Jenkins-kde-ci: plasma-desktop Plasma-5.9 stable-kf5-qt5 » Linux,gcc - Build # 53 - Still Unstable!

2017-02-07 Thread no-reply
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

[Differential] [Commented On] D3578: [RFC] Implement drag from free space also for QtQuickControls

2017-02-07 Thread Anthony Fieroni
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,

[Differential] [Commented On] D3210: make scrollbar size configurable

2017-02-07 Thread Hugo Pereira Da Costa
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

Re: Review Request 129917: Add a cache monitor to the System Load Viewer applet

2017-02-07 Thread Pascal VITOUX
--- 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

[Differential] [Accepted] D4471: Fix KCModule::setAuthAction error checking

2017-02-07 Thread Marco Martin
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:

[Differential] [Updated, 7 lines] D4293: [Task Manager] Show "Mark applications that play audio" option only if plasma-pa is available

2017-02-07 Thread Kai Uwe Broulik
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

Re: Review Request 129932: Make tooltip interactive.

2017-02-07 Thread Marco Martin
--- 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

[Differential] [Accepted] D4472: [Calculator Runner] Port non-Qalculate codepath from QtScript to QtQml

2017-02-07 Thread David Edmundson
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

[Differential] [Request, 23 lines] D4469: Tweak alphabetic (default) sort behavior.

2017-02-07 Thread Eike Hein
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

[Differential] [Request, 2 lines] D4471: Fix KCModule::setAuthAction error checking

2017-02-07 Thread David Edmundson
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

[Differential] [Closed] D4416: Send desktopfilename as part of notifyByPopup hints

2017-02-07 Thread David Edmundson
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

Jenkins-kde-ci: plasma-workspace master kf5-qt5 » Linux,gcc - Build # 725 - Still Unstable!

2017-02-07 Thread no-reply
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

Jenkins-kde-ci: plasma-workspace Plasma-5.9 stable-kf5-qt5 » Linux,gcc - Build # 48 - Still Unstable!

2017-02-07 Thread no-reply
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

[Differential] [Accepted] D4293: [Task Manager] Show "Mark applications that play audio" option only if plasma-pa is available

2017-02-07 Thread Eike Hein
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,

[Differential] [Closed] D4293: [Task Manager] Show "Mark applications that play audio" option only if plasma-pa is available

2017-02-07 Thread Kai Uwe Broulik
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

[Differential] [Commented On] D3578: [RFC] Implement drag from free space also for QtQuickControls

2017-02-07 Thread Hugo Pereira Da Costa
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

[Differential] [Accepted] D4470: [User Switcher] Ungrab keyboard before trying to lock the screen

2017-02-07 Thread David Edmundson
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

Re: Review Request 129917: Add a cache monitor to the System Load Viewer applet

2017-02-07 Thread Sebastian Kügler
--- 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

[Differential] [Closed] D4470: [User Switcher] Ungrab keyboard before trying to lock the screen

2017-02-07 Thread Kai Uwe Broulik
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

[Differential] [Commented On] D3578: [RFC] Implement drag from free space also for QtQuickControls

2017-02-07 Thread David Rosca
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:

[Differential] [Request, 9 lines] D4472: [Calculator Runner] Port non-Qalculate codepath from QtScript to QtQml

2017-02-07 Thread Kai Uwe Broulik
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

[Differential] [Accepted] D4474: [Solid Device Actions KCM] Encode action file name

2017-02-07 Thread David Edmundson
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

[Differential] [Closed] D4474: [Solid Device Actions KCM] Encode action file name

2017-02-07 Thread Kai Uwe Broulik
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

Jenkins-kde-ci: plasma-desktop master kf5-qt5 » Linux,gcc - Build # 598 - Still Unstable!

2017-02-07 Thread no-reply
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

[Differential] [Abandoned] D4475: White list drags from Ark.

2017-02-07 Thread Eike Hein
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,

[Differential] [Commented On] D4414: don't regenerate frames when setting every property

2017-02-07 Thread Marco Martin
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

Jenkins-kde-ci: plasma-desktop Plasma-5.9 stable-kf5-qt5 » Linux,gcc - Build # 54 - Still Unstable!

2017-02-07 Thread no-reply
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

[Differential] [Closed] D4288: Add Kleopatra tray icon

2017-02-07 Thread Yunhe Guo
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,

Jenkins-kde-ci: plasma-tests Plasma-5.9 stable-kf5-qt5 » Linux,gcc - Build # 4 - Still Unstable!

2017-02-07 Thread no-reply
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

Jenkins-kde-ci: plasma-desktop Plasma-5.9 stable-kf5-qt5 » Linux,gcc - Build # 55 - Still Unstable!

2017-02-07 Thread no-reply
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

Jenkins-kde-ci: plasma-desktop master kf5-qt5 » Linux,gcc - Build # 600 - Still Unstable!

2017-02-07 Thread no-reply
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

[Differential] [Closed] D4414: don't regenerate frames when setting every property

2017-02-07 Thread Marco Martin
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

[Differential] [Commented On] D4473: [ScrollViewStyle] Evaluate frameVisible property

2017-02-07 Thread Roman Gilg
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

Jenkins-kde-ci: plasma-desktop master kf5-qt5 » Linux,gcc - Build # 597 - Still Unstable!

2017-02-07 Thread no-reply
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

Jenkins-kde-ci: kde-gtk-config master kf5-qt5 » Linux,gcc - Build # 44 - Fixed!

2017-02-07 Thread no-reply
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

[Differential] [Request, 3 lines] D4474: [Solid Device Actions KCM] Encode action file name

2017-02-07 Thread Kai Uwe Broulik
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.

Jenkins-kde-ci: plasma-workspace master kf5-qt5 » Linux,gcc - Build # 726 - Still Unstable!

2017-02-07 Thread no-reply
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

[Differential] [Accepted] D4401: Pressing Enter should open submenus in Kicker

2017-02-07 Thread Eike Hein
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:

[Differential] [Commented On] D4473: [ScrollViewStyle] Evaluate frameVisible property

2017-02-07 Thread Marco Martin
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,

[Differential] [Changed Subscribers] D4414: don't regenerate frames when setting every property

2017-02-07 Thread David Edmundson
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

[Differential] [Commented On] D4414: don't regenerate frames when setting every property

2017-02-07 Thread David Edmundson
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

[Differential] [Updated, 623 lines] D4414: don't regenerate frames when setting every property

2017-02-07 Thread Marco Martin
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

[Differential] [Accepted] D4473: [ScrollViewStyle] Evaluate frameVisible property

2017-02-07 Thread Marco Martin
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:

[Differential] [Closed] D4472: [Calculator Runner] Port non-Qalculate codepath from QtScript to QtQml

2017-02-07 Thread Kai Uwe Broulik
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

Look and Feel in 5.8

2017-02-07 Thread Sebastian Kügler
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

Jenkins-kde-ci: plasma-workspace Plasma-5.9 stable-kf5-qt5 » Linux,gcc - Build # 49 - Still Unstable!

2017-02-07 Thread no-reply
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

[Differential] [Closed] D4317: Make it possible to change icon themes from elsewhere in kns

2017-02-07 Thread Aleix Pol Gonzalez
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

[Differential] [Request, 23 lines] D4475: White list drags from Ark.

2017-02-07 Thread Eike Hein
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

[Differential] [Updated] D4475: White list drags from Ark.

2017-02-07 Thread Kai Uwe Broulik
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 >

[Differential] [Commented On] D4473: [ScrollViewStyle] Evaluate frameVisible property

2017-02-07 Thread Roman Gilg
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

[Differential] [Commented On] D4416: Send desktopfilename as part of notifyByPopup hints

2017-02-07 Thread David Edmundson
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

Jenkins-kde-ci: kde-gtk-config master kf5-qt5 » Linux,gcc - Build # 44 - Fixed!

2017-02-07 Thread no-reply
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

[Differential] [Updated] D4473: [ScrollViewStyle] Evaluate frameVisible property

2017-02-07 Thread Eike Hein
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

[Differential] [Commented On] D4473: [ScrollViewStyle] Evaluate frameVisible property

2017-02-07 Thread Kai Uwe Broulik
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

[Differential] [Commented On] D4473: [ScrollViewStyle] Evaluate frameVisible property

2017-02-07 Thread Roman Gilg
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

[Differential] [Updated, 3 lines] D4473: [ScrollViewStyle] Evaluate frameVisible property

2017-02-07 Thread Roman Gilg
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

[Differential] [Commented On] D4475: White list drags from Ark.

2017-02-07 Thread Elvis Angelaccio
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

Re: Review Request 129932: Make tooltip interactive.

2017-02-07 Thread Sebastian Kügler
--- 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

[Differential] [Closed] D3009: Fix logout translation message

2017-02-07 Thread Yunhe Guo
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,

[Differential] [Closed] D4473: [ScrollViewStyle] Evaluate frameVisible property

2017-02-07 Thread Roman Gilg
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

[Differential] [Updated, 13 lines] D4476: Reverse TaskManager DragDrop to blacklist Plasma instead of whitelisting URLs

2017-02-07 Thread David Edmundson
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

[Differential] [Commented On] D4476: Reverse TaskManager DragDrop to blacklist Plasma instead of whitelisting URLs

2017-02-07 Thread Kai Uwe Broulik
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

[Differential] [Commented On] D4476: Reverse TaskManager DragDrop to blacklist Plasma instead of whitelisting URLs

2017-02-07 Thread Kai Uwe Broulik
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.

[Differential] [Updated, 13 lines] D4476: Reverse TaskManager DragDrop to blacklist Plasma instead of whitelisting URLs

2017-02-07 Thread David Edmundson
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

[Differential] [Request, 2 lines] D4477: [Icon Applet] More sensible minimum height

2017-02-07 Thread Kai Uwe Broulik
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

[Differential] [Updated, 9 lines] D4476: Reverse TaskManager DragDrop to blacklist Plasma instead of whitelisting URLs

2017-02-07 Thread David Edmundson
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

[Differential] [Abandoned] D4476: Reverse TaskManager DragDrop to blacklist Plasma instead of whitelisting URLs

2017-02-07 Thread David Edmundson
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

2017-02-07 Thread Jonathan Riddell
Plasma 5.9.1 is out now with a week's worth of updates https://www.kde.org/announcements/plasma-5.9.1.php

[Differential] [Commented On] D4477: [Icon Applet] More sensible minimum height

2017-02-07 Thread Kai Uwe Broulik
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

[Differential] [Commented On] D4473: [ScrollViewStyle] Evaluate frameVisible property

2017-02-07 Thread Roman Gilg
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

[Differential] [Commented On] D4473: [ScrollViewStyle] Evaluate frameVisible property

2017-02-07 Thread Eike Hein
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

Jenkins-kde-ci: plasma-desktop master kf5-qt5 » Linux,gcc - Build # 599 - Still Unstable!

2017-02-07 Thread no-reply
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

[Differential] [Commented On] D4416: Send desktopfilename as part of notifyByPopup hints

2017-02-07 Thread Christoph Feck
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

[Differential] [Commented On] D4473: [ScrollViewStyle] Evaluate frameVisible property

2017-02-07 Thread Eike Hein
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 ...

[Differential] [Request, 21 lines] D4476: Reverse TaskManager DragDrop to blacklist Plasma instead of whitelisting URLs

2017-02-07 Thread David Edmundson
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

[Differential] [Commented On] D4477: [Icon Applet] More sensible minimum height

2017-02-07 Thread Marco Martin
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

Re: Review Request 129932: Make tooltip interactive.

2017-02-07 Thread Garry Williams
> 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

[Differential] [Changed Subscribers] D4476: Reverse TaskManager DragDrop to blacklist Plasma instead of whitelisting URLs

2017-02-07 Thread Kai Uwe Broulik
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

Jenkins-kde-ci: plasma-workspace master kf5-qt5 » Linux,gcc - Build # 727 - Still Unstable!

2017-02-07 Thread no-reply
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

Jenkins-kde-ci: plasma-workspace Plasma-5.9 stable-kf5-qt5 » Linux,gcc - Build # 50 - Still Unstable!

2017-02-07 Thread no-reply
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

[Differential] [Closed] D4415: make services disqualification much stricter

2017-02-07 Thread Harald Sitter
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

[Differential] [Accepted] D4476: Reverse TaskManager DragDrop to blacklist Plasma instead of whitelisting URLs

2017-02-07 Thread Eike Hein
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:

[Differential] [Request, 2 lines] D4478: [Applet] Make "no device available" heading consistent with other applets

2017-02-07 Thread Kai Uwe Broulik
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

[Differential] [Accepted] D4479: [Color Picker] Pick color only on left click

2017-02-07 Thread David Edmundson
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

[Differential] [Updated, 21 lines] D4480: [kioslave/remote] Fix porting bugs

2017-02-07 Thread Elvis Angelaccio
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

[Differential] [Request, 17 lines] D4479: [Color Picker] Pick color only on left click

2017-02-07 Thread Kai Uwe Broulik
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

[Differential] [Closed] D4479: [Color Picker] Pick color only on left click

2017-02-07 Thread Kai Uwe Broulik
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

[Differential] [Request, 23 lines] D4480: [kioslave/remote] Fix porting bugs

2017-02-07 Thread Elvis Angelaccio
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

[Differential] [Commented On] D4470: [User Switcher] Ungrab keyboard before trying to lock the screen

2017-02-07 Thread Martin Gräßlin
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

[Differential] [Commented On] D4482: Support creation of PlasmaShellSurface prior to ShellSurface

2017-02-07 Thread Marco Martin
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

[Differential] [Updated, 139 lines] D3815: [Task Manager] Add "Places" for entries belonging to a file manager

2017-02-07 Thread Kai Uwe Broulik
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

[Differential] [Closed] D4477: [Icon Applet] More sensible minimum height

2017-02-07 Thread Kai Uwe Broulik
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

[Differential] [Commented On] D3578: [RFC] Implement drag from free space also for QtQuickControls

2017-02-07 Thread Anthony Fieroni
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

[Differential] [Request, 81 lines] D4485: Require Qt 5.7 and remove pre-5.7 code

2017-02-07 Thread Martin Gräßlin
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

[Differential] [Accepted] D4485: Require Qt 5.7 and remove pre-5.7 code

2017-02-07 Thread Bhushan Shah
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:

[Differential] [Accepted] D4482: Support creation of PlasmaShellSurface prior to ShellSurface

2017-02-07 Thread David Edmundson
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

[Differential] [Closed] D4482: Support creation of PlasmaShellSurface prior to ShellSurface

2017-02-07 Thread Martin Gräßlin
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   2   >