dhruv8sh / plasma6-window-title-applet

Plasma 6 Window Title applet
GNU General Public License v2.0
65 stars 4 forks source link

Single click opening overview seems bugged #20

Open LinusCDE opened 5 months ago

LinusCDE commented 5 months ago

Hi,

I first thought this might be a bug / default of KDEs control panel:

When I do a single click on the widget, it opens the new overview. While I like the new overview, it was a bit confusing for me first (I made it fill space, so I thought the panel might do this and not properly detect a click on the widget).

I'm used to using the Double-Click to Maximize a window functionality. The new single click feature seems to conflict with this as well, since it then does both actions.

It would be cool to have a toggle, to disable the feature. That would first clarify, that this feature exists and allow people to workaround the behaviour.

Ofc being able to distinguish a single and double click and have both work, would also be cool. But being able to turn the single click functionality off (maybe also the double click one, if people don't like that) still seems like a good idea imo.

Though Kudos for porting this! You saved my day!

LinusCDE commented 5 months ago

As a workaround, I found the part in the code, which handles the single click, and removed it locally:

index 8b6ef78..033101b 100644
--- a/contents/ui/ActionsMouseArea.qml
+++ b/contents/ui/ActionsMouseArea.qml
@@ -9,10 +9,6 @@ MouseArea {
     anchors.fill: parent
     property bool wheelIsBlocked: false

     onClicked: function(event){
         if (existsWindowActive && event.button === Qt.MiddleButton) windowInfoLoader.item.requestClose();
-        else if( event.button === Qt.LeftButton ) executable.exec("qdbus org.kde.kglobalaccel /component/kwin invokeShortcut Overview");
     }
     onDoubleClicked: { if (existsWindowActive) windowInfoLoader.item.toggleMaximized(); }
     onWheel: function(wheel) {
         if (wheelIsBlocked || !plasmoid.configuration.actionScrollMinimize) return;