Bug 259709

Summary: Line-Edit deletion "X" overlays text entered
Product: KDE PIM Mobile Reporter: Björn Balazs <kde>
Component: generalAssignee: kdepim bugs <kdepim-bugs>
Status: VERIFIED FIXED    
Severity: major CC: aheinecke, ludwig.reiter
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Unlisted Binaries   
OS: Linux   
Latest Commit: Version Fixed In:

Description Björn Balazs 2010-12-13 12:12:46 UTC
Version:           unspecified (using Devel) 
OS:                Linux

In all applications line edits show the "delete this line X-Grafics" at the end of the line as soon as any text gets entered into the line. This grafic overlays the text and makes the end of the text unreadable in consequence. Please do not allow text below this icon!

Blocker, because it makes it impossible to check the wether the user entered the correct data (e.g. the TLD of an email adress) and to correct any mistakes made when entering the email address.

Reproducible: Always
Comment 1 Volker Krause 2010-12-13 12:56:27 UTC
*** Bug 253953 has been marked as a duplicate of this bug. ***
Comment 2 Tobias Koenig 2011-01-07 13:37:57 UTC
commit da17bee2a0e3167002793af9525eed592e419d3d
branch master
Author: Tobias Koenig <tokoe@kde.org>
Date:   Fri Jan 7 13:37:54 2011 +0100

    Use right padding size for KLineEdit's clear button
    
    Since the mobile apps use a stylesheet, KLineEditStyle won't be
    used anymore by KLineEdits inside the mobile apps. This breaks the
    adapted KLineEditStyle::subElementRect, that substracts the clear button
    width from the contents rect width. We hard-code it in the stylesheet now.
    
    BUG: 259709

diff --git a/mobile/lib/stylesheet.css.cmake b/mobile/lib/stylesheet.css.cmake
index def1a32..f17a013 100644
--- a/mobile/lib/stylesheet.css.cmake
+++ b/mobile/lib/stylesheet.css.cmake
@@ -83,6 +83,16 @@ QSpinBox
   padding: 4px;
 }
 
+KLineEdit
+{
+  background-color: white;
+  border: 2px;
+  border-color: grey;
+  border-radius: 8px;
+  border-style: inset;
+  padding: 4px;
+  padding-right: 48px
+}
 
 
 /*