Bug 157466 - [PATCH] Use KLineEdit's built-in clear button for the Calculate text field
Summary: [PATCH] Use KLineEdit's built-in clear button for the Calculate text field
Status: RESOLVED FIXED
Alias: None
Product: kalzium
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR wishlist
Target Milestone: ---
Assignee: Carsten Niehaus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-02-09 00:43 UTC by David Benjamin
Modified: 2008-02-10 12:06 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Use KLineEdit's built-in clear button (2.14 KB, patch)
2008-02-09 00:46 UTC, David Benjamin
Details

Note You need to log in before you can comment on or make changes to this bug.
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.