Page MenuHomePhabricator

RTL LTR mixed mode on 1.18pre
Closed, ResolvedPublic

Description

While testing on test2.wikipedia.org I noticed a few issues that need taking care of:


Version: 1.18.x
Severity: normal
URL: http://test2.wikipedia.org/wiki/Islam?uselang=he

Details

Reference
bz31031

Event Timeline

bzimport raised the priority of this task from to Unbreak Now!.Nov 21 2014, 11:54 PM
bzimport set Reference to bz31031.

Created attachment 9078
directionality of "hide/show" of toc.

Attached:

Screen_shot_2011-09-20_at_11.19.53.png (94×230 px, 13 KB)

Created attachment 9079
positioning of FlaggedRevs element is always top right

Attached:

Screen_shot_2011-09-20_at_11.19.43.png (133×208 px, 12 KB)

Created attachment 9080
Image positioned at the left but using margins for right aligned element

Attached:

Screen_shot_2011-09-20_at_11.19.59.png (371×300 px, 79 KB)

happened both as anon and logged in.

The TOC works for me on test2.

  • TOC works for me as well
  • The FlaggedRevs position is intended behavior: nothing in the page content area should depend on the user language direction. It is rather the infobox which uses CSS from MediaWiki:Common.css which is flipped (most wikis will need to add @noflip where necessary in their on-wiki CSS).
  • Same for the image, it should be at the right side. I'm figuring out why it isn't (I hope it's also due to the on-wiki CSS).

Ah no, the image is indeed at the left side (..|thumb|left|..), so the margin is changed due to local CSS as well.

the toc is due to the <h2> being rtl. that flips the bidi logic on OS X I think. If I make that text latin script, it seems to be working.

The TOC title issue is fixed with r97622. (Needs merging & software update.)

This bug could be closed, but we might leave it open because I assume more BiDi bugs will pop up :)

I don't quite understand why should this be customized on-wiki and not fixed in the source.

I added this to div.flaggedrevs_short at FlaggedRevs/presentation/modules/flaggedrevs.css :

position: absolute;
right: 0em;
top: -4em;

This fixed the problem nicely for LTR and RTL.

It's a workaround, it's a bit ugly, it may have to be improved a little more, but it's better than customizing Common.css.

There's no problem with the FlaggedRevs box, it aligns correctly according to the page content direction. The problem was with the infobox using local CSS float:right; which is flipped to float:left; causing the infobox to display at the left side, which is wrong. I added @noflip so the infobox CSS is not flipped. (Same for the image).

Awesome. You can make the changes already on MediaWiki:Common.css, it won't break anything.

Per IRC, r96760 will be merged (it was a follow-up to a rev that was already merged).

Created attachment 9081
Screenshot of skin=standard breakage in mixed ltr/rtl mode

Attached:

Screen_shot_2011-09-20_at_18.53.26.png (738×1 px, 342 KB)

Assigned to Robin. Please give it back if you think it shouldn't be :).

All issues reported here were fixed or followed up. It could be re-opened when new RTL/BiDi regressions appear.