[GRASS-git] [OSGeo/grass] e7fded: wxGUI: Fix key binding issues on mac (#818 #1160)

nilason noreply at github.com
Tue Jan 5 02:27:26 PST 2021


  Branch: refs/heads/releasebranch_7_8
  Home:   https://github.com/OSGeo/grass
  Commit: e7fded2202886dc9f6aa60dc562cc394d952a084
      https://github.com/OSGeo/grass/commit/e7fded2202886dc9f6aa60dc562cc394d952a084
  Author: nilason <n_larsson at yahoo.com>
  Date:   2021-01-05 (Tue, 05 Jan 2021)

  Changed paths:
    M gui/wxpython/dbmgr/manager.py
    M gui/wxpython/dbmgr/sqlbuilder.py
    M gui/wxpython/gui_core/forms.py
    M gui/wxpython/gui_core/goutput.py
    M gui/wxpython/gui_core/wrap.py
    M gui/wxpython/lmgr/frame.py
    M gui/wxpython/lmgr/pyshell.py
    M gui/wxpython/modules/import_export.py
    M gui/wxpython/modules/mcalc_builder.py

  Log Message:
  -----------
  wxGUI: Fix key binding issues on mac (#818 #1160)

Fixes #785 (backport of 5fd8c29)

The main problem with stock id buttons is that they all come with a key
binding. E.g. Close, Clear and Copy have the same key binding ctrl+c and
causes problems at least for mac. To remedy this issue we are using
subclassed versions of Button: ClearButton, CancelButton, ApplyButton).

I also made some effort to implement ESC to close dialog (attribute
table manager, mapcalc).

Note: for module dialogs (forms.py) and Map calculator I have replaced
ctrl+c to copy selected text, not to copy the whole command.

This also addresses a subsequent issue (#1156) introduced by 5fd8c29,
with an added check for wxPython version (backport of 1ee82f0).




More information about the grass-commit mailing list