Re: Patch to fix some RTL and Hebrew related problems

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

 



2010/10/21 Elad <el.il@xxxxxxxxxx>:
> 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.
>
> --
>>××××
>

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.


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