Print command incorrectly override on merge request pages
When viewing a merge request, when pressing cmd-p
, it brings up a file search dialog rather than the standard Print dialog.
This goes against all muscle memory and expectations when using a browser.
One comment from slack
cmd+p is commonly the shortcut for jumping to files in other devtools (vs code, as well as both firefox and chrome JS debuggers) so that's probably why it was chosen
We shouldn't be overriding standard browser key bindings. It confuses the user and is non-standard.