Review Request 106110: Make autohide panel do not intercept events in fullscreen app
Aaron J. Seigo
aseigo at kde.org
Wed Mar 20 14:00:25 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106110/#review29551
-----------------------------------------------------------
have you tested the following two use cases:
* panel hides, a window becomes full screen at that point .. does the panel still unhide now?
* window becomes full screen, panel hides (and with the patch no unhide trigger is created), window goes out of full screen
in both cases does the panel unhide or not as expected?
the reason i ask is that the unhide trigger is only created when a resize or move event happens, deskop reserve struts are changed, the theme changes or updatePanelGeometry is called due to screen geometry change or the panel position on screen changes. .. and i'd be surprised if those get called when a window goes into or exits full screen. So this fix would create a new bug in which the hint hide doesn't show until the panel is unhidden once after a fullscreen window shows.
quick tip: to ensure that mouse movements don't interfere and to simulate "things happening all at once without user interaction" usually i trigger full screen with sth like: `sleep 3; qdbus org.kde.konsole-`pidof konsole` /konsole/MainWindow_1/actions/fullscreen` which gives me 3 seconds to hide the panel before the action occurs, but without any further user input.
I think what probably needs to happen here is a bit more straightforward -> whenever the hide area is triggered (e.g. in the "fancy" case, the mouse approaches the trigger area) it should check to see if there is a fullscreen window on the same screen and if so .. don't do anything.
That said, the real solution here will be to move to the single screen edge implementation where the fancy show uses the kwin effect that Martin has written (and which looks really nice, btw, Martin :)
plasma/desktop/shell/panelview.cpp
<http://git.reviewboard.kde.org/r/106110/#comment22036>
Better would simply be to compare screens. If they are on the same screen -> don't interfere.
- Aaron J. Seigo
On March 18, 2013, 6:37 p.m., Alexey Shmalko wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106110/
> -----------------------------------------------------------
>
> (Updated March 18, 2013, 6:37 p.m.)
>
>
> Review request for kwin and Plasma.
>
>
> Description
> -------
>
> Make autohide panel do not intercept events in fullscreen app.
>
> I changed PanelView::createUnhideTrigger() to check whether current active application is in fullscreen mode. So in fullscreen applications panel will be always hidden.
>
>
> This addresses bug 217560.
> http://bugs.kde.org/show_bug.cgi?id=217560
>
>
> Diffs
> -----
>
> plasma/desktop/shell/panelview.cpp 7713740
>
> Diff: http://git.reviewboard.kde.org/r/106110/diff/
>
>
> Testing
> -------
>
> Tested with patched gentoo ebuild. Works fine.
>
>
> Thanks,
>
> Alexey Shmalko
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20130320/ece48319/attachment-0001.html>
More information about the Plasma-devel
mailing list