Bug 356754

Summary:	Wrong notification position offset with vertical panel/taskbar
Product:	[Plasma] plasmashell	Reporter:	Soukyuu <chrno-sphered>
Component:	Notifications	Assignee:	Martin Klapetek <mklapetek>
Status:	RESOLVED FIXED
Severity:	normal	CC:	plasma-bugs-null
Priority:	NOR
Version First Reported In:	5.5.1
Target Milestone:	1.0
Platform:	Arch Linux
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	current notification position of bottom left corner desired notification position of bottom left corner current notification position of bottom right corner desired notification position of bottom right corner sample of notification at the upper middle corner screenshot of both screens

Description Soukyuu 2015-12-15 21:28:44 UTC

I'm currently using the lower left/right corner to display the notifications, with my taskbar being on the left side of the screen. The notification appears about 3 cm above the lower border of the screen, meaning, if the panel was at the bottom of the screen, the offset of the notification would fit perfectly:
- horizontal offset = 10px from the right edge
- vertical offset = panel height + 10 px from the bottom edge



Reproducible: Always

Steps to Reproduce:
1. set panel with systray to left side of the screen
2. set notification position to bottom left/middle/right
3. trigger the notification

Actual Results:  
notification is too far away from the bottom edge

Expected Results:  
IMHO, for a vertical panel on the LEFT and notification on the RIGHT, it should be:
- horizontal = 10px from right edge
- vertical = 10px from bottom edge

and for a vertical panel on the LEFT and notification on the LEFT, it should be:
- horizontal = panel width + 10px from the left edge
- vertical = 10px from the bottom edge

and for a notification in the MIDDLE, it should be
- horizontal = middle of the screen
- vertical = 10px from the bottom edge

The 10px is eyeballed. If possible, it would be nice to make the offsets configurable, so in addition to setting the corner, you could specify the x and y offsets as well.

Comment 1 Soukyuu 2015-12-15 21:30:47 UTC

Created attachment 96106 [details]
current notification position of bottom left corner

There is way too much space between the notification and bottom of the screen

Comment 2 Soukyuu 2015-12-15 21:31:56 UTC

Created attachment 96107 [details]
desired notification position of bottom left corner

Comment 3 Martin Klapetek 2015-12-15 21:32:56 UTC

Thanks for the report

Do you see this wrong position also after reboot/relogin?

Comment 4 Soukyuu 2015-12-15 21:33:31 UTC

Created attachment 96108 [details]
current notification position of bottom right corner

essentially the same as with the two other attachments, but for the bottom right corner

Comment 5 Soukyuu 2015-12-15 21:34:13 UTC

Yes, it's been like that for two days now, ever since I updated to 5.5 from 5.4

Comment 6 Soukyuu 2015-12-15 21:35:10 UTC

Created attachment 96109 [details]
desired notification position of bottom right corner

Comment 7 Martin Klapetek 2015-12-15 21:35:32 UTC

Ok. The code actually /does/ have only a small offset from panel/screen edge
(like 8px, it depends on your font configuration). Can you please try with 5.5.1
(just out) and report back? There are some fixes regarding the positions.

Please change bug status once you verify with 5.5.1.

Comment 8 Martin Klapetek 2015-12-15 21:36:19 UTC

(needs a comment to set as needsinfo)

Comment 9 Soukyuu 2015-12-15 21:40:19 UTC

Created attachment 96110 [details]
sample of notification at the upper middle corner

Actually, this also seems to affect the upper corners as well - I'm not usually using those positions so I didn't notice. I'm not sure if it's relevant, but I'm using two monitors, configured as:

Screen 0: minimum 8 x 8, current 3200 x 1084, maximum 8192 x 8192
DVI-I-2 connected primary 1920x1080+0+0 (normal left inverted right x axis y axis) 531mm x 299mm
   1920x1080     59.93 +  60.00* 
DVI-I-3 connected 1280x1024+1920+60 (normal left inverted right x axis y axis) 330mm x 270mm
   1280x1024     60.02*+

via xorg.conf

Comment 10 Martin Klapetek 2015-12-15 22:18:27 UTC

Can you please also attach a screenshot of your whole desktop?
Ie. select "Full Screen" in ksnapshot.

Comment 11 Soukyuu 2015-12-15 22:28:53 UTC

Created attachment 96111 [details]
screenshot of both screens

Comment 12 Soukyuu 2015-12-15 22:29:41 UTC

I've tried it with 5.5.1 and so far it seems to be working as intended, the offsets are correct now. Sorry for the jumbled replies, I'm on a slow connection so uploading the attachments took a while.

Comment 13 Martin Klapetek 2015-12-15 22:38:42 UTC

Ah, great. So I'll close this here.

Feel free to reopen if you encounter the issue again.