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 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
>



-- 
>××××
-- 
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