Thank you for making this great base theme!

I reviewed the theme rtl support and added all needed css:

1. Added /* LTR */ marking to original css files
2. Created or modified corresponding rtl css files
3. Added rtl images (attached below)

The following guidelines were considered:

Original css file RTL css file
Adding a css property that has "right" (or "left") as value
The same property should be added with value "left" (or "right")
Adding the css property "left" (or "right")
Property "left" (or "right") should be reverted to "auto", and "right" (or "left") set to the same value
Adding properties like padding, margin, background-position ...
  1. padding-left:20px;
  2. margin:10px 5px 20px 15px;
  3. background:url(images/decline.png) no-repeat right top;
    "left" and "right" values should be swapped;
    If only one is set the other should be reset to default value (0 for paddings and margins) if required
    1. padding-left:0;
    2. margin:10px 15px 20px 5px;
    3. background:url(images/decline.png) no-repeat left top;
    Adding a background image that has a relation with language direction (for example an arrow from left to right)
    A new rtl image should be created from the original with required changes, and the corresponding css adjusted to point to it

    Note: patch generated in 7.x-3.0 version


    omega_rtl_images.tar_.gz6.92 KBahwebd
    omega_rtl.patch43.3 KBahwebd
    alpha_rtl.patch140.92 KBahwebd


    ahwebd’s picture

    Version:7.x-3.0» 7.x-3.x-dev
    ahwebd’s picture

    no one?

    Cellar Door’s picture

    Assigned:Unassigned» himerus

    Thanks for the patch!

    himerus’s picture

    Status:Needs review» Needs work

    alpha_rtl.patch does not apply cleanly to latest 7.x-3.x dev.

    Checking patch alpha/css/alpha-alpha-rtl.css...
    Checking patch alpha/css/alpha-alpha.css...
    Checking patch alpha/css/alpha-debug-rtl.css...
    Checking patch alpha/css/alpha-debug.css...
    Checking patch alpha/css/alpha-mobile.css...
    error: while searching for:
      position: relative;
      margin-left: 10px;
      margin-right: 10px;
    error: patch failed: alpha/css/alpha-mobile.css:8
    error: alpha/css/alpha-mobile.css: patch does not applyalpha_rtl.patch does not apply cleanly to latest 7.x-3.x dev.

    I'd love to see this get in, but will be moving it the 3.2 release roadmap... and that being said, in 3.2 the CSS is going to wildly change, and the current css will still be present for legacy sites, but a new, cleaner version of CSS styling will be enabled in newer versions.

    @ahwebd will you be willing to A.) update this patch to apply cleanly to latest dev version, and B.) work with us on the updated CSS coming in 3.2 to ensure it is properly RTL compliant?!

    ahwebd’s picture

    Thanks for all your efforts himerus, I'll be really happy to work on this (a & b) when time permits.

    i2man’s picture

    Title:RTL support» RTL support css file

    Hi all, i am new to Druplal, i am working on site with two languages ( French (ltr) and Arabic (rtl), i am using Omega 3 for theming with subtheme, i implemented all the files provided with this theme, the problem is on Arabic (rtl) section, i want to note that languages are added perfectly, i can switch between theme with no problem Ltr and Rtl, the issue is when i want to theme RTL section with custom file css that do not exist in the package of Omega, how to add this file that will be styled separatly for the Arabic section with graphics and fonts, i know the file global.css is perfect but juste for French section, i made some searching and find LESS but the problem is already how to add this CSS file to my Subtheme CSS folder, i need your help please.
    Thanks all

    i2man’s picture

    Please can any one help ?

    i2man’s picture

    Issue summary:View changes