[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