Page MenuHomePhabricator

BiDi: render Monobook links mainly / only according to LTR / RTL property of selected user interface; adjust /skins/monobook/user.gif properly
Closed, InvalidPublic

Description

Author: gangleri

Description:
Hallo!

The screen dump http://bugzilla.wikimedia.org/attachment.cgi?id=1109 ilustrates
the problem. If you want to reproduce the mentioned behaviour log in at
http://test.wikipedia.jp/ and select 'he' as user interface.

To my understanding rendering of the Monobook links for <user name>, My talk,
Preferences, My watchlist, My contributions, Log out is done at the moment
according to the LTR / RTL property of the *content language*.

However if a user interface is selected with the "oposite" LTR / RTL property
rendering might be a mess.

I suppose that the proper code should care about:
a) <user name> can contain LTR and / or RTL including characters where the
browsers automatic BiDi algorithm applies (as brackets, ".", "," etc.)
a continued) I propose to "embed" <user name> in the general (BiDi) punctuation
characters according to LTR /RTL property of the *content language*

b) The "translation" of [[MediaWiki:My talk]], [[MediaWiki:Preferences]],
[[MediaWiki:My watchlist]], [[MediaWiki:My contributions]], [[MediaWiki:Log
out]] is taken from the selected user interface. These strings should be
"embeded" in the general (BiDi) punctuation characters according to LTR /RTL
property of the *user content language*. This avoids problems with characters
where the browsers automatic BiDi algorithm applies.

c) The icon user.gif from
http://cvs.sourceforge.net/viewcvs.py/*checkout*/wikipedia/phase3/skins/monobook/user.gif
should be placed according to the LTR /RTL property of the *user interface language*
c1) at the left for LTR
c2) at the right fort RTL

d) The order of the links from b) should be according to the LTR /RTL property
of the *user interface language*. This will preserve the *known* "look and feel"
from a wiki in the content language as the user interface language.

Last but not least: Please use proper spacing between the links from d).

Thanks for fixing this.

best regards reinhardt [[user:gangleri]]


Version: unspecified
Severity: enhancement
URL: http://test.wikipedia.org/w/index.php?title=Search&uselang=he&useskin=monobook

Details

Reference
bz4105

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 8:58 PM
bzimport set Reference to bz4105.
bzimport added a subscriber: Unknown Object (MLST).

gangleri wrote:

*notes*

  • this bug may be related to particular browsers only, please add additional

dependencies from tracking bugs if necessary

  • I found the bug using: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US;

rv:1.7.5) Gecko/20051012 Netscape/8.0.4
Build number: 20051012

  • this bug may be solved by changing / fixing *default* "MediaWiki:Monobook.css"

configurations

Can't reproduce anymore/invalid if FF bug.