Page MenuHomePhabricator

VisualEditor: Link input widget is not oriented for RTL
Closed, ResolvedPublic

Description

A very simple mockup that I made in GIMP to show how it is supposed to look

VisualEditor: Link inspector is not oriented for RTL.

The arrow icon is supposed to point to the right, and be placed in the right corner, separated by a vertical line.

The title is supposed to be aligned to the right.

The chain icon is supposed to be next to the title on its left side.

The trash bin is supposed to be in the left corner.

See the attached file for a very simple mock-up of how it is supposed to look (it is off by a few pixels here and there - I'm not much of a graphic designer).

This bug is similar to bug 49613.


Version: unspecified
Severity: normal

attachment link-inspector-rtl.png ignored as obsolete

Details

Reference
bz50490

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 1:55 AM
bzimport set Reference to bz50490.

Created attachment 12713
A simple mockup that I made in GIMP to show how it is supposed to look

The V icon in the menu is supposed to be moved to the right side, too.

Updated screenshot.

Attached:

link-inspector-rtl.png (202×305 px, 11 KB)

This depends on the Frame rtl fix, which will add directionality inside frames, and allow for conditional flips (and/or also allow CSSJanus to eventually work inside the frames)

I took a look at the code, the flip itself is relatively easy, but I am missing the ve-rtl/ve-ltr classes that would be there after the frame fix.

Change 72056 had a related patch set uploaded by Mooeypoo:
Layout fix for Link Surface in RTL

https://gerrit.wikimedia.org/r/72056

Change 72056 abandoned by Mooeypoo:
Layout fix for Link Surface in RTL

Reason:
This was fixed in another (much more massive) gerrit fix that was merged. Works in master.

https://gerrit.wikimedia.org/r/72056

En layout is currently:

< (chain icon) 'Hyperlink' (trash icon)

He layout is currently:

(trash icon) '(Hyperlink in Hebrew)' (chain icon) >

This is not quite Amir's design attached:

(trash icon) (chain icon) '(Hyperlink in Hebrew)' >

The current layout is good. There's a mistake in my image. Marking as FIXED.