[#MBS-1559] Right-to-left parentheses are broken - MusicBrainz JIRA

MusicBrainz Server

Right-to-left parentheses are broken

Created: 01/Mar/11 09:32 PM Updated: 18/Apr/12 05:17 AM

Component/s:

Data display

Affects Version/s:

NGS - Release Candidate 1

Fix Version/s:

None

Issue Links:

Relates

This issue has related issue:

~~MBS-4568~~ Add <bdi> tags to help with rendering of RTL text

Description

« Hide

As you can see in http://test.musicbrainz.org/release/d53e4a09-7123-475d-bfef-a1be77990f2c , right to left parentheses (Hebrew, Arabic, etc) don't appear correctly. I'm sure this is part of a bigger problem with MB not being too good with RTL (which probably should be fixed at some time in the future), but this at least is very visible and it would be quite important to display them correctly, IMO.

Sort Order:

[ Permalink | « Hide ]

nikki added a comment - 01/Mar/11 09:53 PM

It could be fixed by checking the strings we print for RTL characters and setting dir="rtl" (I have a user script which does that)

The problem is that we're displaying RTL text inside a LTR element, you get the opposite problem if you try and display LTR text inside a RTL element, it's not a MB-specific thing.

Original Trac ticket: http://bugs.musicbrainz.org/ticket/3338

[ Permalink | « Hide ]

monxton added a comment - 10/Apr/12 02:27 PM

See ARs for Pinchas Zuckerman see http://musicbrainz.org/recording/a5013bc1-0d72-45e9-ac59-5dd57a1abfae

[ Permalink | « Hide ]

nikki added a comment - 10/Apr/12 03:49 PM

HTML5 has a new element <bdi> which we could use around things like artist names coming from the db - it isolates text from the surrounding text, i.e. it should prevent the rtl-ness from leaking from the artist name into the surrounding text like in monxton's example.

There's limited browser support so far (reosarevok confirmed it works in Chrome, but it's not working in Safari or Firefox for me), but it's something we could probably consider because it wouldn't require any extra processing on our side, just extra markup in the templates in the appropriate places.

I don't know if it will help with the original example in this ticket though, where the brackets are part of the name in the database not part of the surrounding text.

[ Permalink | « Hide ]

nikki added a comment - 18/Apr/12 05:04 AM

I've created ~~MBS-4568~~ for monxton's example because they're not quite the same problem.