Bug 157466

Summary: [PATCH] Use KLineEdit's built-in clear button for the Calculate text field
Product: [Applications] kalzium Reporter: David Benjamin <davidben>
Component: generalAssignee: Carsten Niehaus <cniehaus>
Status: RESOLVED FIXED    
Severity: wishlist    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: Use KLineEdit's built-in clear button

Description David Benjamin 2008-02-09 00:43:31 UTC 
Version:           1.9.91 (using KDE 4.0.0)
Installed from:    Compiled From Sources
OS:                Linux

KLineEdit provides a clear button that is integrated into the text field, with a nice fade effect and everything. :-)

Kalzium should use this for instead of a separate clear button in the Calculate page of the left toolbox.
Comment 1 David Benjamin 2008-02-09 00:46:02 UTC 
Created attachment 23483 [details]
Use KLineEdit's built-in clear button

For some reason, the diff for the .ui file seems slightly bulky. (It used to be
really bulky, but I manually reverted some of the stuff... it wanted to change
the margin to marginLeft,Right,etc.) All I did was delete the clear button and
save the file.
Comment 2 Carsten Niehaus 2008-02-09 15:02:41 UTC 
After applying your patch I see no clear-button at all. 

I will have another look tomorrow.
Comment 3 David Benjamin 2008-02-09 15:08:35 UTC 
Did you type something in the field?

The clear button appears as soon as there is text in the dialog, and fades out when the text is blank. (The breadcrumb in Dolphin uses the same thing, although that one is usually not blank.) It's nifty.
Comment 4 Carsten Niehaus 2008-02-10 12:06:37 UTC 
Patch accepted. Thank you.