Summary: | "Backspace" silently assigned to "Remove Point" after set custom shortcut to "Fill with Background Color" | ||
---|---|---|---|
Product: | [Applications] krita | Reporter: | mvowada <freebox64> |
Component: | General | Assignee: | Krita Bugs <krita-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | halla, nb64367 |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Ubuntu | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/calligra/c02946c86c44a86a912b5c2100db8eaec5d92209 | Version Fixed In: |
Description
mvowada
2014-12-28 07:52:36 UTC
[I am a GCI student] For me Remove point is already assigned Backspace from the beginning which is no problem as long as the two functions can't be used at the same time. It is not a key or a combination of keys that is assigned to a function, but the other way around. Hi, you're right, they're separate functions. Do you think could be superfluous alerting the users when switching back to default shortcut? Summarizing: - "Remove point" isn't listed in the shortcuts list at launch of Krita - "Remove point" will appear as soon as a document is opened in Krita - Therefore a custom shortcut for "Fill" has nothing to do with the appearing of the "Remove point" entry in the shortcuts list - By default "Remove point" will share the same key "Backspace" of "Fill" - The switching back from a custom key to the default one (Backspace) in "Fill", opens an alert dialog with the message: 'The "Backspace" shortcut is ambiguous with the following shortcut. Do you want to assign an empty shortcut to this action? Shortcut 'Backspace' for action 'Remove point'' [I am a GCI student] I think it is strange to use Backspace twice by default. But this was chosen as default so I suggest: -change default shortcut -if a shortcut ambiguity occurs on defaults it should not display a warning Thank you for your reply. In my opinion double shortcuts reveal to be versatile using different tools, like i.e. when editing of vectors or when filling the canvas with a color (being in this case the shared key "Backspace"). I can see why multiple functions could share the same key shortcut, and I think this is a strong point. That said, I find a bit: - redundant the alert message when switching back to the default key - not clear to me why not all the shortcuts are appearing in the shortcuts list at Krita launch, without opened documents. [I am a GCI student] Point 1: true Point 2: I agree. People don't want to open a document to edit their shortcuts. However I understand that usually it is good only to show the functions that can be used at a certain mode. "People don't want to open a document to edit their shortcuts. " That's something I'm working on fixing... For a bunch of reasons, mostly to do with inertia and a long, long history, this is pretty tough. But we'll get there :-) Git commit c02946c86c44a86a912b5c2100db8eaec5d92209 by Boudewijn Rempt. Committed on 03/01/2015 at 11:06. Pushed by rempt into branch 'calligra/2.9'. Create a master action collection for the shortcut editor This is still a half-way solution, ideally we'd just create actions from the .action file and fetch those actions in the code where we need an action, so there's still duplication. If we make the master action collection the source for new actions, we will also have to make the krita.action file translatable. If you create an action which should have an editable shortcut, you need to add that to a .action file. Make sure that all the text is exactly the same, so it gets translated in the shortcut dialog. On closing the shortcut dialog, changed shortcuts are saved. All mainwindows's actioncollection will re-read the configuration so they are synchronized. There's no local krita.rc file anymore, so the windows won't be able to save the shortcuts to the local krita.rc, all shorcuts are saved in the kritarc config file. Related: bug 313218, bug 342184, bug 337368 A +336 -0 krita/krita.actions M +1 -14 krita/ui/KisMainWindow.cpp M +92 -3 krita/ui/KisPart.cpp M +19 -0 krita/ui/KisPart.h M +0 -23 krita/ui/KisView.cpp M +0 -22 krita/ui/KisViewManager.cpp http://commits.kde.org/calligra/c02946c86c44a86a912b5c2100db8eaec5d92209 |