Bug 429187

Summary: Text field's clear button overlaps text
Product: [Frameworks and Libraries] libplasma Reporter: medin <med.medin.2014>
Component: componentsAssignee: Marco Martin <notmart>
Status: RESOLVED FIXED    
Severity: normal CC: nate, plasma-bugs
Priority: NOR    
Version: 5.76.0   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In: 5.77
Attachments: RTL language and clear icon

Description medin 2020-11-16 12:11:45 UTC
Created attachment 133376 [details]
RTL language and clear icon

When I use right to left language the clear icon is shown on top of searched text. See attached image for more info.
Comment 1 Nate Graham 2020-11-16 20:47:07 UTC
Looks like the PlasmaComponents3.TextField does not reverse its clear button properly in RTL mode.
Comment 2 Nate Graham 2020-11-16 21:01:34 UTC
Actually it does reverse itself properly when running plasmashell in RTL mode (plasmashell --reverse) so part of the problem is that you're using an RTL language, but not putting Plasmashell into RTL mode. That will likely end up as another but or a local configuration problem.

The icon overlapping the text is a real issue though. I'll fix that.
Comment 3 Bug Janitor Service 2020-11-16 21:04:29 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/145
Comment 4 medin 2020-11-17 09:34:50 UTC
(In reply to Nate Graham from comment #2)
> Actually it does reverse itself properly when running plasmashell in RTL
> mode (plasmashell --reverse) so part of the problem is that you're using an
> RTL language, but not putting Plasmashell into RTL mode. That will likely
> end up as another but or a local configuration problem.
> 
> The icon overlapping the text is a real issue though. I'll fix that.

Thanks, I didn't know I can put whole Plasma in RTL mode, is there any GUI settings where I can set it ? because I use french and arabic keyboard layouts (unfortunately Tifinagh is not available on KDE Plasma) at the same time and use only Alt+Shift or Ctrl+Alt+K to switch between them.
Comment 5 Nate Graham 2020-11-17 15:33:56 UTC
I'm afraid I don't know the answer to that question, sorry. :(
Comment 6 Nate Graham 2020-11-24 17:01:44 UTC
Git commit c5fa23f8582b2397295906ed57e3ec9002432700 by Nate Graham.
Committed on 24/11/2020 at 17:01.
Pushed by ngraham into branch 'master'.

[textfield] Fix clear button overlapping text

This change mirrors what the Kirigami ActionTextField does to solve the
same problem.
FIXED-IN: 5.77

M  +4    -3    src/declarativeimports/plasmacomponents3/TextField.qml

https://invent.kde.org/frameworks/plasma-framework/commit/c5fa23f8582b2397295906ed57e3ec9002432700