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

Rustlings v6.0.1 no longer respects NO_EMOJI #2020

Closed
eliminmax opened this issue Jul 4, 2024 · 4 comments
Closed

Rustlings v6.0.1 no longer respects NO_EMOJI #2020

eliminmax opened this issue Jul 4, 2024 · 4 comments

Comments

@eliminmax
Copy link

The new version of rustlings shows emoji even when the NO_EMOJI environment variable is set to a non-empty value.
Specifically, the crab emoji appears in the 'when you are done experimenting...' message whether or not it's set.

@mo8it
Copy link
Contributor

mo8it commented Jul 4, 2024

Could you please explain why not having emojis is important?

@eliminmax
Copy link
Author

I don't mind them, but I know that #364 argued that excessive use of Emoji and color is an accessibility issue, leading to the support for the NO_EMOJI environment variable being merged in #676, so I figured I'd open an issue.

There was an issue I ran into with the interaction between the byobu wrapper around tmux and rustlings 5.6.1 that caused text to overwrite the dividing line between tmux panes on lines that had emoji and text longer than the terminal pane width, but I think the bug there is almost certainly not in rustlings. The use of emoji in v6.0.1 does not appear to cause the same bug, and that bug was the only reason I added NO_EMOJI to my .profile as a lazy workaround.

@mo8it
Copy link
Contributor

mo8it commented Jul 5, 2024

Yes, but version 6 uses less emojis than before. So I don't think that the usage is excessive.

I am closing the issue, but if people still see this as a problem, I can remove that crab emoji. I prefer removing emojis over the overhead of supporting NO_EMOJI.

About the problem with tmux: This is a tmux issue. Just in case you are interested in an alternative: Zellij is awesome!

@mo8it mo8it closed this as completed Jul 5, 2024
@eliminmax
Copy link
Author

That's an understandable decision. Thanks for explaining.

Not sure about switching to zellij just yet, but I've heard good things about it.

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

No branches or pull requests

2 participants