Re: Review Request: Add button to allow changing iconText() in KEditToolBar

View: New views
10 Messages — Rating Filter:   Alert me  

Parent Message unknown Re: Review Request: Add button to allow changing iconText() in KEditToolBar

by Bugzilla from christoph@maxiom.de :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message


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

(Updated 2009-08-09 17:26:54.042628)


Review request for kdelibs and usability.


Changes
-------

Added usability to reviewers.


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
-------


Thanks,

Christoph

_______________________________________________
kde-usability mailing list
kde-usability@...
https://mail.kde.org/mailman/listinfo/kde-usability

Re: Review Request: Add button to allow changing iconText() in KEditToolBar

by Bugzilla from christoph@maxiom.de :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message


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

(Updated 2009-08-09 17:39:25.008942)


Review request for kdelibs and usability.


Changes
-------

Added a screen shot. Demonstrated here is changing the "Home Page" text to just "Home". As you can see, the original text or description is displayed below the lists, in case the user cannot remember the meaning of his abbreviation.


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

_______________________________________________
kde-usability mailing list
kde-usability@...
https://mail.kde.org/mailman/listinfo/kde-usability

Re: Review Request: Add button to allow changing iconText() in KEditToolBar

by Celeste Lyn Paul :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message


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


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.

- Celeste Lyn


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
>
>

_______________________________________________
kde-usability mailing list
kde-usability@...
https://mail.kde.org/mailman/listinfo/kde-usability

Re: Review Request: Add button to allow changing iconText() in KEditToolBar

by Bugzilla from christoph@maxiom.de :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message



> 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.

Well, look at the toolbar menu ... there it is always called text (text only, text under icon etc.)


- Christoph


-----------------------------------------------------------
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
>
>

_______________________________________________
kde-usability mailing list
kde-usability@...
https://mail.kde.org/mailman/listinfo/kde-usability

Re: Review Request: Add button to allow changing iconText() in KEditToolBar

by Celeste Lyn Paul :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message



> 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
>
>

_______________________________________________
kde-usability mailing list
kde-usability@...
https://mail.kde.org/mailman/listinfo/kde-usability

Re: Review Request: Add button to allow changing iconText() in KEditToolBar

by Parker Coates :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message


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


At first I was a bit suspicious of this change. Do we really *need* this functionality? It seems like overkill.

But the more I think about it, the more sense it makes. We have this wonderfully powerful platform with XMLGUI and sometimes I don't think we flaunt it enough. We already allow the user to change the icon and the shortcut of an action, so why not let them change the label too. It adds only a single button to a dialog most "basic" users will never see. The benefit isn't huge, but the cost is almost zero.

Of course, I hope you realise this opens the doors to all kinds of funny/nasty pranks. Switching the icons and labels of certain toolbar actions could make things quite interesting...

- Parker


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
>
>

_______________________________________________
kde-usability mailing list
kde-usability@...
https://mail.kde.org/mailman/listinfo/kde-usability

Re: Review Request: Add button to allow changing iconText() in KEditToolBar

by Celeste Lyn Paul :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message



> On 2009-08-10 13:54:06, Parker Coates wrote:
> > At first I was a bit suspicious of this change. Do we really *need* this functionality? It seems like overkill.
> >
> > But the more I think about it, the more sense it makes. We have this wonderfully powerful platform with XMLGUI and sometimes I don't think we flaunt it enough. We already allow the user to change the icon and the shortcut of an action, so why not let them change the label too. It adds only a single button to a dialog most "basic" users will never see. The benefit isn't huge, but the cost is almost zero.
> >
> > Of course, I hope you realise this opens the doors to all kinds of funny/nasty pranks. Switching the icons and labels of certain toolbar actions could make things quite interesting...

It would be awesome that in the future we could collect customized labels somehow. We could see if there are any trends and use that data to improve the labels upstream so people wouldn't have to change them at all. It would also really help translators who might be directly translating the labels from English to another language without really thinking of of the usability of the label in the other language/culture.


- Celeste Lyn


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


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
>
>

_______________________________________________
kde-usability mailing list
kde-usability@...
https://mail.kde.org/mailman/listinfo/kde-usability

Re: Review Request: Add button to allow changing iconText() in KEditToolBar

by Bugzilla from MathStuf@gmail.com :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message



> On 2009-08-10 13:54:06, Parker Coates wrote:
> > At first I was a bit suspicious of this change. Do we really *need* this functionality? It seems like overkill.
> >
> > But the more I think about it, the more sense it makes. We have this wonderfully powerful platform with XMLGUI and sometimes I don't think we flaunt it enough. We already allow the user to change the icon and the shortcut of an action, so why not let them change the label too. It adds only a single button to a dialog most "basic" users will never see. The benefit isn't huge, but the cost is almost zero.
> >
> > Of course, I hope you realise this opens the doors to all kinds of funny/nasty pranks. Switching the icons and labels of certain toolbar actions could make things quite interesting...
>
> Celeste Lyn Paul wrote:
>     It would be awesome that in the future we could collect customized labels somehow. We could see if there are any trends and use that data to improve the labels upstream so people wouldn't have to change them at all. It would also really help translators who might be directly translating the labels from English to another language without really thinking of of the usability of the label in the other language/culture.

A tool (opt-in) to send all/select ui.rc files in ~/.kde to some central repository (anonymously of course)?


- Ben


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


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
>
>

_______________________________________________
kde-usability mailing list
kde-usability@...
https://mail.kde.org/mailman/listinfo/kde-usability

Re: [KDE Usability] Review Request: Add button to allow changing iconText() in KEditToolBar

by macias :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

On Monday 10 August 2009 16:42:28 Ben Boeckel wrote:

> A tool (opt-in) to send all/select ui.rc files in ~/.kde to some
> central repository (anonymously of course)?

Remarks:
Why of course? It should be up to user -- data coming from registered
users is more reliable by definition.

On the other end translation maintainers could compile translations
basically relying on statistics.



I like this idea and is should not be limited only to toolbar.

Cheers,
_______________________________________________
kde-usability mailing list
kde-usability@...
https://mail.kde.org/mailman/listinfo/kde-usability

Re: [KDE Usability] Review Request: Add button to allow changing iconText() in KEditToolBar

by Bugzilla from faure@kde.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message


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

Ship it!


Nice addition.

- David


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
>
>

_______________________________________________
kde-usability mailing list
kde-usability@...
https://mail.kde.org/mailman/listinfo/kde-usability