This is a bit clearer, and reinforces the keyboard icon used for the menu item.
BUG: 39488
FIXED-IN: 5.66
If accepted, will wait until after tagging to land it so as not to break the string freeze.
ndavis |
VDG |
This is a bit clearer, and reinforces the keyboard icon used for the menu item.
BUG: 39488
FIXED-IN: 5.66
If accepted, will wait until after tagging to land it so as not to break the string freeze.
Automatic diff as part of commit; lint not applicable. |
Automatic diff as part of commit; unit tests not applicable. |
This doesn't seem wrong, but why is it needed? Do people get confused about the type of shortcuts? Are there non-keyboard shortcuts? If there are, would we put their configuration menu under a different menu option?
There was a bug report about it that had some people agreeing with it. I think it makes a bit of sense because yes, this dialog is indeed only about keyboard shortcuts, and at least to my ears, the phrase "keyboard shortcuts" instantly connotes what this is about, while "shortcuts" is a more generic term that doesn't have such an instantly recognizable computer-related meaning.
Please commit it after the commit for the new Frameworks is made (so probably from Sunday onwards).
Yep, in fact I mentioned this in the description section of the patch:
If accepted, will wait until after tagging to land it so as not to break the string freeze.
Oh sorry, I missed that. It's the kind of information I wouldn't expect in the commit message.