Ticket #862 (closed defect: fixed)
Opened 2011-11-04T14:08:56-05:00
Last modified 2011-11-04T17:15:14-05:00
Zoom In and Zoom Out menu entries do not have correct accelerators
| Reported by: | bdezonia | Owned by: | curtis |
|---|---|---|---|
| Priority: | major | Milestone: |
|
| Component: | Core | Version: | |
| Severity: | serious | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: |
Description
The zoom in and zoom out menu commands have no accelerators defined. This was in response to fixing bug #861. Right now they have poor hints to show which keys to press to activate zooms.
We need to renable the keyboard accelerators. However the AcceleratorHandler has to be smart enough to not fire when the menu handler will fire a zoom plugin rather than the AcceleratorHandler. This would be very kludgy and somewhat platform specific (control vs. cmd/meta).
Note that if we just reenable the zoom accel keys in the plugins then depending upon the key typed sequences zoom happens once or twice.
The real fix to this problem is to allow plugins to have more than one hotkey defined. It could display a default but react to numerous ones. If we had a accelerator definition like "any MINUS" (as opposed to "control MINUS") then we could zoom out when minus is typed regardless of modifier key status.
Fixed in d4b29390057791d92d1d627aa52bd1d799044889.