Review Request: Fix missing "..." in KBookmarkAction displayed text

Christoph Feck christoph at maxiom.de
Wed Aug 10 11:38:10 BST 2011



> On Aug. 9, 2011, 10:45 p.m., David Faure wrote:
> > Since you're setting iconText, this is about actions into a toolbar, not into a menu, right?
> > 
> > So Qt strips "..." from toolbar button texts? What's the logic there? I'm just curious, the patch looks ok, from your description.

The logic behind this stripping is to show "Open" instead of "Open..." on toolbar buttons.

Ideally, Qt would only strip trailing ellipsis, but it doesn't, probably to handle trailing spaces etc.


- Christoph


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


On Aug. 9, 2011, 3:32 p.m., Yoann Laissus wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102262/
> -----------------------------------------------------------
> 
> (Updated Aug. 9, 2011, 3:32 p.m.)
> 
> 
> Review request for kdelibs.
> 
> 
> Summary
> -------
> 
> When a bookmark name, is too long, its name is truncated with "..." at the middle.
> But, QAction strip those three dots by default.
> This patch solves this issue by defining imageText. 
> 
> 
> Diffs
> -----
> 
>   kio/bookmarks/kbookmarkmenu.cc 873f4a8 
> 
> Diff: http://git.reviewboard.kde.org/r/102262/diff
> 
> 
> Testing
> -------
> 
> It works with Konqueror and rekonq.
> 
> 
> Thanks,
> 
> Yoann
> 
>

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


More information about the kde-core-devel mailing list