Bug ID 1062420
Summary plasma5-desktop: mispositioned text field when renaming right-aligned folder-view icon
Classification openSUSE
Product openSUSE Distribution
Version Leap 42.3
Hardware All
OS openSUSE 42.3
Status NEW
Severity Minor
Priority P5 - None
Component KDE Workspace (Plasma)
Assignee opensuse-kde-bugs@opensuse.org
Reporter markohaas@yahoo.de
QA Contact qa-bugs@suse.de
Found By ---
Blocker ---

Created attachment 743630 [details]
mispositioned text field when renaming right-aligned folder-view icon

When renaming icons on my Plasma5 desktop after having those icons
right-aligned, the actual renaming is invisible while typing; cause:
off-display position of the text field showing the typing (see screenshot).


List of steps to reproduce this:

0. Log into a KDE-5/Plasma 5 session (in my case, v5.8)
1. Configure Desktop (Alt+D, Alt+S); choose ��Folder View Settings�� ��� ��Show the
Desktop folder��
2. right-click empty Desktop area ��� Icons ��� Align ��� Right
3. if Desktop is empty, create item on Desktop (right-click ��� Create New ���)
4. press F2 (or right-click item, choose ��Rename��)
5. enter new name & press Return; the item is displayed with the new name. 


Expected behavior: A text field directly beneath the item���s icon shows the
keyboard input. 

Actual behavior: No cursor or changing text is visible while typing (or,
depending on the length of the original item name, a small part of the editable
text area is showing on the left-hand edge of the Desktop; see screenshot
��Screenshot_20171009_191206��).


Patch suggestion for package plasma5-desktop-5.8.7.1-3.1.x86_64, using file 
/usr/share/plasma/plasmoids/org.kde.desktopcontainment/contents/ui/FolderView.qml


diff --git a/FolderView.qml b/FolderView.qml
index 5617e0b..6443c74 100644
--- a/FolderView.qml
+++ b/FolderView.qml
@@ -1041,6 +1041,9 @@ Item {
                    _y = pos.y - __style.padding.top;
                 } else {
                    _x = targetItem.x + units.largeSpacing + units.smallSpacing
- __style.padding.left;
+                   if(_x < 0) {
+                      _x += gridView.width;
+                   }
                    _y = pos.y + units.smallSpacing - __style.padding.top;
                 }
                 return([ _x, _y ]);

My quick hack to fix it on my Leap 42.3 rig may not work in the future (for
example, if somebody wants to have their desktop icons centered), and it may
have implications/side effects for other files (e.g. FolderView.cpp). Eike
Hein, author of most of these files at KDE, might want to cross-check for
similar cases or dependent code elsewhere.

Bug 967052 (��plasma5-desktop: obsolete plasmoid-folderview��) may render my
suggestion moot.


You are receiving this mail because: