Main nav broken in RTL mode in IE7 on BP-Default

[DJPaul]

The main navigation menu is not displaying correctly in RTL mode in IE7 on BP-Default. This problem has existed since BP 1.5. LTR on IE7 works.

See #3595 for a lot of discussion and screenshots of the problem.

[r-a-y]

Attached patch addresses various issues with the header in IE7.

The patch fixes:

• Navigation hover menu so child list items properly show up underneath the main list item on hover.
• h1 and search menu so it displays
• Horizontal scrollbar so it doesn't show up - see line 351 of the patch. Opera users still see the horizontal scrollbar regardless though. For some alternative accessibility techniques, try the ones [listed here ​http://adaptivethemes.com/using-css-clip-as-an-accessible-method-of-hiding-content].

The patch does not fix:

• the right offset of the page. (This bug has existed since BP 1.2.) The problem has something to do with the max-width and width declarations on the body element.
• aligning the #nav to the left.

I've tested the patch on IE8, Firefox 12 and Chrome and it's non-destructive. As mentioned above, Opera users will still see the horizontal scrollbar with or without the patch.

With the patch applied, it will bring us pretty much on par with BP 1.2's RTL support.

[r-a-y]

02.patch fixes issues with the .accessibly-hidden class showing up when using LTR.

The left declaration is needed for all good browsers. text-indent is needed for IE7 when using RTL because of the horizontal scrollbar issue stated above.

The text-indent declaration could be moved to the default-rtl.css file instead.

[DJPaul]

Closing as wontfix as we shouldn't waste time on IE7 support nowadays, plus weary of breaking BP-Default and causing other issues at this point in its life.