GNOME Bugzilla – Bug 118083
ImageMap, Delete Key should delete/clear instead of Ctrl+K
Last modified: 2005-05-19 18:25:26 UTC
As Gimp is now GTK2 the delete key can be used for delete instead of Ctrl+K which I expect was due to a limitation in GTK1 (dont have a copy of Gimp 1.3.x on this machine but I think the GIMP uses Delete key for Delete/Clear. I'll check when i get to my other machine).
Trivial change. Will commit asap.
GIMP has always used Ctrl-K for Clear and still uses it. I don't think this was due to any GTK+ limitations. You should not change the ImageMap plug-in since that would be inconsistent with the GIMP application. Propose to close as NOTABUG.
Ok, I will keep this consistent with the GIMP menu. However, I still think that both GIMP and the plug-in should use the Delete key so we move a little bit closer to HIG compliancy. Is there any reason for not doing this?
Historically Delete was used to clear any dynamically assigned GTK menu keybindings, which meant that it was unavailable for use as the Delete shortcut Backspace also cleared the keybinding In GTK2 only Backspace is available for clearing menu keybindings, Delete is available for deleting things. I dont think this bug should be closed as NOTABUG however I do agree that ImageMap should match the GIMP
oops posted prematurely too many windows open created a bug report to get the change made in GIMP first http://bugzilla.gnome.org/show_bug.cgi?id=118115
Changing target milestone to future (for several of my bugs at once) as requested by the developers. Most of my reports are things that would be nice but non-essentail and certainly are not blockers for 2.0.
I have given this some thought and I think the Imagemap `Clear' should be renamed to `Delete'. There is absoluty no relation between the GIMP use of the word `Clear' (clearing an area" and the Imagemap `Clear' (deleting an object). Sven: do you agree with this renaming action? Since it's so trivial and has been around for more than a year I want to close this report asap.
Sure, I agree.
Fixed in current CVS during the move to action based menus: 2005-01-23 Maurits Rijk <m.rijk@chello.nl> * plug-ins/imagemap/imap_toolbar.[ch] * plug-ins/imagemap/imap_tools.[ch]: removed * plug-ins/imagemap/Makefile.am * po-plug-ins/POTFILES.in: changed accordingly * plug-ins/imagemap/imap_grid.c * plug-ins/imagemap/imap_main.[ch] * plug-ins/imagemap/imap_menu.[ch] * plug-ins/imagemap/imap_misc.[ch] * plug-ins/imagemap/imap_object_popup.c * plug-ins/imagemap/imap_selection.c: more conversion to action based menus. Still have to attach some actions.
Verified against gimp 2.3