[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: Review Request: Add button to allow changing iconText() in
From:       "Celeste Lyn Paul" <celeste () kde ! org>
Date:       2009-08-10 13:28:03
Message-ID: 20090810132803.16063.5575 () localhost
[Download RAW message or body]



> On 2009-08-09 20:24:05, Celeste Lyn Paul wrote:
> > Semantically, I would rather see "Change Label..." rather than "Text". However, \
> > I'm not sure if the user knows the distinction and if it would make a difference.
> 
> Christoph Feck wrote:
> Well, look at the toolbar menu ... there it is always called text (text only, text \
> under icon etc.)

Ah yes, very good point. Then I'm fine with Change Text...

+1


- Celeste Lyn


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


On 2009-08-09 17:39:25, Christoph Feck wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1243/
> -----------------------------------------------------------
> 
> (Updated 2009-08-09 17:39:25)
> 
> 
> Review request for kdelibs and usability.
> 
> 
> Summary
> -------
> 
> This patch adds a "Change Text..." button to the tool bar edit dialog. This helps \
> users who add many tool buttons to their tool bars, even those that have a quite \
> long text. For example, the user could change "View Document Source" to just \
> "Source". 
> The code respects any iconText() set by the application, and stores the changes \
> together with the changed icon in the user's ui.rc file of the respective \
> application. 
> It is intended that the changed text only shows in the right (active) action list, \
> as it does not affect the menu texts. 
> Maybe we should have a button to revert to default text (and icon) for individual \
> actions, I may add that separately. 
> 
> This addresses bug 201346.
> https://bugs.kde.org/show_bug.cgi?id=201346
> 
> 
> Diffs
> -----
> 
> /trunk/KDE/kdelibs/kdeui/dialogs/kedittoolbar.cpp 1009073 
> /trunk/KDE/kdelibs/kdeui/dialogs/kedittoolbar_p.h 1009073 
> 
> Diff: http://reviewboard.kde.org/r/1243/diff
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> KEditToolBar dialog with added "Change Text..." button
> http://reviewboard.kde.org/r/1243/s/167/
> 
> 
> Thanks,
> 
> Christoph
> 
> 


[prev in list] [next in list] [prev in thread] [next in thread]