Bug 332850 - Starting widget explorer from panel controller overlaps the panel
Summary: Starting widget explorer from panel controller overlaps the panel
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Panel (show other bugs)
Version: master
Platform: unspecified Linux
: NOR normal
Target Milestone: 1.0
Assignee: David Edmundson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-03-31 11:55 UTC by Bhushan Shah
Modified: 2014-04-02 17:40 UTC (History)
1 user (show)

See Also:
Latest Commit: http://commits.kde.org/plasma-framework/d0908ad73536cee158b066645d8b993511ae9bdd
Version Fixed In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Bhushan Shah 2014-03-31 11:55:31 UTC 
1) Start panel controller
2) Start widget explorer from there.
Comment 1 Martin Klapetek 2014-04-01 15:37:32 UTC 
I think I have an idea about this, I'm on it.
Comment 2 Martin Klapetek 2014-04-02 16:34:14 UTC 
Investigation:

As soon as the panel config is opened, the panel looses struts. As a result, when opening the widget explorer, it covers the panel because it does not have the struts anymore.

D_ed says: we could loose the struts only when you click the "screen edge" button or start resizing the panel.
Comment 3 Marco Martin 2014-04-02 17:38:54 UTC 
Git commit ada24e2bc7978dd23a1b6bc7b57d68967fea1cc3 by Marco Martin.
Committed on 02/04/2014 at 17:36.
Pushed by mart into branch 'master'.

explicitly move at topLeft of availableRect
Related: bug 332702

M  +4    -1    desktoppackage/contents/views/Desktop.qml

http://commits.kde.org/plasma-desktop/ada24e2bc7978dd23a1b6bc7b57d68967fea1cc3
Comment 4 Marco Martin 2014-04-02 17:40:59 UTC 
Git commit d0908ad73536cee158b066645d8b993511ae9bdd by Marco Martin.
Committed on 02/04/2014 at 17:39.
Pushed by mart into branch 'master'.

use views geometries to calculate availableArea

Don't trust anything X11: use our internal tracking of panel
geometries to compute the corona availablescreenarea
also because the space behind a panel not autohide but without struts is to be considered unavailable as well

M  +77   -6    src/shell/shellcorona.cpp

http://commits.kde.org/plasma-framework/d0908ad73536cee158b066645d8b993511ae9bdd