Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#97 - Changed CMD+E to show Recent Files not Recent Projects popup #98

Merged
merged 3 commits into from
Dec 19, 2017

Conversation

spik3s
Copy link
Contributor

@spik3s spik3s commented Dec 6, 2017

Replaced workbench.action.openRecent command with workbench.action.quickOpenNavigateNext that better reflects IntelliJ's CMD+E shortcut's behaviour

Edit: Turns out the problems with shortcuts not working often occurs when too many actions are bound to a single shortcut. In case of CMD+E, to make it work I had to remove the default binding to actions.find and workbench.action.openRecent.

…popup + fix

Too many actions bound to the shortcut stopped it from working. I removed two default bindings 'actions.find' and 'workbench.action.openRecent'. The shortcut now works as expected.
@kasecato kasecato self-requested a review December 9, 2017 02:42
@kasecato kasecato self-assigned this Dec 9, 2017
@kasecato kasecato modified the milestones: v0.2.18, v0.2.17 Dec 9, 2017
@kasecato kasecato merged commit 21d4fe3 into kasecato:master Dec 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants