PLT-2755 Shortcut to upload file #2848
Conversation
|
Thanks @DavidLu1997 for the pull request! Per the CONTRIBUTING.md file displayed when you created this pull request, we need to add you to the list of approved contributors for the Mattermost project. Please help complete the Mattermost contribution license agreement? This is a standard procedure for many open source projects. Your form should be processed within 24 hours and reviewers for your pull request will be able to proceed. Please let us know if you have any questions. We are very happy to have you join our growing community! |
|
|
||
| document.addEventListener('keydown', (e) => { | ||
| //CTRL+U or CMD+U for file uploads | ||
| if ((e.ctrlKey || e.metaKey) && e.keyCode === 85) { |
There was a problem hiding this comment.
Can you add a constant to KeyCodes in utils/contants.jsx for 85 and use that here instead?
It would look like
if ((e.ctrlKey || e.metaKey) && e.keyCode === Constants.KeyCodes.U) {
for example.
|
Looks good! +1 |
|
+1 |
coreyhulen
added
4: Reviews Complete
Added code to use CTRL+U and CMD+U to open the file upload dialog.
Note: CTRL+U opens page source in the latest versions of Firefox and Chrome. CMD+U is currently untested, due to lack of a CMD button.