2010/10/21 Elad <el.il@xxxxxxxxxx>: > 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. > > > -- >>×××× > Since i'm part of the team now, i will make all those fixes myself. thanks. -- -Elad. -- websites mailing list websites@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/websites