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

fix: force overlay LTR #4943

Merged
merged 1 commit into from
Sep 16, 2021
Merged

fix: force overlay LTR #4943

merged 1 commit into from
Sep 16, 2021

Conversation

MaMrEzO
Copy link
Contributor

@MaMrEzO MaMrEzO commented Sep 16, 2021

Make overlay window readable(Force it in LTR) in RTL development environment.

Description

In RTL develop environment the whole overlay direction and text-align is in RTL and right to left. which make it har to read!

Make overlay window readable in RTL development environment.
@MaMrEzO MaMrEzO changed the title Fore overlay LTR Force overlay LTR Sep 16, 2021
@Shinigami92 Shinigami92 changed the title Force overlay LTR fix: force overlay LTR Sep 16, 2021
@Shinigami92
Copy link
Member

Could you provide a screenshot? Just for clarification

@Shinigami92 Shinigami92 added the p2-nice-to-have Not breaking anything but nice to have (priority) label Sep 16, 2021
@Shinigami92
Copy link
Member

Optional: you could add a comment to the change for now, maybe we want to support RTL later. Who knows.

@MaMrEzO
Copy link
Contributor Author

MaMrEzO commented Sep 16, 2021

This is the current behavior:
Screenshot_20210916_220638

But must be:
Screenshot_20210916_220740

@patak-dev patak-dev merged commit f8d8b73 into vitejs:main Sep 16, 2021
@patak-dev
Copy link
Member

Do tools like terminals normally include an option to set your preference to RTL? Is this something that we should add to Vite?

@MaMrEzO
Copy link
Contributor Author

MaMrEzO commented Sep 16, 2021

Do tools like terminals normally include an option to set your preference to RTL? Is this something that we should add to Vite?

For clarification, we set that manually, No there is no any option to do that in terminal tools.
But a question, I did a npm update vite but this merge not included! If this will includes in next version, when we can get it?

@patak-dev
Copy link
Member

This is going to be released next Monday as part of the 2.6.0-beta.0

aleclarson pushed a commit to aleclarson/vite that referenced this pull request Nov 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p2-nice-to-have Not breaking anything but nice to have (priority)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants