Code review comment for lp:~themuso/ubiquity/maybe-ubiquity-a11y-profiles

Revision history for this message
Luke Yelavich (themuso) wrote :

One thing I forgot to add to merge comments. I added keypress events as well as keyboard shortcuts to menu items for the indicator menu items because 1) the keyboard shortcuts in the ui file alone don;t work naturally, so the keypress events are needed. 2) I couldn't work out how to show the shortcut keys in the menu any other way. If there is a better way, I'm happy to be pointed in the right direction.

« Back to merge proposal