Patch to fix some RTL and Hebrew related problems

Elad el.il at doom.co.il
Thu Oct 21 07:01:46 UTC 2010


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.

-- 
>אלעד



More information about the websites mailing list