Bug 332245

Summary: Applet handle should have minimum size
Product: [Plasma] plasmashell Reporter: Bhushan Shah <bhush94>
Component: generalAssignee: Sebastian Kügler <sebas>
Status: RESOLVED FIXED    
Severity: normal CC: kde
Priority: NOR    
Version: master   
Target Milestone: 1.0   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: problem
problem with close

Description Bhushan Shah 2014-03-17 12:29:17 UTC
Created attachment 85611 [details]
problem

In analog clock, since it does not have minimum height or width it looks like its screwed up.
Comment 1 David Edmundson 2014-03-26 19:08:44 UTC
Git commit 344974c4bb9e76bd0566694d0a4b0334d8fee7b4 by David Edmundson.
Committed on 26/03/2014 at 19:06.
Pushed by davidedmundson into branch 'master'.

Fix minimum size on the Applet Handle

- center align the noBackgroundHandle and obey the minimumHeight

- made the mouse event listener that controls whether to show the
handle separate from the size of the applet, so we can still reach the
controls when the applet is smaller than the handle

M  +14   -2    plasma-desktop/containments/desktop/package/contents/ui/AppletAppearance.qml
M  +5    -1    plasma-desktop/containments/desktop/package/contents/ui/AppletHandle.qml

http://commits.kde.org/kde-workspace/344974c4bb9e76bd0566694d0a4b0334d8fee7b4
Comment 2 Bhushan Shah 2014-03-27 15:23:24 UTC
Created attachment 85789 [details]
problem with close

Close button is still out of applet
Comment 3 Marco Martin 2014-04-01 17:43:33 UTC
Git commit e6f47721d6da005d2109df621f824891f8df3f82 by Marco Martin.
Committed on 01/04/2014 at 17:41.
Pushed by mart into branch 'master'.

don't move the handle on resize

also fix handle sizing, and don't hide when the mouse is over an handle button
Related: bug 332695

M  +11   -12   containments/desktop/package/contents/ui/AppletAppearance.qml
M  +61   -12   containments/desktop/package/contents/ui/AppletHandle.qml

http://commits.kde.org/plasma-desktop/e6f47721d6da005d2109df621f824891f8df3f82