I am not actually sure there is a place where this currently makes any 
difference visually.

Most text areas are used in automove mode and automatically change their 
position and size based on the text, so text alignment doesn't matter unless 
they are resized from the outside. In the cases where such resizing is done 
(e.g. window titles) the alignment is kCenter. So visially nothing has changed 
yet.

I made the fix mainly so that RTL would be displayed correctly when we use 
single-line textareas in non-center mode and with properly set layout. And we 
already do the same adjustment for multi-line textareas.

Where it theoretically could make a difference is the campaign select menu. 
That one uses some single-line textareas, but we don't have a layout defined 
there that would resize the text areas, so they stay collapsed to the text 
width and don't react to my fix yet. (And we might even put everything there in 
a multi-line textarea when we do a proper box layout.)

If you want to just quickly test it, set one of the menu title alignments to 
Left or Right. Without the fix, language doesn't make a diffence, with the fix 
alignment is swapped properly for Arabic.


Side note: Even the menus where we have defined a proper layout with multi-line 
textareas, some of them are not resized properly (like game/map name in the 
description area), so RTL is ignored.

I guess all this RTL stuff has very low priority anyway. Aside from a small 
amount of Arabic, we don't even seem to have translations.
-- 
https://code.launchpad.net/~widelands-dev/widelands/RTL-alignment-fix/+merge/358596
Your team Widelands Developers is requested to review the proposed merge of 
lp:~widelands-dev/widelands/RTL-alignment-fix into lp:widelands.

_______________________________________________
Mailing list: https://launchpad.net/~widelands-dev
Post to     : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to