posted 6 years ago
Right - got something I'm happy with. The passing focus thing didn't really work for me, so instead I'm defining key bindings for the table as well as menu item accelerators. It's a bit of a cheat because the accelerators don't actually get used - they're just a way of displaying the key bindings to the user.
I like it because there's no duplication of action name strings/keys etc. - the action is defined for example:
then the key binding is defined using:
and the menu item is defined using:
Full code - feedback welcome: