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

Conversation

@zanqi
Copy link
Contributor

@zanqi zanqi commented Nov 3, 2012

CodeMirror do not have "copy", "cut" and "paste" commands. These extrakeys are ignored by CodeMirror.

@ghost ghost assigned njx Nov 5, 2012
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI, these shortcuts work on Win7, but not Mac 10.6, for me.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But is that the same both before and after this change? If so then it should be safe to remove...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change should be transparent. It never added new shortcuts for Brackets. What copy, cut and paste shortcuts are depends on the OS.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, I verified that those shortcuts still work on Win7 in this branch. Good catch!

@ghost ghost assigned redmunds Nov 5, 2012
@redmunds
Copy link
Contributor

redmunds commented Nov 5, 2012

Looks good. Merging.

redmunds added a commit that referenced this pull request Nov 5, 2012
Remove unnecessary extrakey for CodeMirror
@redmunds redmunds merged commit 0ae8804 into adobe:master Nov 5, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants