Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Fixes bug #7667: Keyboard shortcut overlap with Chrome #8076

Closed
wants to merge 1 commit into from
Closed

Fixes bug #7667: Keyboard shortcut overlap with Chrome #8076

wants to merge 1 commit into from

Conversation

tacryt-socryp
Copy link

Changed shortcut for reloading without extensions to Opt-Shift-R and
created a browserForceReload function that is triggered by Cmd-Shift-R.
Changes to src/nls/root/strings.js will need to be localized for other
languages as well.

Changed shortcut for reloading without extensions to Opt-Shift-R and
created a browserForceReload function that is triggered by Cmd-Shift-R.
Changes to src/nls/root/strings.js will need to be localized for other
languages as well.
@iegik
Copy link

iegik commented Jun 10, 2014

Related to #8100 Ctrl+Alt v.s. AltGr conflict

@tacryt-socryp
Copy link
Author

This should resolve Issue #7667

@dangoor
Copy link
Contributor

dangoor commented Jun 12, 2014

Thanks for the pull request! (Triage complete... ready for review)

@tacryt-socryp
Copy link
Author

Thanks @dangoor! Looking forward to working more on Brackets.

@dangoor
Copy link
Contributor

dangoor commented Jun 17, 2014

@lkcampbell points out that #8080 is fixing the same problem

@tacryt-socryp
Copy link
Author

So will this be merged, or?

@dangoor
Copy link
Contributor

dangoor commented Jun 19, 2014

@loganallenc That's ultimately up to whoever reviews this. They'll take a look at both and decide if it makes sense to go with one or the other...

@dangoor dangoor assigned adragomir and dangoor and unassigned adragomir Jun 23, 2014
@dangoor
Copy link
Contributor

dangoor commented Jun 23, 2014

@loganallenc Thanks for this PR, but I think #8080 is a little closer to where we want to end up (keeps the command in LiveDevelopment, maps it to ctrl-Shift-R). The two pull requests are quite similar, and I appreciate you sending this our way.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants