[Bug 1062420] New: plasma5-desktop: mispositioned text field when renaming right-aligned folder-view icon
http://bugzilla.opensuse.org/show_bug.cgi?id=1062420 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 --> http://bugzilla.opensuse.org/attachment.cgi?id=743630&action=edit 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: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=1062420 Marko Haas <markohaas@yahoo.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|Leap 42.3 |Leap 15.0 OS|openSUSE 42.3 |SLED 15 -- You are receiving this mail because: You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply@novell.com