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

[Brisset, Nicolas]

These changes look great!  Much more sensible.  Thanks!

Thanks for the feedback (especially as it's overall positive!)

 

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.

OK, we can remove it. It's also bad if we click it inadvertently.

 

  -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 do like to have them here. I use them quite a lot.

 

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

I see it as an "axis tie option" just like tied zoom... But it's a bit
confusing, I agree. Still, that's the best place I could find for it. It
is a "Mode" toolbar, not a "zoom" one. Maybe that's a bit more open.

 

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

Yes, when we reintroduce those zoom modes we'll have to do something.
They're clearly exclusive, and I think the approach I best like is as we
had at some point in kst 1.x: a "combobutton". Do you know what I mean?

 

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

Agreed, as I said in the commit message it still needs to be improved.
The source is taken from oxygen and the .svgz it came from is in
src/images/svg_icons/tiedzoom.svg. I wanted to make it more contrasted
but it was getting late yesterday :-) For small sizes (we have 22x22
right now) it definitely needs to be tweaked a bit.

 

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

I have no issue with that. Or we do the easy changes right away before
we forget and add the ideas to the zoom modes to the bug I opened
yesterday. I'll try to do the changes today or tomorrow, I'll leave you
the X-only and Y-only stuff to have fun with :-)

Nicolas



Eurocopter Deutschland GmbH
Sitz der Gesellschaft/Registered Office: Donauwoerth
Registergericht/Registration Court: Amtsgericht Augsburg HRB 16508
Vorsitzender des Aufsichtsrates/Chairman of the Supervisory Board: Dr. Lutz Bertling
Geschaeftsfuehrung/Board of Management:
Dr. Wolfgang Schoder, Vorsitzender/CEO; Friedrich-Wilhelm Hormel; Ralf Barnscheidt

CONFIDENTIALITY NOTICE 

This communication and the information it contains is intended for the addressee(s) named above and for no other persons or organizations. It is confidential and may be legally privileged and protected by law. The unauthorized use, copying or disclosure of this communication or any part of it is prohibited and may be unlawful. 
If you have received this communication in error, kindly notify us by return e-mail and discard and/or delete the communication. Thank you very much. 
It is possible for e-mails to be intercepted or affected by viruses. Whilst we maintain virus checks on our e-mails, we accept no liability for viruses or other material which might be introduced with this message. 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kst/attachments/20100923/20d12243/attachment-0001.htm