After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 527565 - Tool Keyboard shortcuts broken
Tool Keyboard shortcuts broken
Status: RESOLVED FIXED
Product: dia
Classification: Other
Component: general
CVS head
Other All
: Normal normal
: 0.97
Assigned To: Dia maintainers
Dia maintainers
: 558222 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2008-04-11 15:51 UTC by Steffen Macke
Modified: 2009-01-31 14:50 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Steffen Macke 2008-04-11 15:51:59 UTC
This came up on the mailing list:

Not all of the keyboard shortcuts are working right now.

For example: Shift + Alt + M for the Magnify tool.

Reproduced on Windows and Linux
Comment 1 Hans Breuer 2008-04-12 07:27:58 UTC
It is working fine for me with the "classic theme" (no menubar, no integrated GUI). Some information regarding he Gtk+ version used may also help.
Comment 2 Steffen Macke 2008-04-12 12:40:36 UTC
GTK+ 2.12.0
Problem exists with or without Integrated UI.
Comment 3 Hans Breuer 2008-04-12 14:26:19 UTC
it does _not_ work for me only with the integrated ui (now checked on Linux as well)
Comment 4 Rolf Leggewie 2008-08-10 04:24:29 UTC
actually, Ctrl+Alt+M reliably crashes dia for me on hardy.  Many keyboard shortcut actions seems to crash dia here.  Same of them reliably, others not.

https://bugs.launchpad.net/ubuntu/+source/dia/+bug/256519
Comment 5 Hans Breuer 2008-08-10 14:36:26 UTC
Can't make it crash with the very same version. Strange thing is: alt-shift-M does not work after starting with "Use menu bar" (the default). It starts to work after switching off the menu bar. It remains to work when switching on the menu bar again in the same session.
Seems there is something not properly initialized for the display creation with use menu bar (or integrated).
Comment 6 Hans Breuer 2008-08-10 14:47:22 UTC
bug #547173 adds a more detail description (select two objects, alt-shift-M) which does not crash on me either. But it shows a possible further problem: the two objects are aligned to middle, i.e. Tools/Magnify and Objects/Align/Middle are sharing the same shortcut.
Comment 7 Hans Breuer 2008-08-10 14:52:56 UTC
sharing the same shortcut may be fine because Objects/Align/Middle does only work with a selection of objects, switching to Magnify could be enabled for the other cases ...
Comment 8 Rolf Leggewie 2008-09-20 15:06:30 UTC
(In reply to comment #7)
> sharing the same shortcut may be fine because Objects/Align/Middle does only
> work with a selection of objects, switching to Magnify could be enabled for the
> other cases ...

Then again, it could have something to do with the problem the OP is experiencing.  Both commands being assigned the same shortcut not only likely leads to confusion, but it is also not possible to magnify with a shortcut when two objects are selected.
Comment 9 Hans Breuer 2008-10-28 18:41:46 UTC
*** Bug 558222 has been marked as a duplicate of this bug. ***
Comment 10 Hans Breuer 2009-01-31 11:00:13 UTC
part of the problem is app/interface.c:1715

  /* TODO: integrated-UI --> Figure out how to deal with updateable menu items!!! 
  menus_initialize_updatable_items (&ddisp->updatable_menu_items, ddisp->actions); */

but there are more issues - all that ui-manager code is crying for a cleanup
Comment 11 Hans Breuer 2009-01-31 14:50:36 UTC
2009-01-31  Hans Breuer  <hans@breuer.org>

	* app/display.[ch] app/diagram.[ch] app/menus.[ch] : get rid of nearly
	unmaintainable UpdatableMenuItems and a lot of ui-variant-specific code
	by dynamic action lookup when updating sensitivity. This not only fixes
	bug #527565 but also bug #322190 and allows ...
	* app/interface.c : make Tool accelerator single key strokes - e.g. no
	more <alt><shift>L to draw a line but just "L" - another benefit of the
	dedicated test edit mode ;) - bug #439854.