Re: Patch to fix some RTL and Hebrew related problems

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2010/10/20 shaiton <shaiton@xxxxxxxxxxxxxxxxx>:
> On Wed, Oct 20, 2010 at 5:52 PM, Elad <el.il@xxxxxxxxxx> wrote:
>> 2010/10/20 Elad <el.il@xxxxxxxxxx>:
>>> 2010/10/20 shaiton <shaiton@xxxxxxxxxxxxxxxxx>:
>>>> On Mon, Oct 18, 2010 at 12:17 PM, ×××× <el.il@xxxxxxxxxx> 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.

-- 
>××××
-- 
websites mailing list
websites@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/websites



[Index of Archives]     [Fedora Users]     [Linux ARM]     [ARM Kernel]     [Older Fedora Users]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Announce]     [Fedora Package Review]     [Fedora Music]     [Fedora Packaging]     [Centos]     [Fedora SELinux]     [Coolkey]     [Yum Users]     [Yosemite News]     [KDE Users]     [Fedora Art]     [Fedora Docs]     [Asterisk PBX]

  Powered by Linux