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

make tab key insert tabs and set tab size to 4 in playground #1626

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

samualtnorman
Copy link
Contributor

these changes make it so that you can easily use tabs in the playground

@edemaine
Copy link
Collaborator

Is this accessible? How would a keyboard user navigate away from the text box? (We've discussed the escape key working in this case. Is that automatic somehow or do you need to add code?)

@STRd6
Copy link
Contributor

STRd6 commented Nov 29, 2024

I think using escape to unfocus the code editor and allow navigating to the next interactive element with tab is a good idea.

It would also be nice to indent every line within the selection, if present, as well.

@samualtnorman
Copy link
Contributor Author

escape to unfocus sounds like a good idea, indenting every selected line sounds a little tricky to implement so probably out of scope for this PR

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.

3 participants