Review Request: Keep clickmessage text visible when empty and focused

Laszlo Papp lpapp at kde.org
Thu Dec 13 06:20:01 GMT 2012



> On Dec. 12, 2012, 6:43 p.m., Laszlo Papp wrote:
> > It did not get a ship it back then for the plasma components, but I would not like to block the rediscussion, just saying what happened earlier. :-)
> > 
> > https://git.reviewboard.kde.org/r/104895/
> 
> David Edmundson wrote:
>     One of the key arguments people had then was because it would have led to seriously inconsistent behaviour between plasma components and the desktop. 
>     
>     Aurelien is fixing that by also proposing to patch KDE libs, which turns my personal opinion from a definite "no" to something I'm happy with.
>

At least, I do not see such a key argument personally in the previous review comments. As for me, it seems that certain people would prefer one way, and certain people would prefer the other way. So, it is a hard decision to take in my opinion. Certain projects do this way, and certain do not.

It will stay inconsistent with the QLineEdit based applications for instance (I know one could bring different examples, too). I am not envy for the decision maker. :-)


- Laszlo


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107678/#review23366
-----------------------------------------------------------


On Dec. 12, 2012, 5:10 p.m., Aurélien Gâteau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107678/
> -----------------------------------------------------------
> 
> (Updated Dec. 12, 2012, 5:10 p.m.)
> 
> 
> Review request for kdelibs.
> 
> 
> Description
> -------
> 
> This patch changes the way KLineEdit and KTextEdit handle the clickMessage property. The message remains visible when the widget is focused, as long as no text has been typed in. This is useful when the widget is focused by default, and also gives another opportunity to the user to read the message before typing, removing the need for workarounds  like clicking in another text entry to bring back the clickMessage.
> 
> I filed a similar review-request for Plasma components: https://git.reviewboard.kde.org/r/107600/
> 
> 
> Diffs
> -----
> 
>   kdeui/widgets/klineedit.cpp d96c1c4 
>   kdeui/widgets/ktextedit.cpp dfaf5b8 
> 
> Diff: http://git.reviewboard.kde.org/r/107678/diff/
> 
> 
> Testing
> -------
> 
> Tested KLineEdit in Dolphin "add place" dialog and KDevelop. Tested KTextEdit with Gwenview "description" image field.
> 
> 
> Thanks,
> 
> Aurélien Gâteau
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20121213/193075da/attachment.htm>


More information about the kde-core-devel mailing list