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

2011-08-14 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102262/#review5682 --- This review has been submitted with commit 3cc10496aa6eb4aae27b

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

2011-08-14 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102262/#review5681 --- This review has been submitted with commit ddcf5c0fd58bbe4e8572

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

2011-08-11 Thread Yoann Laissus
> On Aug. 11, 2011, 9:13 a.m., David Faure wrote: > > Ah, I see. Indeed after reading QAction::iconText() it's all clear ;) > > > > But yeah this is quite obviously a bug in qaction (qt_strippedText), which > > should remove "..." only at the end. It's my first patch in kdelibs, where should i

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

2011-08-11 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102262/#review5621 --- Ship it! Ah, I see. Indeed after reading QAction::iconText() it

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

2011-08-10 Thread Yoann Laissus
> 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.

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

2011-08-10 Thread Christoph Feck
> 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.

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

2011-08-09 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102262/#review5554 --- Since you're setting iconText, this is about actions into a tool

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

2011-08-09 Thread Yoann Laissus
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102262/ --- Review request for kdelibs. Summary --- When a bookmark name, is too