https://bugzilla.wikimedia.org/show_bug.cgi?id=4030

--- Comment #14 from Amir E. Aharoni <[email protected]> ---
(In reply to comment #13)
> regarding patch: is there any test-suit this thing has? i would not want to
> fiddle with the core of this thing without some unit-test available. my
> familiarity with timeline and plotikus (or is it ploticus?) is not such that
> i
> would feel comfi with manual testing i devise myself.

No, very sadly there's no test suite. I asked about this, too, when I started
poking it.

I'll try to fix it again when I'm back from my travels.

> as to bidi library: i'm afraid it's too late for that now. doing something
> intelligent will break numerous plots that people already prepared on hewiki,
> arwiki, and fawiki where they painfully reversed all the legends manually.

That's why I made proper bidi handling optional.

> i believe my "solution" of adding new attributes for "rtl text/legend" will
> not
> disturb any of the existing plots, which use the "old" attribute with
> reversed
> content.

If you have time, I suggest trying to hack on the current patch:
https://gerrit.wikimedia.org/r/#/c/47336/

When it works, that should solve everything for all languages without making
special changes for rtl. Special changes for rtl won't help Arabic anyway,
because it won't connect the letters.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to