KClearableLineEdit

Michel Hermier michel.hermier at gmail.com
Fri Aug 25 22:43:02 BST 2006


2006/8/25, Frank Osterfeld <frank.osterfeld at gmx.de>:
> On Wednesday 23 August 2006 19:32, Aaron J. Seigo wrote:
>
> > bool clearButtonShown();
>
> How about isClearButtonShown()?

Instead of show/hide why not use visible ?
so that it become bool isClearButtonVisible() const
and the setter setClearButtonVisible(bool)

> KLineEdit is inconsistent here:
>
> bool isContextMenuEnabled () const
> bool urlDropsEnabled () const
>
> As Qt uses isFoo() in most places, how about making it consistent and using
> isUrlDropsEnabled() and isClearButtonShown()?
>
> Frank
>
>
>




More information about the kde-core-devel mailing list