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

RTL and bidirectional text #472

Closed
mikemorris opened this issue Jun 18, 2014 · 8 comments
Closed

RTL and bidirectional text #472

mikemorris opened this issue Jun 18, 2014 · 8 comments
Assignees

Comments

@mikemorris
Copy link
Contributor

Need to implement Unicode bidi algorithm for RTL and bidirectional text to reverse the display order of glyphs.

Should look like this:
casa

Currently looks like:
screen shot 2014-05-29 at 1 12 33 pm

@mikemorris mikemorris self-assigned this Jun 18, 2014
@mikemorris
Copy link
Contributor Author

Might be useful for test fixtures http://www.unicode.org/Public/7.0.0/ucd/

@mourner mourner added this to the future milestone Jun 24, 2014
@mikemorris
Copy link
Contributor Author

Possibly useful JavaScript implementation of ICU Bidi by Twitter https://github.com/twitter/twitter-cldr-js#handling-bidirectional-text

@mourner
Copy link
Member

mourner commented Feb 10, 2015

Is this duplicate of #471?

@mikemorris
Copy link
Contributor Author

@mourner Nope, RTL is pretty straightforward for non-complex scripts like Hebrew.

@jfirebaugh jfirebaugh removed this from the future milestone Jun 15, 2015
@fnicollet
Copy link
Contributor

Also interested in this issue but I have no idea how to patch it

@mushon
Copy link

mushon commented Oct 20, 2015

Just mentioning I ran into this problem as well in the Mapbox Studio beta and was forwarded here after reporting it. I can't help fix it but I will be happy to test a fix once one comes up. I also want to stress the importance of this issue as without this fix Mapbox-GL cannot be used by approximately half a billion people. Hope you make sure to prioritize this fix and definitely not launch without it.
Keep up the good work and let me know if and how I can help.

@mushon
Copy link

mushon commented Oct 20, 2015

Here's the bug documentation I sent for your reference:
https://www.dropbox.com/s/ggcjnb3d3s5nhea/2015-10-20%2023.21.48.gif?dl=0

@mikemorris
Copy link
Contributor Author

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

5 participants