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

allowing unfocusing the editor field by keyboard for accessibility #515

Open
KiaraGrouwstra opened this issue Jan 4, 2023 · 2 comments
Open

Comments

@KiaraGrouwstra
Copy link

Is your feature request related to a problem? Please describe.
It would be nice to use this editor by keyboard, but currently codemirror 5's rebinding of the tab key makes this harder, as reported at go-gitea/gitea#20752.

Describe the solution you'd like
Ensure the tab key functions as normal (at least outside of markdown lists).
This could perhaps be achieved by patching codemirror, so as to not require a broader upgrade.

Describe alternatives you've considered
Having this repo update to codemirror 6 (#324 / #505). This appears to require a larger rewrite (#447), making it less viable as a short-term fix.

Additional context
Would you perhaps take PRs for this?
(I'm seeing there's some open ones still as well. Is there a plan for maintainership for this repo?)

@Ionaru
Copy link
Owner

Ionaru commented Jan 10, 2023

Would you perhaps take PRs for this?

I do accept pull requests, the ones that are currently open either have a stalled discussion or I haven't gotten around to testing the functionality.

KiaraGrouwstra added a commit to KiaraGrouwstra/easy-markdown-editor that referenced this issue Jan 11, 2023
KiaraGrouwstra added a commit to KiaraGrouwstra/easy-markdown-editor that referenced this issue Jan 11, 2023
@mrdavidlaing
Copy link

Any update on this issue?

Is specific; I'm wondering what prevented #518 from being merged.

Thanks!

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 a pull request may close this issue.

3 participants