RTL CSS files

Lets patch`ify this zip.

I also attach the reversed arrow as a separate png

Looks and works very good. Thanks!

Tested and found no problems.

Theme information should probably be LTR, but views doesn't supply any classes or IDs for styling it.

z.stolar,
I added your request as #396994: Theming information must be LTR even on RTL displays, but it will need a little more love as my patch breaks the theme information a little bit. Lets keep problems separated.

Dear views maintainer,
we do need you to apply this patch ASAP so we can address #396994: Theming information must be LTR even on RTL displays and incrementally fix the display bug I described there (but it should only be fixed after this patch is applied and released).

elcuco: Please be patient. DrupalCon was at the beginning of March, then I went on vacation, and I've been drowning in this stuff. I apologize but I am not able to react to this stuff very quickly most of the time. This queue creates roughly 30 new issues every day. Please imagine what that means when you give me little hurry up speeches.

Also, the patch does not apply when I use 'patch -p0' because you did not create this patch using cvs diff -up from CVS. Please provide a proper patch.

Lets try it again:

This time I am attaching only the *-rtl.css files, and I remind you that the reversed arrow is here: http://drupal.org/files/issues/arrow-active-rtl.png

When this is in the CVS, I will provide a few patches which will in turn:
* add "padding-right:0" to the LTR css when they are overridden in the RTL css
* add /* LTR */ comments
* fix minor problems in the RTL css

I will do this in a few stages, as this is hard to find, but I hope that in every iteration the display will be usable, and will improve a little every time.

ping.... anything new with this issue?

ping once more...

You need to set it back to needs review. Nobody ever sees stuff in needs work.

Sigh. I thought there was a patch here. I will not try to figure out what's different between existing files and randomly posted files.

I've already asked for a proper patch. You went backwards. If you're not capable of providing proper patches, please don't bother. I don't have the time or energy to try and decipher this stuff.

These are new files, I am not touching existing files (yet), that will come in another patch.
I am trying to make this a multi-step process since it's a huge one, I went back since I don't think it's possible to do this in a one big step (lack of time from both of us, I have see the issue list of this project). See comment #7, in which my patch broke due to CVS changes.

My strategy is:
1) include this new files (affect only RTL users)
2) patch the LTR files
3) fix http://drupal.org/node/381994
4) continue adding patches to fix minor things that get broken in the way

@elcuco: you can include new files in a CVS patch. It's a bit nasty, but I believe it will make merlinofchaos' life much easier if it is just a patch to apply. See more details at http://drupal.org/patch/create (look for Changing CVS/Entries to add and remove files manually)

@elcuco (or whoever): If you want the throbber (ajax waiting icon thingie) to look ok in IE you need to make sure the containing tag hasLayout, my views-rtl.css looks like:

html.js a.views-throbbing,
html.js span.views-throbbing {
  background-position: left center; /* RTL */
  padding-left: 18px; /* RTL */
  padding-right: 0;  /* RTL */
  display: inline-block; /* hasLayout hack */
}

tested on IE6-8, FF3.5 and Chrome. I only tested it on sortable tables, so it might have adverse effects on other parts of the UI. zoom: 1; will probably do the trick if you don't care about valid css.

If someone wants to take this up and actually work on it, great. I'm leaving it open for that reason.

There's not much point into getting this into 6.x at this point, and 7.x has this.