Bug 376529

Summary:	The panel is placed over the desktop icons, when both are located to the right.
Product:	[Plasma] plasmashell	Reporter:	Nibaldo G. <nibgonz>
Component:	Desktop icons & Folder View widget	Assignee:	Eike Hein <hein>
Status:	RESOLVED FIXED
Severity:	minor	CC:	kde, notmart, plasma-bugs-null
Priority:	NOR
Version First Reported In:	5.9.2
Target Milestone:	1.0
Platform:	Neon
OS:	Linux
Latest Commit:	https://commits.kde.org/plasma-desktop/d37b57dae09d1d9d0e2b26101514ad216139c063	Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	Desktop icons and panel, aligned to the left and right of the screen.

Description Nibaldo G. 2017-02-16 05:32:17 UTC

The desktop panel is located above, covering the desktop icons (the icons should be on the side of the panel, not below).

The bug occurs only if:
- The desktop panel is set to the right of the screen.
- The desktop has the "Folder View" layout and the icons are aligned to the right.

The bug does not occurs if the panel is up, down or left the screen, or the icons are aligned to the left of the desktop.

The bug appeared in Plasma 5.9.1 (before it did not happen).

Comment 1 Nibaldo G. 2017-03-01 10:17:10 UTC

Created attachment 104289 [details]
Desktop icons and panel, aligned to the left and right of the screen.

When the desktop icons are aligned to the right of the screen, the panel covers them. If the icons are aligned to the left, they are displayed correctly next to the panel.

Comment 2 Kai Uwe Broulik 2017-03-01 10:21:30 UTC

I think Eike mentioned a bug in GridView which causes this, when icons are right-aligned and (important) aligned in columns. Otherwise it works.

Comment 3 Kai Uwe Broulik 2017-03-01 10:27:52 UTC

Seems this is intentional...

// Don't apply the right margin if the folderView is in column mode and not overflowing.
        // In this way, the last column remains droppable even if a small part of the icon is behind a panel.
        rightMargin: folderViewLayer.ready && (folderViewLayer.view.overflowing  || folderViewLayer.view.flow == GridView.FlowLeftToRight)
            && (isContainment && plasmoid.availableScreenRect) && parent
            ? parent.width - (plasmoid.availableScreenRect.x + plasmoid.availableScreenRect.width) : 0

Comment 4 Eike Hein 2017-03-07 20:20:50 UTC

Git commit d37b57dae09d1d9d0e2b26101514ad216139c063 by Eike Hein.
Committed on 07/03/2017 at 20:20.
Pushed by hein into branch 'Plasma/5.8'.

Do apply margin if in right-to-left mode.

M  +2    -1    containments/desktop/package/contents/ui/main.qml

https://commits.kde.org/plasma-desktop/d37b57dae09d1d9d0e2b26101514ad216139c063