pcb

The accel key for the settings menu clashes with the accel to increase drill size

Bug #1496603 reported by KaiMartin on 2015-09-16
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
gEDA project
Low
Unassigned
pcb
Low
Unassigned

Bug Description

Current git head defines the accel to open the settings menu as [alt-S]. This collides with the accel to increase the drill size of the via or pin under the mouse cursor. Apparently, the menu takes catches the event first. This renders holes impossible to increase via the keyboard.

The drill accel is part of a whole group of related actions. The accels [s], [shift-s], [alt-s], [shift-alt-s] all affect vias and pins. So it's better to not change these in order to resolve the collision.

Proposal: change the accel for the settings menu to [alt-t]. This combo does not seem to be used anywhere.

I attached a rather trivial patch to gpcb-menu.res.in which does just this.

---<)kaimartin(>---

Traumflug (mah-jump-ing) on 2015-09-27
Changed in geda-project:
importance: Undecided → Low
summary: - the accel key for the settingss menu clashes with the accel to increase
+ The accel key for the settingss menu clashes with the accel to increase
drill size
summary: - The accel key for the settingss menu clashes with the accel to increase
+ The accel key for the settings menu clashes with the accel to increase
drill size
KaiMartin (kmk-familieknaak) wrote :

The menu accel keys have been added to pcb fairly recently. Since then the accel to increase hole size is broken. I for one use this feature with almost every layout.
How come. this regression is considered "low" importance? I'd have expected a 'medium'.

---<)kaimartin(>---

Changed in pcb:
status: New → Confirmed
importance: Undecided → Low
milestone: none → next-bug-release
Bert Timmerman (bert-timmerman) wrote :

Alt+V for the "View" pull down menu is affecting the "Via Tool drill + 5 mil" too.

Maybe better to drop the whole accel key for pull down menus functionality.

Kind regards,

Bert Timmeman.

Changed in pcb:
milestone: next-bug-release → future-bug-release
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers