[Kst] [Bug 249799] UI usability: reorganize kst2 menus

[Barth Netterfield]

These changes look great!  Much more sensible.  Thanks!

Minor comments:

  -why is 'exit application' in the toolbar?  The window manager already has
such a button, and I've never seen it in any other apps' toolbars.

  -For the sake of reducing the number of toolbar entries: Do we really want
save, open, and print in the toolbar?  I'm not sure.  (some other apps do,
others don't.)

  -I don't think that 'shared axis box' belongs with the zoom options.
 Although it's not really an annotation, it is more like an annotation, than
a mouse mode... maybe.

  -would it be better to turn the Layout Mode toggle into a radio set
between the upcoming 'x zoom mode', 'y zoom mode', 'xy zoom mode' and
'layout mode'?  This is what okular does in the same situation.  Takes more
space, but might be more clear.

  -where did the tied zoom icon come from?  Do you have svg source?  The
idea is right, but it looks sort of murky.

  -should we re-open the bug for these comments?

On Wed, Sep 22, 2010 at 5:03 PM, Nicolas Brisset <
nicolas.brisset at eurocopter.com> wrote:

> https://bugs.kde.org/show_bug.cgi?id=249799
>
>
> Nicolas Brisset <nicolas.brisset at eurocopter.com> changed:
>
>           What    |Removed                     |Added
>
> ----------------------------------------------------------------------------
>             Status|NEW                         |RESOLVED
>         Resolution|                            |FIXED
>
>
>
>
> --- Comment #10 from Nicolas Brisset <nicolas brisset eurocopter com>
>  2010-09-22 23:03:04 ---
> SVN commit 1178378 by brisset:
>
> Finish the first round of menu/toolbar reorganization. This change has one
> very
> visible impact: the create actions are
> moved from the data manager to the "Create" menu. There was a little bit of
> discussion on that a while ago and it seems to be a
> recurrent question from users (why are they here anyway?), plus it uses up
> valuable screen real estate. And they were almost
> always hidden, which wasn't so nice.
>
> I'd suggest trying this for a while and then deciding whether we want to
> re-add
> something to the data manager. The QActions are
> now created from mainwindow.cpp, we can add them to a QToolBox which we
> export
> to the data manager.
> I'd also appreciate if a real C++ developer checked the changes. It is a
> bit
> more than shuffling around some code and I'm not
> 100% sure of everything.
>
> For the other small points (list of recent files, x-only and y-only zoom,
> ...)
> buried in the comments of this bug, I'll create
> separate entries. Some icons are also waiting to be improved, but we can do
> that bit by bit and it is not directly the intention
> of this bug.
>
> BUG: 249799
>
>
>  M  +1 -94     datamanager.cpp
>  M  +0 -11     datamanager.ui
>  M  +6 -10     libkstapp.pro
>  M  +26 -1     mainwindow.cpp
>  M  +4 -1      mainwindow.h
>  A             pluginmenuitemaction.cpp   [License: GPL (v2+)]
>  A             pluginmenuitemaction.h   [License: GPL (v2+)]
>
>
> WebSVN link: http://websvn.kde.org/?view=rev&revision=1178378
>
> --
> Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
> ------- You are receiving this mail because: -------
> You are the assignee for the bug.
> _______________________________________________
> Kst mailing list
> Kst at kde.org
> https://mail.kde.org/mailman/listinfo/kst
>



-- 
C. Barth Netterfield
University of Toronto
416-845-0946
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kst/attachments/20100923/33ef615a/attachment.htm