The directionality attribute on ul (which was ltr instead of rtl) caused aligning problems in IE7 and maybe in other versions as well.
Removing them shouldn't cause any change in normal use.
This patch removes them (ul and ol).

CommentFileSizeAuthor
#2 remove_ul_ol.patch577 bytestsi

Comments

z.stolar’s picture

z.stolar commented
Status: Needs review » Active

You forgot to attach a patch :)

tsi’s picture

tsi commented
StatusFileSize
new remove_ul_ol.patch577 bytes

Very strange...

z.stolar’s picture

z.stolar commented

Actually I think we should do the other way around - leave only the list elements, and take out the list items.
This will cause the bullets to be on the correct side (right on RTL and left on LTR).

tsi’s picture

tsi commented

The issue was that the list (ul) got the wrong directionality (ltr instead of rtl) and only the list items got the right rtl direction.
So removing li will cause the whole list to be aligned to the left.
furthermore, I think giving the directionality to li only should be enough to position the bullets on the correct side, but I haven't tested this in every possible browser.

z.stolar’s picture

z.stolar commented
Status: Active » Postponed (maintainer needs more info)

@tsi, please let me know if this is still relevant, and I'll apply your patch.