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 BiDi Text Issues App Wide #4959

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

Conversation

alirezaalavi87
Copy link

Added dir="auto" to all the elements that may have BiDi text.

This PR fixes the issue addressed in #4882

Added dir="auto" to all the elements that may have BiDi text
@mattermost-build
Copy link
Contributor

Hello @alirezaalavi87,

Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

@mattermost-build
Copy link
Contributor

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

@giuseongit
Copy link

@alirezaalavi87 please note that this entire project seems quite abandoned, I found this fork following this discussion that could be a replacement for a community-driven product

@alirezaalavi87
Copy link
Author

@giuseongit that project hasn't added/fixed anything new as far as I see. and the fact that the maintainer has only that repo and the account seems like a fake account makes me really not feel good about it.
As of now, I have been using my own fork that has fixed BiDi issues mostly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants