Patch to fix some RTL and Hebrew related problems

[Elad]

2010/10/21 Elad <el.il at doom.co.il>:
> 2010/10/20 shaiton <shaiton at fedoraproject.org>:
>> On Wed, Oct 20, 2010 at 5:52 PM, Elad <el.il at doom.co.il> wrote:
>>> 2010/10/20 Elad <el.il at doom.co.il>:
>>>> 2010/10/20 shaiton <shaiton at fedoraproject.org>:
>>>>> On Mon, Oct 18, 2010 at 12:17 PM, אלעד <el.il at doom.co.il> wrote:
>>>>>> I'm sorry for flooding your mailing list, but i've updated my second
>>>>>> patch again,
>>>>>> now it also moves the globe of the language list to the right side of
>>>>>> the list, fixes the positions of the features page tabs, and fixes a
>>>>>> little problem with the links in the left side of the main page.
>>>>>>
>>>>>> now there are only one problem, Hebrew specific, that i need to fix. :-)
>>>>>
>>>>> I've pushed an update.
>>>>> Please, let me know understand what your "Hebrew specific" fix is doing.
>>>>>
>>>>> +.lang-he #header>.container_16>.grid_3 {
>>>>> +width: 180px;
>>>>> +}
>>>>> Should not be applied for me (it is broking the four foundations columns.)
>>>>>
>>>>> There is two things that should be corrected.
>>>>> The slideshow has a padding-right who need to be removed (under
>>>>> Firefox4, you won't see it under Chrome).
>>>>> The slideshow numbers are still displayed on the left using Chrome
>>>>> (looks good with Firefox4).
>>>>>
>>>>>
>>>>> --
>>>>> Kévin Raymond
>>>>> GPG-Key: A5BCB3A2
>>>>>
>>>> the Hebrew specific fix is for a problem that doesn't appear in chrome
>>>> (but does appear in FF): the ok button (אישור) of the language list is
>>>> under, and not beside the list. it caused because of the word אישור is
>>>> longer than the word ok.
>>>>
>>> ignore that, this commit
>>>  http://git.fedorahosted.org/git?p=fedora-web.git;a=commit;h=2cf67805f1b1b7dc368a69db7cb64e27069494bf
>>> caused a strange behavior in RTL, so my fix for  isn't needed anymore
>>> (i need another fix though).
>>>> also, i think you should add !important to this property
>>>> .webpage-list, .checkmark-list {
>>>> padding-right:15px;
>>>> }
>>>>
>>>> 40px padding is too much for those lists.
>>>>
>>>> screenshots that shows the problems are attached.
>>>>
>>>> thank you for applying my patches.
>>>>
>>>>
>>>> --
>>>>>Elad
>>>>
>>>
>>>
>>>
>>> --
>>>>אלעד
>>>
>>
>> fixed
>>
>>
>> --
>> Kévin Raymond
>> GPG-Key: A5BCB3A2
>>
> Sorry about the top posting i made in my previous message.
>
> anyway, i've found another problematic rule in rtl_960.css:
> .container_16 .prefix_14 {
>    padding-right: 840px;
> }
> should be deleted or otherwise overridden, as it causes the "back to
> top" button to be in a wrong position.
>
> --
>>אלעד
>

another problem: while the tutorials and interviews text is
untranslatable, the text is still right aligned. in order to fix this,
you should surround the  content in another div, and define that div
as direction: ltr. don't do direction: ltr to the whole page/content,
since the titles and the short descriptions are translated.


-- 
>אלעד