[Differential] [Closed] D3916: TaskManager: Hide tooltip after activating task

This revision was automatically updated to reflect the committed changes. Closed by commit R119:4c8b2ea3ba88: TaskManager: Hide tooltip after activating task (authored by drosca). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3916?vs=9601&id=9665 REVI

[Differential] [Closed] D3917: AppletConfig: Use correct icon state for category icons

This revision was automatically updated to reflect the committed changes. Closed by commit R119:a54b04dd86c1: AppletConfig: Use correct icon state for category icons (authored by drosca). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3917?vs=9602&id=96

[Differential] [Request, 2 lines] D3917: AppletConfig: Use correct icon state for category icons

drosca created this revision. drosca added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY QIcon::Selected should only be used when current item && active focus. TEST PLAN It now display same icon st

[Differential] [Request, 3 lines] D3916: TaskManager: Hide tooltip after activating task

drosca created this revision. drosca added reviewers: Plasma, hein. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY So that the tooltip doesn't unnecessarily cover activated window. Also under high load it can happen

[Differential] [Closed] D3901: Applet: Toggle mute of items with middle mouse button

This revision was automatically updated to reflect the committed changes. Closed by commit R115:8acf4e6a78c8: Applet: Toggle mute of items with middle mouse button (authored by drosca). REPOSITORY R115 Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3901?vs=

[Differential] [Closed] D3900: Set mute when changing volume to 0 with slider

This revision was automatically updated to reflect the committed changes. Closed by commit R115:db9f523a03c2: Set mute when changing volume to 0 with slider (authored by drosca). REPOSITORY R115 Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3900?vs=9574&id

[Differential] [Closed] D3899: Applet: Make sure to never show empty icon

This revision was automatically updated to reflect the committed changes. Closed by commit R115:26dfed1e5df8: Applet: Make sure to never show empty icon (authored by drosca). REPOSITORY R115 Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3899?vs=9573&id=959

[Differential] [Closed] D3898: Applet: Don't delete delegates while drag is active

This revision was automatically updated to reflect the committed changes. Closed by commit R115:f21aa35e9946: Applet: Don't delete delegates while drag is active (authored by drosca). REPOSITORY R115 Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3898?vs=95

[Differential] [Commented On] D3900: Set mute when changing volume to 0 with slider

drosca added a comment. That's what changing volume with hotkeys is already doing. Also, we show mute icon when volume is 0, so we should also actually mute it. And at last, some hw have leds indicating mute, so this makes it light up when moving the slider to zero. REPOSITORY R115 Plas

[Differential] [Updated, 5 lines] D3904: MediaController: Stop queuedPositionUpdate timer when receiving mpris position change

drosca updated this revision to Diff 9581. drosca added a comment. Also don't call SetPosition when slider value equals position REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3904?vs=9579&id=9581 BRANCH mediacontroller-fix1 (branched from Plasm

[Differential] [Commented On] D3904: MediaController: Stop queuedPositionUpdate timer when receiving mpris position change

drosca added a comment. It's wrong, sorry. Removing the timer.running check from if is correct, but stopping it is not. In that state, it now fixes issue with seeking to 0 but it still seeks to *current* position - so you may hear pause in the playing song. REPOSITORY R120 Plasma Workspace

[Differential] [Request, 3 lines] D3904: MediaController: Stop queuedPositionUpdate timer when receiving mpris position change

drosca created this revision. drosca added reviewers: Plasma, broulik. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Fixes sometimes reseting position to 0 after expanding applet. TEST PLAN No longer resets to start o

[Differential] [Request, 6 lines] D3901: Applet: Toggle mute of items with middle mouse button

drosca created this revision. drosca added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REPOSITORY R115 Plasma Audio Volume Applet BRANCH middle-mute (branched from master) REVISION DETAIL https://phabricator.k

[Differential] [Request, 4 lines] D3900: Set mute when changing volume to 0 with slider

drosca created this revision. drosca added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REPOSITORY R115 Plasma Audio Volume Applet BRANCH mute-slider (branched from master) REVISION DETAIL https://phabricator.k

[Differential] [Request, 12 lines] D3899: Applet: Make sure to never show empty icon

drosca created this revision. drosca added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Streams needs to have icon to be possible to drag them to different devices. Show "unknown" icon when icon is

[Differential] [Request, 2 lines] D3898: Applet: Don't delete delegates while drag is active

drosca created this revision. drosca added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY BUG: 374288 TEST PLAN No crash, delegates are still correctly deleted REPOSITORY R115 Plasma Audio Volume

[Differential] [Closed] D3813: Draw scrollbar as focused when scrollbar itself have focus

This revision was automatically updated to reflect the committed changes. Closed by commit R31:278876855b37: Draw scrollbar as focused when scrollbar itself have focus (authored by drosca). REPOSITORY R31 Breeze CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3813?vs=9367&id=9529 REV

[Differential] [Commented On] D3813: Draw scrollbar as focused when scrollbar itself have focus

drosca added a comment. Currently, it tries to find QAbstractScrollArea in parent chain and use focused state of this scroll area (+ there is a special case for KTextView). There is no standard way to specify the widget that is being scrolled by the scrollbar. This only works with standard Q

[Differential] [Request, 6 lines] D3813: Draw scrollbar as focused when scrollbar itself have focus

drosca created this revision. drosca added reviewers: Plasma, hpereiradacosta. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Fixes case when scrollbar have the scrolling widget as focus proxy. REPOSITORY R31 Breeze B

[Differential] [Closed] D3753: Fix comparing pa_cvolume objects

This revision was automatically updated to reflect the committed changes. Closed by commit R115:825e64edc982: Fix comparing pa_cvolume objects (authored by drosca). REPOSITORY R115 Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3753?vs=9204&id=9210 REVISIO

[Differential] [Request, 2 lines] D3753: Fix comparing pa_cvolume objects

drosca created this revision. drosca added reviewers: Plasma, davidedmundson. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY pa_cvolume_equal returns 1 when true, 0 otherwise BUG: 373943 REPOSITORY R115 Plasma Aud

[Differential] [Accepted] D3739: Correctly initialise and compare pa_cvolume structs

drosca accepted this revision. drosca added a reviewer: drosca. This revision is now accepted and ready to land. REPOSITORY R115 Plasma Audio Volume Applet BRANCH master REVISION DETAIL https://phabricator.kde.org/D3739 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailp

[Differential] [Commented On] D3736: Initialise m_volume in VolumeObject

drosca added a comment. Perhaps it would be better to use `pa_cvolume_init`? In this case it really doesn't matter, but invalid volume value (for `pa_cvolume::values`) is not zero, but `UINT32_MAX`. REPOSITORY R115 Plasma Audio Volume Applet REVISION DETAIL https://phabricator.kde.org/D

[Differential] [Closed] D3268: Applet: Show device icons and active port name

This revision was automatically updated to reflect the committed changes. Closed by commit R115:117ff0b08dea: Applet: Show device icons and active port name (authored by drosca). REPOSITORY R115 Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3268?vs=8025&id

[Differential] [Closed] D3638: Implement drag from free space also for QtQuickControls

This revision was automatically updated to reflect the committed changes. Closed by commit R113:c5f1504d7ce2: Implement drag from free space also for QtQuickControls (authored by drosca). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D3638?vs=8914&id=9035#toc REPOSITORY R113 Oxygen The

[Differential] [Request, 126 lines] D3638: Implement drag from free space also for QtQuickControls

drosca created this revision. drosca added reviewers: Plasma, hpereiradacosta. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Use window manager to handle window move when dragging from free space also with QtQuickContr

[Differential] [Closed] D3585: Fix building library with qmake

This revision was automatically updated to reflect the committed changes. Closed by commit R169:48b9c50a88d4: Fix building library with qmake (authored by drosca). REPOSITORY R169 Kirigami CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3585?vs=8740&id=8876 REVISION DETAIL https://

[Differential] [Closed] D3607: AppletConfiguration: Remove custom handling of moving window

This revision was automatically updated to reflect the committed changes. Closed by commit R119:e9abd83d02b3: AppletConfiguration: Remove custom handling of moving window (authored by drosca). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3607?vs=8809&

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

This revision was automatically updated to reflect the committed changes. Closed by commit R31:bc7304068610: Implement drag from free space also for QtQuickControls (authored by drosca). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D3578?vs=8808&id=8810#toc REPOSITORY R31 Breeze CHAN

[Differential] [Request, 17 lines] D3607: AppletConfiguration: Remove custom handling of moving window

drosca created this revision. drosca added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Breeze now handles moving QtQuickControls windows too. BUG: 352986 REPOSITORY R119 Plasma Desktop BRAN

[Differential] [Updated, 123 lines] D3578: [RFC] Implement drag from free space also for QtQuickControls

drosca updated this revision to Diff 8808. drosca added a comment. "using" instead of defines REPOSITORY R31 Breeze CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3578?vs=8804&id=8808 BRANCH master REVISION DETAIL https://phabricator.kde.org/D3578 AFFECTED FILES kstyle/CM

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

drosca added inline comments. INLINE COMMENTS > hpereiradacosta wrote in breezewindowmanager.cpp:467 > should be "return false" > Comments few lines below says we never eat events. It needs to be `return true`. If we return false here, then we won't get following move and release events. We ca

[Differential] [Updated, 123 lines] D3578: [RFC] Implement drag from free space also for QtQuickControls

drosca updated this revision to Diff 8804. drosca added a comment. Attempt to make it build with KDE4, untested. I don't have KDE4 anymore. REPOSITORY R31 Breeze CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3578?vs=8721&id=8804 BRANCH master REVISION DETAIL https://phabric

[Differential] [Closed] D3572: systemtray: Don't propagate unhandled wheel events

This revision was automatically updated to reflect the committed changes. Closed by commit R120:d1ea67c1fcf0: systemtray: Don't propagate unhandled wheel events (authored by drosca). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3572?vs=8704&id=8803

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

drosca added a comment. In https://phabricator.kde.org/D3578#66969, @hpereiradacosta wrote: > - Also note that you have no guarantee that all QtQuickItems pass through IsQtQuick > - This guy is called only in cases when you need slightly different render path between widgets and quick.

[Differential] [Request, 6 lines] D3585: Fix building library with qmake

drosca created this revision. drosca added a reviewer: Kirigami. Restricted Application added a project: Kirigami. Restricted Application added a subscriber: plasma-devel. TEST PLAN Resulting library from qmake no longer have undefined symbols REPOSITORY R169 Kirigami BRANCH master REVISI

[Differential] [Request, 91 lines] D3578: [RFC] Implement drag from free space also for QtQuickControls

drosca created this revision. drosca added reviewers: Plasma, hpereiradacosta. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Use window manager to handle window move when dragging from free space also with QtQuickContr

[Differential] [Request, 5 lines] D3572: systemtray: Don't propagate unhandled wheel events

drosca created this revision. drosca added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Scrolling over empty space or applet that doesn't handle wheel events no longer activates desktop containment

[Differential] [Closed] D3469: [taskmanagerrulesrc] Add Rewrite Rule for chromium

This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMAWORKSPACE97ff71b5efe0: [taskmanagerrulesrc] Add Rewrite Rule for chromium (authored by drosca). REPOSITORY rPLASMAWORKSPACE Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org

[Differential] [Request, 7 lines] D3469: [taskmanagerrulesrc] Add Rewrite Rule for chromium

drosca created this revision. drosca added reviewers: Plasma, hein. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Chromium also changed its window class to Chromium (was chromium before) TEST PLAN Chromium and its app

[Differential] [Updated, 21 lines] D3268: Applet: Show device icons and active port name

drosca updated this revision to Diff 8025. drosca added a comment. Change label to "PortName (DeviceName)" REPOSITORY rPLASMAPA Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3268?vs=7913&id=8025 BRANCH device-icons (branched from master) REVISION DE

[Differential] [Commented On] D3268: Applet: Show device icons and active port name

drosca added a comment. Updated screenshot: F498804: Spectacle.TJ7708.png REPOSITORY rPLASMAPA Plasma Audio Volume Applet REVISION DETAIL https://phabricator.kde.org/D3268 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferen

[Differential] [Closed] D3269: kcm_keyboard: Change text of config action in tray icon to Configure Layouts

This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMADESKTOPb2e176f15d98: kcm_keyboard: Change text of config action in tray icon to Configure Layouts (authored by drosca). REPOSITORY rPLASMADESKTOP Plasma Desktop CHANGES SINCE LAST UPDATE https:/

[Differential] [Accepted] D3300: [Stream] Expose "corked" property

drosca accepted this revision. drosca added inline comments. This revision is now accepted and ready to land. INLINE COMMENTS > stream.h:82 > quint32 deviceIndex() const; > +bool corked() const; > The getter for `corked` property should be `isCorked()`. REPOSITORY rPLASMAPA Plasma

[Differential] [Abandoned] D3057: Fix drawing QtQuickControls ComboBox popups

drosca abandoned this revision. REPOSITORY rOXYGEN Oxygen Theme REVISION DETAIL https://phabricator.kde.org/D3057 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: drosca, #plasma, broulik, hpereiradacosta Cc: mart, plasma-devel, lesliezhai, ali-mohamed, j

[Differential] [Request, 2 lines] D3269: kcm_keyboard: Change text of config action in tray icon to Configure Layouts

drosca created this revision. drosca added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Also add icon. REPOSITORY rPLASMADESKTOP Plasma Desktop BRANCH master REVISION DETAIL https://phabrica

[Differential] [Commented On] D3268: Applet: Show device icons and active port name

drosca added a comment. F397499: Screenshot_20161105_110504.png F397501: Screenshot_20161105_110557.png REPOSITORY rPLASMAPA Plasma Audio Volume Applet REVISION DETAIL https://phabricator.kde.org/D3268 EMAIL

[Differential] [Updated] D3268: Applet: Show device icons and active port name

drosca added a reviewer: Plasma: Design. REPOSITORY rPLASMAPA Plasma Audio Volume Applet REVISION DETAIL https://phabricator.kde.org/D3268 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: drosca, #plasma, #plasma:_design Cc: plasma-devel, lesliezhai, ali-

[Differential] [Request, 21 lines] D3268: Applet: Show device icons and active port name

drosca created this revision. drosca added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Show the device name in format "Active Port: Device Name" and icon corresponding to the active port. Pulsea

[Differential] [Closed] D3168: Use microphone volume icon for capture devices/streams in applet

This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMAPA3c760965e6b6: Use microphone volume icon for capture devices/streams in applet (authored by drosca). REPOSITORY rPLASMAPA Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabricat

[Differential] [Commented On] D2314: Context Menu to Mute, Set Default Sink/Source, Active Port

drosca added a comment. In https://phabricator.kde.org/D2314#59454, @colomar wrote: > However, all the actions in the context menu must be available in the main UI as well. > So, this patch is good to go, but there needs to be a way to do these things without the context menu as well.

[Differential] [Commented On] D2314: Context Menu to Mute, Set Default Sink/Source, Active Port

drosca added a comment. In https://phabricator.kde.org/D2314#58816, @Zren wrote: > All I did was copy the Menu widget's pattern (sorta) since it was the first example I could find. Yeah, PlasmaComponents.ContextMenu sucks but if possible, I would prefer the declarative approach t

[Differential] [Updated] D3168: Use microphone volume icon for capture devices/streams in applet

drosca added a reviewer: Plasma: Design. REPOSITORY rPLASMAPA Plasma Audio Volume Applet REVISION DETAIL https://phabricator.kde.org/D3168 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: drosca, #plasma, #plasma:_design Cc: andreaska, plasma-devel, lesli

[Differential] [Commented On] D3168: Use microphone volume icon for capture devices/streams in applet

drosca added a comment. F379870: Spectacle.M21792.png REPOSITORY rPLASMAPA Plasma Audio Volume Applet REVISION DETAIL https://phabricator.kde.org/D3168 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: drosca, #pla

[Differential] [Request, 62 lines] D3168: Use microphone volume icon for capture devices/streams in applet

drosca created this revision. drosca added a reviewer: Plasma. drosca added a subscriber: andreaska. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY It doesn't make sense to show speaker volume icon for capture streams w

[Differential] [Changed Subscribers] D2314: Context Menu to Mute, Set Default Sink/Source, Active Port

drosca added subscribers: colomar, drosca. drosca added a comment. @colomar Can you please comment on this? We have been talking about using a context menu in plasmoid, but I don't remember what was the final decision from the VDG? REPOSITORY rPLASMAPA Plasma Audio Volume Applet REVISION

[Differential] [Closed] D3083: Mute volume when decreasing volume to zero

This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMAPA6366791aaa50: Mute volume when decreasing volume to zero (authored by drosca). REPOSITORY rPLASMAPA Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3083?vs=74

[Differential] [Accepted] D3124: Fix crash in application shutdown (alternate fix)

drosca accepted this revision. drosca added a comment. This revision is now accepted and ready to land. In https://phabricator.kde.org/D3124#58077, @davidedmundson wrote: > In https://phabricator.kde.org/D3124#58026, @drosca wrote: > > > Sorry, I've noticed this one only after acceptin

[Differential] [Requested Changes To] D3124: Fix crash in application shutdown (alternate fix)

drosca requested changes to this revision. drosca added a reviewer: drosca. drosca added a comment. This revision now requires changes to proceed. Sorry, I've noticed this one only after accepting the previous. This looks better, as long as it works without regressions. So, let's push the fi

[Differential] [Accepted] D3123: Fix crash on application shutdown

drosca accepted this revision. drosca added a reviewer: drosca. drosca added inline comments. This revision is now accepted and ready to land. INLINE COMMENTS > pulseaudio.cpp:321 > + > + > Source *SourceModel::defaultSource() const extra newline REPOSITORY rPLASMAPA Plasma Audio Volume Appl

[Differential] [Request, 20 lines] D3083: Mute volume when decreasing volume to zero

drosca created this revision. drosca added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY When decreasing the volume to minimal value (zero) with global shortcuts, the OSD shows the volume as muted.

[Differential] [Updated] D3057: Fix drawing QtQuickControls ComboBox popups

drosca updated the test plan for this revision. drosca added reviewers: broulik, hpereiradacosta. REPOSITORY rOXYGEN Oxygen Theme REVISION DETAIL https://phabricator.kde.org/D3057 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: drosca, #plasma, broulik,

[Differential] [Request, 127 lines] D3057: Fix drawing QtQuickControls ComboBox popups

drosca created this revision. drosca added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Implement drawing QStyleOptionFrame QFrame::StyledPanel shape. Adaptation of the same commit from Breeze, but

[Differential] [Closed] D2985: Quicklaunch: Listen to external configuration changes

This revision was automatically updated to reflect the committed changes. Closed by commit rKDEPLASMAADDONS0f4ec3e6da9c: Quicklaunch: Listen to external configuration changes (authored by drosca). REPOSITORY rKDEPLASMAADDONS Plasma Addons CHANGES SINCE LAST UPDATE https://phabricator.kde.org

[Differential] [Updated, 18 lines] D2985: Quicklaunch: Listen to external configuration changes

drosca updated this revision to Diff 7241. drosca added a comment. Same logic also for popup launchers REPOSITORY rKDEPLASMAADDONS Plasma Addons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2985?vs=7226&id=7241 BRANCH Plasma/5.8 REVISION DETAIL https://phabricator.kde.org/

[Differential] [Updated, 9 lines] D2985: Quicklaunch: Listen to external configuration changes

drosca updated this revision to Diff 7226. drosca added a comment. Disconnect form urlsChanged while setting urls to model REPOSITORY rKDEPLASMAADDONS Plasma Addons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2985?vs=7212&id=7226 BRANCH Plasma/5.8 REVISION DETAIL https://

[Differential] [Closed] D2833: Quicklaunch: Add support for jumplist actions

This revision was automatically updated to reflect the committed changes. Closed by commit rKDEPLASMAADDONS5e175a976967: Quicklaunch: Add support for jumplist actions (authored by drosca). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D2833?vs=6869&id=7213#toc REPOSITORY rKDEPLASMAADDO

[Differential] [Request, 7 lines] D2985: Quicklaunch: Listen to external configuration changes

drosca created this revision. drosca added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Fixes changing applet config from scripting. BUG: 370285 TEST PLAN Changed default panel layout.js as p

[Differential] [Request, 87 lines] D2833: Quicklaunch: Add support for jumplist actions

drosca created this revision. drosca added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REPOSITORY rKDEPLASMAADDONS Plasma Addons BRANCH master REVISION DETAIL https://phabricator.kde.org/D2833 AFFECTED FILES

[Differential] [Accepted] D2832: [plasma-pa] Set NotificationAreaCategory to Hardware

drosca accepted this revision. drosca added a reviewer: drosca. This revision is now accepted and ready to land. REVISION DETAIL https://phabricator.kde.org/D2832 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma, #plasma:_design, drosca Cc:

[Differential] [Closed] D2824: Make sure that default devices are not null

This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMAPA270110e5520f: Make sure that default devices are not null (authored by drosca). REPOSITORY rPLASMAPA Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2824?vs=6

[Differential] [Request, 43 lines] D2824: Make sure that default devices are not null

drosca created this revision. drosca added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Sometimes at the time when default sink/source change is signalled by pa, we don't have that device in sinks/

[Differential] [Commented On] D1126: Remove Desktop.SystemSettings import

drosca added a comment. @davidedmundson What happened? I think I updated the diff, waited for review and then just forgot, sorry. It won't happen again. REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D1126 EMAIL PREFERENCES https://phabricator.kde

[Differential] [Closed] D1126: Remove Desktop.SystemSettings import

This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMADESKTOP2acf6b8c4225: Remove Desktop.SystemSettings import (authored by drosca). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D1126?vs=3655&id=6339#toc REPOSITORY rPLASMADESKTOP Plasma Des

[Differential] [Closed] D989: Don't update size when minimum size changes while animation is running

This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMADESKTOP6ba90e0467d8: Don't update size when minimum size changes while animation is running (authored by drosca). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D989?vs=2395&id=6338#toc REPOS

[Differential] [Accepted] D2520: Add option for module-switch-on-connect

drosca accepted this revision. This revision is now accepted and ready to land. REPOSITORY rPLASMAPA Plasma Audio Volume Applet BRANCH master REVISION DETAIL https://phabricator.kde.org/D2520 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: davidedmund

[Differential] [Requested Changes To] D2520: Add option for module-switch-on-connect

drosca requested changes to this revision. drosca added a reviewer: drosca. drosca added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > modulemanager.cpp:74 > + > +if (switchOnConnect) { > +m_switchOnConnectConfig->set(QStringLiteral("name0"), > QSt

[Differential] [Closed] D2121: DesktopView: Don't call ensureWindowType on FocusIn event

This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMAWORKSPACE30b7e3f2423b: DesktopView: Don't call ensureWindowType on FocusIn event (authored by drosca). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D2121?vs=5026&id=5888#toc REPOSITORY rP

[Differential] [Commented On] D2121: DesktopView: Don't call ensureWindowType on FocusIn event

drosca added a comment. In https://phabricator.kde.org/D2121#45267, @graesslin wrote: > Patch for setting through surface event: No, that doesn't help, nor does calling ensureWindowType in QEvent::Expose. As it is now, calling ensureWindowType in QEvent::FocusIn is pointless (

[Differential] [Updated] D2314: Context Menu to Mute, Set Default Sink/Source, Active Port

drosca added reviewers: Plasma, Plasma: Design. REPOSITORY rPLASMAPA Plasma Audio Volume Applet REVISION DETAIL https://phabricator.kde.org/D2314 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: Zren, #plasma, #plasma:_design Cc: plasma-devel, ali-mohamed

[Differential] [Closed] D2219: Add option to enable volume feedback

This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMAPA6814a96da412: Add option to enable volume feedback (authored by drosca). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D2219?vs=5299&id=5542#toc REPOSITORY rPLASMAPA Plasma Audio Volume

[Differential] [Commented On] D2219: Add option to enable volume feedback

drosca added a comment. > Can we expect canberra to be available everywhere PA is? KF5 kmix has it as non-optional dependency too, so I think that should not be issue. > Is the feedback only played when there is no other source playing? It's played even if there is something pla

[Differential] [Closed] D2270: Try to be more clever about what sink to use for volume control

This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMAPA6c2f9f27543b: Try to be more clever about what sink to use for volume control (authored by drosca). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D2270?vs=5453&id=5498#toc REPOSITORY rPL

[Differential] [Commented On] D2270: Try to be more clever about what sink to use for volume control

drosca added inline comments. INLINE COMMENTS > davidedmundson wrote in device.cpp:68 > Use the values from pa_sink_state ? libpulse has pa_sink_state and pa_source_state, but here it is in Device - so for both sink and source. I can change it to use either PA_SOURCE_* or PA_SINK_* values (whi

[Differential] [Request, 172 lines] D2270: Try to be more clever about what sink to use for volume control

drosca created this revision. drosca added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY This changes the applet to display volume icon and control volume through scroll / global shortcuts of "prefe

[Differential] [Commented On] D2212: Reduce Calendar size in Digital clock

drosca added a comment. > But it takes more than half of my screen !!! Yes, because gridUnit sizing is broken. Here is how it looks on my system with patched plasma-framework: F189452: PDCLXFY.png REPOSITORY rPLASMAWORKSPACE Plasma Workspace REV

[Differential] [Commented On] D2212: Reduce Calendar size in Digital clock

drosca added a comment. https://git.reviewboard.kde.org/r/125773/ REPOSITORY rPLASMAWORKSPACE Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D2212 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: ochurlaud, #plasma:_design, #plasma, davi

[Differential] [Commented On] D2219: Add option to enable volume feedback

drosca added a comment. F188094: Screenshot_20160719_133541.png REPOSITORY rPLASMAPA Plasma Audio Volume Applet REVISION DETAIL https://phabricator.kde.org/D2219 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: dr

[Differential] [Request, 284 lines] D2219: Add option to enable volume feedback

drosca created this revision. drosca added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Play "pop" sound when changing volume BUG: 365197 REPOSITORY rPLASMAPA Plasma Audio Volume Applet BRANCH

[Differential] [Updated] D2219: Add option to enable volume feedback

drosca added a reviewer: Plasma: Design. REPOSITORY rPLASMAPA Plasma Audio Volume Applet REVISION DETAIL https://phabricator.kde.org/D2219 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: drosca, #plasma, #plasma:_design Cc: plasma-devel, jensreuterberg,

[Differential] [Abandoned] D2217: Add option to disable OSD on volume change

drosca abandoned this revision. drosca added a comment. > That looks like a way better way to do it, if we decide for it. Agree REPOSITORY rPLASMAPA Plasma Audio Volume Applet REVISION DETAIL https://phabricator.kde.org/D2217 EMAIL PREFERENCES https://phabricator.kde.org/settings/

[Differential] [Commented On] D2217: Add option to disable OSD on volume change

drosca added a comment. I think there already is global option to disable OSD, but I think it makes sense to just disable OSD for volume. Also kmix has this option and there are users that want it. REPOSITORY rPLASMAPA Plasma Audio Volume Applet REVISION DETAIL https://phabricator.kde.o

[Differential] [Commented On] D2217: Add option to disable OSD on volume change

drosca added a comment. F188070: Screenshot_20160719_122725.png REPOSITORY rPLASMAPA Plasma Audio Volume Applet REVISION DETAIL https://phabricator.kde.org/D2217 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: dr

[Differential] [Updated] D2217: Add option to disable OSD on volume change

drosca added a reviewer: Plasma: Design. REPOSITORY rPLASMAPA Plasma Audio Volume Applet REVISION DETAIL https://phabricator.kde.org/D2217 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: drosca, #plasma, #plasma:_design Cc: plasma-devel, jensreuterberg,

[Differential] [Request, 96 lines] D2217: Add option to disable OSD on volume change

drosca created this revision. drosca added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY BUG: 363983 TEST PLAN I'm not really sure on the label "Show OSD" as that may not be clear to user what it

[Differential] [Commented On] D2212: Reduce Calendar size in Digital clock

drosca added a comment. The calendar popup size is fine for me (with patched plasma-framework), what I *still* think is broken is the gridSize scaling - see https://git.reviewboard.kde.org/r/125773/ You can also test it simply by switching font to DejaVu Sans. REPOSITORY rPLASMAWORKSP

[Differential] [Accepted] D2073: Add a KCM setting to control pulse audio module combine-sinks

drosca accepted this revision. drosca added a comment. This revision is now accepted and ready to land. Please fix that last style issue and push, thanks! INLINE COMMENTS > modulemanager.h:42 > +public: > +ModuleManager(QObject *parent=nullptr); > +~ModuleManager(); This needs spaces

[Differential] [Requested Changes To] D2073: Add a KCM setting to control pulse audio module combine-sinks

drosca requested changes to this revision. drosca added a reviewer: drosca. drosca added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > modulemanager.cpp:33 > +QObject(parent), > +m_combineSinksConfig(new GConfItem(PA_GCONF_PATH_MODULES"/combine", > thi

[Differential] [Commented On] D2121: DesktopView: Don't call ensureWindowType on FocusIn event

drosca added a comment. snip from xprop -spy on desktop window when clicking on desktop window and other windows: _NET_WM_USER_TIME(CARDINAL) = 894873 _NET_WM_USER_TIME(CARDINAL) = 894853 _NET_WM_WINDOW_TYPE(ATOM) = _NET_WM_WINDOW_TYPE_DESKTOP _MOTIF_WM_HINTS(_MOTIF_WM_HINTS

[Differential] [Request, 3 lines] D2121: DesktopView: Don't call ensureWindowType on FocusIn event

drosca created this revision. drosca added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Fixes desktop window losing the keep-below flag. BUG: 365014 TEST PLAN The real issue is probably some

  1   2   3   >