majn / telegram-purple

Adds support for Telegram to Pidgin, Adium, Finch and other Libpurple based messengers.
GNU General Public License v2.0
735 stars 81 forks source link

Consider RTL-/LTR-overrides #141

Open BenWiederhake opened 8 years ago

BenWiederhake commented 8 years ago

Here's how it looks when anything like that happens

Here's a list of important characters (and a funny comic to lighten your mood)

Here's what we can to:

Note that this has to be applied on (1) user names, (2) chat messages, (3) chat titles, and maybe more.

I'm strongly in favor of the second option (replace by U+FEFF), as I expect that the third option would require reading and understanding a lot documentation about how Unicode, Pidgin's unicode handling, text rendering, and Telegram's Unicode support works. And then a lot of work to investigate in how far all these components actually implement their specification. And then a lot of support, because complex things like that WILL break.

BenWiederhake commented 8 years ago

Addendum: If we do anything except "ignore", then keep in mind that there's also representations of U+202A using more than three bytes.

majn commented 8 years ago

It will also cause issues in Telegram for Android. screenshot_2015-10-14-09-54-11

BenWiederhake commented 8 years ago

Forgot the RLM and LRM symbols