Contextual lack rtl styling

I had a crack at this and took a slightly different approach - moving where the borders/border radius are applied and padding etc.

Fixed the border radius stuff, chrome and opera were showing me the errors as they both support border-radius, Safari needed love also.

I think this looks better in IE7 and in general in other browsers also.

Its not perfect... you will see, maybe you can ice the cake :)

The patch in #5 doesnt look good in IE7, its jumping around on hover and still has the border issue.

That method is basically never going to work, the border will never trigger on the active class in IE7, so on the initial hover there is no border on div.contextual-links-active a.contextual-links-trigger, the border has to be on the anchor all the time, i.e. on a.contextual-links-trigger.

The problem I missed on my first attempt was the top: 19px; on div.contextual-links-wrapper ul.contextual-links - I went back to my original patch, removed the border-bottom on the trigger and changed the top value on ul.contextual-links to 18px and now, finally, its perfect.

I tested this in FF3, Opera10, Safari, Chrome, IE7/8 (all on Win). Needs testing in Mac browsers, should be good but you never know.

Thats not the only problem in IE7, see the screenshots, its very bad in IE7 - the icon jumps around on hover, and border top on the UL shows over the trigger, the border on the trigger anchor doesnt show until you hover the UL links - 3 issues - do you have IE7 to test in?

When you see this hovering over something like the submit button for Search its actually preferable to have a background as you can barely see the icon, and what if the background is the exact same color as the icon - then its simply invisible. I can't see that having a background is "bad", in many cases it could be very good, by increasing the visibility of the trigger in low contrast conditions.

Ops, forgot the screenshot (IE7), see how the borders are working - its hard for me to show the trigger icon moves as you hover back and fourth, its jumps and then locks in position.

My personal preference, is actually to have no background, and if we can make it work it would be good, but I dont think its bad to have a background, especially if it works and actually improves accessibility.

OK, so heres the updated patch -this one has the transparent background on the icon, lets not creep the scope of the patch by introducing the background on the icon, thats another issue.

I got around the no-triggered-border thing in IE7 by declaring a transparent border on the anchor (sneaky eh?) which has tricked IE7 into working correctly, and I fixed the moving icon issue.

Not sure why that last patch failed, didnt do anything different than usual when rolling. See if the bot likes this one:

Probably a Garland issue in RTL, I tested with Stark and all is good, we'll need to try and nail down under what conditions this happens, I notice in Corolla its not even clickable in IE7 RTL.

Be good if we could get this committed sooner rather than later and not creep the scope and try a fix every issue in every core theme, otherwise these RTL improvements are going to drag on forever.

Committed to CVS HEAD. Thanks!