Arrow keys don't work in Gerbview

Bug #1795855 reported by John Beard
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
KiCad
Fix Released
Undecided
John Beard

Bug Description

On the current master (db5386467), the arrow keys don't work in Gerbview to pan the view or move the cursor.

In Pcbnew:

* Arrow key: move cursor 1 pixel
* Shift-Arrow: pan view
* Ctrl-Arrow: move cursor one grid interval

Gerbview should work in the same way.

Tags: gerbview
Revision history for this message
Seth Hillbrand (sethh) wrote :

Please add your full version information from Help->About KiCad. Many of these issues are system-dependent, so keeping track of this critical to ensuring we address the issue correctly.

Revision history for this message
John Beard (john-j-beard) wrote :

This isn't platform related, it's because the GERBVIEW_CONTROL class doesn't bind the cursor TOOL_ACTIONS in GERBVIEW_CONTROL::setTransitions.

In PCBNEW_CONTROL and CVPCB_CONTROL, these are bound.

In, say, Pcbnew, the actions are declared and defined. For the "Up" key:

pcbnew_actions.h

    static TOOL_ACTION cursorUp;

pcbnew_control.cpp:

    TOOL_ACTION PCB_ACTIONS::cursorUp( "pcbnew.Control.cursorUp",
        AS_GLOBAL, WXK_UP, "", "", NULL, AF_NONE, (void*) CURSOR_UP );

In Gerbview, they are only declared in gerbview_actions.h, never defined, or used.

Changed in kicad:
assignee: nobody → John Beard (john-j-beard)
Revision history for this message
KiCad Janitor (kicad-janitor) wrote :

Fixed in revision 445a9fa963690642e14b4bb229db7aea00efddb6
https://git.launchpad.net/kicad/patch/?id=445a9fa963690642e14b4bb229db7aea00efddb6

Changed in kicad:
status: New → Fix Committed
Changed in kicad:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.