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

Replace onTab for keyBindingFn #69

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

erikmartinjordan
Copy link

· Solves onTab warnings. onTab is deprecated and after key pressing it, it doesn't indent the text.

· Solves line indention on selecting text adding spaces at the start of the line.

· onTab is deprecated and after key pressing it, it doesn't indent the text.

· Indent line on selection adding spaces at the start of the line.
@kristiandupont
Copy link

I am not sure what the test failure indicates. Is this hard to fix? I would really like to get rid of the warning in my console..

@erikmartinjordan
Copy link
Author

@kristiandupont The error in the test is due to my lack of permissions in the repository. I hope the author has time to take a look. 🤔

@kristiandupont
Copy link

@ngs sorry to bother you, just wondering if this might be simple to merge?

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.

2 participants