Summary: | UI usability: reorganize kst2 menus | ||
---|---|---|---|
Product: | [Applications] kst | Reporter: | Nicolas Brisset <nicolas.brisset> |
Component: | ui | Assignee: | kst |
Status: | RESOLVED FIXED | ||
Severity: | wishlist | CC: | netterfield |
Priority: | NOR | ||
Version: | 2.0.1 | ||
Target Milestone: | 2.0.1 | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: |
Description
Nicolas Brisset
2010-09-01 23:08:01 UTC
There is also an inconsistent capitalization when menu items contain more than one word. Looking at KDE the standard seems to be only one capital on the first word. It should be fixed while we are at it. A couple more ideas: File menu: - add a "Recent files" sub-menu under Open... in the File menu and add a separator between open nd save actions (a la kst 1.x) - move "New tab" to the "Create" menu - move "Clear session" down between "Close tab" and "Exit" Edit menu: add copy/cut/paste when we have code to support those actions! Tools menu: some icons are missing I like the three new menu categories. Are you asking for X and Y only zoom modes, apart from using <ctrl> and <shift> modifiers? If so, we can discuss this separately. I agree that 'Create Layout' should be removed, because, as far as I can tell, it no longer does anything (except maybe break things...) Nicholas, do you want to take a stab at all this, or should I? SVN commit 1170831 by kuemmel: resort existing menu entries CCBUG:249799 M +50 -51 mainwindow.cpp M +2 -1 mainwindow.h WebSVN link: http://websvn.kde.org/?view=rev&revision=1170831 OK, looks good :-) I hope I'm not the only one who likes it... I'd still implement the extra things mentioned in comment #2, and then we have to cleanup the toolbars (that is, make them consistent with the menus in terms of names and contents). I'm not necessarily for an approach like 1 menu = 1 toolbar. For example, annotation objects could/should have a dedicated toolbar. Should I detail my idea regarding the layout of toolbars a bit more? And then of course, we are still missing icons. Time to contact the KDE icon creators? To Barth, comment #3: yes, I am talking about X-only and Y-only zoom modes from kst 1.x, without using modifiers (they're cool but lack "discoverability" => good for power users, but definitely worth a menu item, which is much more visible. Do we really need to discuss this separately? To Peter: create layout is still there, following Barth's comments I'd suggest removing it. I don't know how much code there is left to cleanup though, probably removing it from the interface is only part of the job... SVN commit 1171059 by kuemmel: remove "Create Layout" toolbutton CCBUG:249799 M +0 -19 mainwindow.cpp M +0 -2 mainwindow.h WebSVN link: http://websvn.kde.org/?view=rev&revision=1171059 SVN commit 1172397 by brisset: Try to reach a reasonable half-finished state for menus: - Reorganize a couple of things which did not really make sense (e.g "Create Shared Axis Box" in "Annotation") - Apply KDE capitalization rules as per http://techbase.kde.org/Projects/Usability/HIG/Capitalization - Reinstate the "Range" menu as in kst 1.x, and move the "Change Data Sample Range" tool there - Try to have a more or less consistent use of ellipsis (...) - not sure that's completely OK, though Still missing: - some icons, at least the old ones and why not some improved ones? - create plugin entries in a submenu (the same as in the data manager with the string changes suggested in bug 249805) => Peter/Barth, you have to take care of that one as it is a bit more complex, resulting from a scan of available plugins - X-only and Y-only zoom modes in the Mode menu - some toolbar cleanup CCBUG:249799 M +45 -19 mainwindow.cpp M +2 -1 mainwindow.h WebSVN link: http://websvn.kde.org/?view=rev&revision=1172397 SVN commit 1177262 by brisset: - Cleanup toolbars (don't create one for each menu, only frequently used actions - and grouped slightly differently than menus) - Re-add "event monitor" in the Create menu - Move some code around so that the order resembles that of the UI, otherwise it's very easy to get lost! It's still a bit ugly due to missing icons, but these are easy to add => next step CCBUG: 249799 M +217 -210 mainwindow.cpp M +8 -3 mainwindow.h WebSVN link: http://websvn.kde.org/?view=rev&revision=1177262 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 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? SVN commit 1181416 by brisset: Oops, there was no easy way to create the basic types any longer (the small buttons in other dialogs don't really count, do they?). I think I forgot them when moving creation stuff from the data manager to the "Create" menu. Now they're back :-) CCBUG: 249799 M +41 -1 mainwindow.cpp M +8 -0 mainwindow.h WebSVN link: http://websvn.kde.org/?view=rev&revision=1181416 There is no version list for "Version Fixed in". Use Target Milestone as indicator when the bug was fixed. |