Bug 340549 - Panel doesn't autohide if it's placed on the right side
Summary: Panel doesn't autohide if it's placed on the right side
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Panel (show other bugs)
Version: master
Platform: Arch Linux Linux
: NOR normal
Target Milestone: 1.0
Assignee: David Edmundson
URL:
Keywords:
: 340814 345886 (view as bug list)
Depends on:
Blocks:
 
Reported: 2014-11-01 10:51 UTC by Marius Orcsik
Modified: 2015-04-05 14:06 UTC (History)
8 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.3


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Marius Orcsik 2014-11-01 10:51:40 UTC
When moving the default panel to the right side of the screen it stops following the Autohide checkbox.

It behaves as if the Windows Go Below setting was checked.

Reproducible: Always

Steps to Reproduce:
1. Change the behaviour of the panel to Autohide
2. Move the panel to the right side of the screen
3. The panel doesn't hide when losing focus


Expected Results:  
The panel should hide and the autohide highlight should appear if the mouse hovers around the right side.
Comment 1 David Edmundson 2014-11-05 22:16:16 UTC
It's working here 
Do you have any other monitors to the right of the screen with the panel on the right edge? 

I do run git latest, so maybe it's fixed?

I'm going to close it as working for me, please reopen it if you still have this problem after 5.2.0
Comment 2 Marius Orcsik 2014-11-06 17:34:29 UTC
Ok, I will give it a try as soon as 5.2.0 lands in the Arch repos and I'll come back with an update.

Thank you for your clarification David.
Comment 3 Marius Orcsik 2014-11-06 17:35:26 UTC
I forgot to reply to your question: no, no other monitors.
Comment 4 Martin Klapetek 2014-11-18 17:12:27 UTC
It still is reproducible with master - if I put a panel on the right, resize it, I can see the panel no longer sitting on the edge, there is 1px space in between and in this state the panel will not auto hide. If I open the panel controller and drag it towards the edge, so that the 1px line is gone, auto-hide will start working.

Marius - can you confirm this behavior?
Comment 5 Marius Orcsik 2014-11-27 20:41:03 UTC
Hello, 

I have installed the git version and indeed, when moving the panel to the right, sometimes there's a pixel remaining to the side of the monitor. 

If moving again more carefully, it snaps in place and it auto-hides.
Comment 6 Bhushan Shah 2015-01-09 15:54:32 UTC
*** Bug 340814 has been marked as a duplicate of this bug. ***
Comment 7 Elias Probst 2015-02-15 19:40:47 UTC
Seeing the problem here too.

The "1 pixel between panel and right screen edge" is inserted as soon as I change the panel's width.
Then I'll have to move it again to the very right edge to make it auto-hide again.
Comment 8 Elias Probst 2015-02-16 14:40:28 UTC
(In reply to Elias Probst from comment #7)
> Seeing the problem here too.
> 
> The "1 pixel between panel and right screen edge" is inserted as soon as I
> change the panel's width.
> Then I'll have to move it again to the very right edge to make it auto-hide
> again.

Looks like this "1 pixel margin" is also inserted on each plasmashell restart.
Comment 9 Elias Probst 2015-02-18 05:52:52 UTC
Now I have 3 reproducable cases where the "1 pixel between panel and screen edge" gets inserted:
- changing the panel's width
- (un-)plugging a 2nd screen
- plasmashell restart
Comment 10 Bhushan Shah 2015-03-27 12:22:04 UTC
Git commit 975e5009c176eb378daa5c8f74cd4cffb64ff459 by Bhushan Shah.
Committed on 26/03/2015 at 15:50.
Pushed by bshah into branch 'master'.

Fix one pixel gap between screenedge and right placed vertical panel

Appearantly right() / bottomRight() / topRight() returns values with 1
pixel x off for historical reasons and hence 1px gap. This causes the
autohide panel not hiding automatically.
Related: bug 339323
FIXED-IN: 5.3
REVIEW: 123136

M  +6    -5    shell/panelview.cpp

http://commits.kde.org/plasma-workspace/975e5009c176eb378daa5c8f74cd4cffb64ff459
Comment 11 Bhushan Shah 2015-04-05 14:06:00 UTC
*** Bug 345886 has been marked as a duplicate of this bug. ***