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.

Sign up for GitHub

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

Jump to bottom

RTL text confuses MELS #4779

Closed
turt2live opened this issue Aug 11, 2017 · 6 comments
Closed

RTL text confuses MELS #4779

turt2live opened this issue Aug 11, 2017 · 6 comments
Labels
P1 S-Minor Impairs non-critical functionality or suitable workarounds exist S-Tolerable Low/no impact on users T-Defect

Comments

@turt2live
Copy link
Member

Description

image

Browser locale is en_US. Riot is set to en_US. Expanding the MELS is okay though:
image

Steps to reproduce

Uncertain. Page refresh did not resolve issue.

Log: not sent

Version information

  • Platform: web (in-browser)
  • Browser: Chrome 59
  • OS: Windows 10
  • URL: riot.im/develop
@t3chguy
Copy link
Member

t3chguy commented Aug 11, 2017

That's an RTL issue, has been reported more generically

Closing as superseded

@t3chguy t3chguy closed this as completed Aug 11, 2017
@t3chguy
Copy link
Member

t3chguy commented Aug 11, 2017

Also, appears to be fixed : #2922 (comment)

@turt2live
Copy link
Member Author

@t3chguy Not fixed on /develop (07ca70f-react-02217c8bd21e-js-92a75aaa0861). Please re-open this or direct me to the open ticket that this is a duplicate of.

@turt2live turt2live changed the title Wrong language on MELS RTL text confuses MELS Aug 12, 2017
@t3chguy
Copy link
Member

t3chguy commented Aug 12, 2017

@lukebarnard1 apparently not fixed

@t3chguy t3chguy reopened this Aug 12, 2017
@lampholder lampholder added T-Defect S-Tolerable Low/no impact on users S-Minor Impairs non-critical functionality or suitable workarounds exist P1 ui/ux labels Aug 14, 2017
@richvdh
Copy link
Member

richvdh commented Mar 12, 2018

this is something of a dup of #1712

@lukebarnard1
Copy link
Contributor

Yep. Closing in favour of #1712.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
P1 S-Minor Impairs non-critical functionality or suitable workarounds exist S-Tolerable Low/no impact on users T-Defect
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@turt2live @richvdh @lampholder @t3chguy @lukebarnard1