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

Do not use standardKeymap in vim mode #1145

Merged
merged 1 commit into from
Nov 11, 2024
Merged

Conversation

lutzky
Copy link
Contributor

@lutzky lutzky commented Nov 9, 2024

The standard keymap conflicts with the vim mode, e.g. not allowing arrow keys to move through the vim commandline for / and : commands. The keys described in
https://codemirror.net/docs/ref/#commands.standardKeymap seem to work just fine without it in vim mode.

Fixes #976

The standard keymap conflicts with the vim mode, e.g. not allowing arrow
keys to move through the vim commandline for `/` and `:` commands. The
keys described in
https://codemirror.net/docs/ref/#commands.standardKeymap seem to work
just fine without it in vim mode.

Fixes silverbulletmd#976
@zefhemel
Copy link
Collaborator

Alright, let's try, people will scream if this breaks stuff :)

@zefhemel zefhemel merged commit 4457927 into silverbulletmd:main Nov 11, 2024
1 check passed
@lutzky
Copy link
Contributor Author

lutzky commented Nov 11, 2024

Uhh... screamer here. I don't know how I missed this, but this breaks the Return key for ordinary paragraphs (it does nothing in Insert mode now). Super weird. Workaround is hitting o in Command mode instead; seeing as the previous version was destructive, this seems less bad, but still worth fixing. I'll take a look.

@lutzky
Copy link
Contributor Author

lutzky commented Nov 11, 2024

I'll move discussion back to #976.

lutzky added a commit to lutzky/silverbullet that referenced this pull request Nov 11, 2024
@lutzky lutzky mentioned this pull request Nov 11, 2024
zefhemel pushed a commit that referenced this pull request Nov 11, 2024
This is a workaround for replit/codemirror-vim#182

Fixes #1145 (harder)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug report: vim search mode would leak delete key
2 participants