D16417: Improve replacement text when action description is not provided

ngraham accepted this revision. ngraham added a comment. This revision is now accepted and ready to land. Thanks, this works. INLINE COMMENTS > bruns wrote in AuthDialog.cpp:359 > Btw, "not provided" is your own proposal ;-) What!? I must be going senile already... REPOSITORY R121

D12498: Fully remove `Application Name` from Details panel

ngraham accepted this revision. ngraham added a comment. This revision is now accepted and ready to land. Thanks! REPOSITORY R121 Policykit (Polkit) KDE Agent BRANCH arcpatch-D12498 REVISION DETAIL https://phabricator.kde.org/D12498 To: bruns, ngraham, davidedmundson, sharvey Cc:

D16365: share common values for both Breeze and Breeze-dark GTK themes

ngraham accepted this revision. ngraham added a subscriber: grmat. ngraham added a comment. This revision is now accepted and ready to land. I haven't noticed any problems running with this over the past few days. It's unfortunate that neither the maintainer (@jackg) nor anyone else familiar

[Powerdevil] [Bug 304696] Display is dimmed in half the time you configure to dim

https://bugs.kde.org/show_bug.cgi?id=304696 Chris Holland changed: What|Removed |Added CC||zrenf...@gmail.com --- Comment #14 from Chris

D16419: [kickoff] Wordwrap long sentence in config

davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. davidedmundson requested review of this revision. REVISION SUMMARY Text should always be constrained and should almost always elide or wrap.

D16417: Improve replacement text when action description is not provided

bruns added inline comments. INLINE COMMENTS > ngraham wrote in AuthDialog.cpp:359 > I would prefer "not available" over "not provided", but I won't choose this > as my hill to die on. :) > > Do we actually need to repeat the word 'Description' here? Btw, "not provided" is your own proposal

D15885: Clean UI file for `details` portion of UI

bruns abandoned this revision. bruns added a comment. Folded back into D12498 REPOSITORY R121 Policykit (Polkit) KDE Agent REVISION DETAIL https://phabricator.kde.org/D15885 To: bruns, ngraham, davidedmundson, sharvey Cc: bruns, plasma-devel,

D16403: Add services and appstream runner debug categories to categories file

This revision was automatically updated to reflect the committed changes. Closed by commit R120:17a0d34d0e83: Add services and appstream runner debug categories to categories file (authored by bruns). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE

D16417: Improve replacement text when action description is not provided

bruns added inline comments. INLINE COMMENTS > ngraham wrote in AuthDialog.cpp:356 > So I gather `.isEmpty()` works now? Yes. > ngraham wrote in AuthDialog.cpp:359 > I would prefer "not available" over "not provided", but I won't choose this > as my hill to die on. :) > > Do we actually need

D12498: Fully remove `Application Name` from Details panel

bruns updated this revision to Diff 44184. bruns added a comment. Update UI file Split out "description" change REPOSITORY R121 Policykit (Polkit) KDE Agent CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D12498?vs=42375=44184 BRANCH arcpatch-D12498 REVISION DETAIL

D16417: Improve replacement text when action description is not provided

ngraham added inline comments. INLINE COMMENTS > AuthDialog.cpp:356 > -// replace blank description > -// neither isEmpty() or isNull() worked (?) > -if (actionDescription.description() == "") { So I gather `.isEmpty()` works now? > AuthDialog.cpp:359 >

D16417: Improve replacement text when action description is not provided

bruns created this revision. bruns added reviewers: Plasma, ngraham, davidedmundson. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY The polkit rules specification mandates a description for actions,

D16327: Update libdbusmenuqt folder

This revision was not accepted when it landed; it landed in state "Needs Revision". This revision was automatically updated to reflect the committed changes. Closed by commit R884:bb680e8dbfc2: Update libdbusmenuqt folder (authored by Zren). REPOSITORY R884 Active Window Control Applet for

D16394: Hide Buttons With Empty Text (Gtk Separators)

This revision was automatically updated to reflect the committed changes. Closed by commit R884:942169541d33: Hide Buttons With Empty Text (Gtk Separators) (authored by Zren). REPOSITORY R884 Active Window Control Applet for Plasma CHANGES SINCE LAST UPDATE

D15206: [Kickoff] Add a subtle separator line between the header and the content view

ngraham added a comment. No, the blue highlight doesn't touch, just like people requested. It looks just fine now. If we want to make the blue highlight touch the bounds of its whole view, that would be a significant design change that would need to be made elsewhere (if indeed it would be

D15855: Port Kickoff to shared view component using single-MouseArea pattern

ngraham added a comment. Oops, sorry, I forgot to change my status! REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D15855 To: hein, #plasma, ngraham, davidedmundson Cc: davidedmundson, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed,

D16212: [Device Notifier] Add a button to unmount all devices

ngraham added a comment. Ping! Can we get a review from a #plasma person? REPOSITORY R120 Plasma Workspace BRANCH arc_unmountall (branched from master) REVISION DETAIL https://phabricator.kde.org/D16212 To: thsurrel, #plasma, #vdg, ngraham

D16410: [Bookmarks Runner] Avoid leaking FetchSqlite instances

bruns created this revision. bruns added a reviewer: Frameworks. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY The favicon and places instances are created in the prepare() call, leaking the instance from the

D16409: [Bookmarks Runner] Open database connection in the query thread

bruns created this revision. bruns added a reviewer: Frameworks. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY QSqlDatabase connection instances are global reference counted objects which can be crated using

D16405: Add debug category for bookmarks runner

bruns added a comment. Only applies cleanly after D16403 . REPOSITORY R120 Plasma Workspace BRANCH master REVISION DETAIL https://phabricator.kde.org/D16405 To: bruns, #frameworks, davidedmundson Cc: plasma-devel, ragreen, Pitel, ZrenBot,

D16405: Add debug category for bookmarks runner

bruns added a comment. Depends on the cleanup in D16404 , as the autogenerated `bookmarks_debug.cpp` is not linked to the tests/testChromeBookmarks binary otherwise. REPOSITORY R120 Plasma Workspace BRANCH master REVISION DETAIL

D16327: Update libdbusmenuqt folder

Zren added a comment. For now, I'm going to merge this so I can keep the widget in sync with the global menu widget. We can revisit linking to a shared library later on. REPOSITORY R884 Active Window Control Applet for Plasma REVISION DETAIL https://phabricator.kde.org/D16327 To: Zren

D16405: Add debug category for bookmarks runner

bruns created this revision. bruns added a reviewer: Frameworks. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY Dependency for followup patches, no functional change TEST PLAN make REPOSITORY R120 Plasma

D16402: Cleanup - remove unnecessary QIcon include

This revision was automatically updated to reflect the committed changes. Closed by commit R120:9945aecaf2eb: Cleanup - remove unnecessary QIcon include (authored by bruns). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D16402?vs=44159=44163

D16404: [Bookmarks Runner] Cleanup tests CMakeList

bruns created this revision. bruns added a reviewer: Frameworks. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY Build the test dependencies as static library. Remove duplicate and commented out lines. Use

D16403: Add services and appstream runner debug categories to categories file

bruns created this revision. bruns added a reviewer: Frameworks. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY Make the debug categories visible in kdebugsettings REPOSITORY R120 Plasma Workspace BRANCH

D16402: Cleanup - remove unnecessary QIcon include

bruns created this revision. bruns added a reviewer: Frameworks. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY QIcon is only used in the implementation, not in the interface. TEST PLAN compile tested, no

[Powerdevil] [Bug 304696] Display is dimmed in half the time you configure to dim

https://bugs.kde.org/show_bug.cgi?id=304696 S changed: What|Removed |Added CC||sb56...@gmail.com --- Comment #13 from S --- Created

D15616: [Comic] Handle error state correctly

This revision was automatically updated to reflect the committed changes. Closed by commit R114:7c7c00798048: [comicprovider] Restart timer on redirect request (authored by anthonyfieroni). REPOSITORY R114 Plasma Addons CHANGES SINCE LAST UPDATE

D16400: Prefer to set current mode in resolution combobox

volkov created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY Preferred resolution may be lower than current resolution (e.g. on KVM), in this case it will be selected in the combobox instead

D14796: Port Keyboard input modules and Expand scope to cover input method configuration

lavender added a comment. This redesign seems like a good place to start thinking about integrating accessibility options T with the rest of the settings. I included some screenshots of what OSX does (Switch Control

D7038: [server] Respect input region of sub-surfaces on pointer surface focus

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit R127:24bdc06747ed: [server] Respect input region of sub-surfaces on pointer surface focus (authored by romangg).

D7038: [server] Respect input region of sub-surfaces on pointer surface focus

romangg updated this revision to Diff 44149. romangg added a comment. Rebase on master. REPOSITORY R127 KWayland CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7038?vs=42733=44149 BRANCH inputRegionSubSurfaces REVISION DETAIL https://phabricator.kde.org/D7038 AFFECTED

D15855: Port Kickoff to shared view component using single-MouseArea pattern

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit R119:48b32259e73f: Port Kickoff to shared view component using single-MouseArea pattern (authored by hein). REPOSITORY

D16368: Swap Trash/Delete when Shift is used while the menu is open

This revision was automatically updated to reflect the committed changes. Closed by commit R119:c4285ed1b407: Swap Trash/Delete when Shift is used while the menu is open (authored by hein). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE

KDE CI: Plasma » systemsettings » stable-kf5-qt5 FreeBSDQt5.11 - Build # 10 - Fixed!

BUILD SUCCESS Build URL https://build.kde.org/job/Plasma/job/systemsettings/job/stable-kf5-qt5%20FreeBSDQt5.11/10/ Project: stable-kf5-qt5 FreeBSDQt5.11 Date of build: Wed, 24 Oct 2018 06:16:31 + Build duration: 27 min and counting

D15616: [Comic] Handle error state correctly

anthonyfieroni added a comment. @davidedmundson to commit or you have other objections? REPOSITORY R114 Plasma Addons REVISION DETAIL https://phabricator.kde.org/D15616 To: anthonyfieroni, davidedmundson, #plasma, broulik Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai,