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

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

Attachment: with-heb-specific.png
Description: PNG image

Attachment: without-heb-specific.png
Description: PNG image

Attachment: too-much-padding.png
Description: PNG image

Attachment: good-padding.png
Description: PNG image

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