86229 - Changing the dir with javascript does not work.

Status: RESOLVED FIXED

(Core :: Layout: Text and Fonts, defect)

Description

[José Jeria]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:0.9.1+)
Gecko/20010615
BuildID:    2001061520

onMouseOver="this.dir='rtl';" does not work as expected.

Reproducible: Always
Steps to Reproduce:
1. Run code
2. MouseOver the gray box
3. Nothing happends

Expected Results:  Expected the word "bug" to move to the right. And back when
the mouse leaves the box.

This works fine in Netscape 6.01

<TABLE BGCOLOR="#dcdccc" WIDTH="300" onMouseOver="this.dir='rtl';"
onMouseOut="this.dir='ltr';">
<TR>
	<TD>
	Bug
	</TD>
</TR>
</TABLE>

Comment 1

[Fabian Guisset]

You mean Netscape 6.1, right?

Comment 2

[José Jeria]

No, I mean Netscape 6.01. Works as expected in 6.01. Not working on the nightly 
builds.

Comment 3

[Johnny Stenback (:jst)]

The value does change in the DOM, which means that the correct information is
stored in the content model, but we're not correctly showing it, over to layout.

Comment 4

[basic]

Attachment: testcase

Comment 5

[basic]

this should go to BiDi

Comment 6

[José Jeria]

Not that  I understand this to 100%, but how can this bug be "BiDi Hebrew &
Arabic"??
Shouldn't this be display?

Comment 7

[Zach Lipton [:zach]]

Mass-move all BiDi Hebrew and Arabic qa to me, [e-mail]. 
Thank you Gilad for your service to this component, and best of luck to you 
in the future.

Sholom. 

Comment 8

[Mike Kaply [:mkaply]]

Simon, do you have any idea where the code in layout that changes 
directionality us?

I'm not convinced this is a Bidi problem, as dir=rtl should work regardless of 
language.

did we put in any new code to work with this?

Mike Kaply
IBM

Comment 9

[Simon Montagu :smontagu]

If the direction changes in the DOM but isn't reflected in rendering, this is
presumably an invalidation/reflow issue. 

Playing with the testcase, it seems that the |onMouseOver| works on the <td> but
not on the <table>, am I right?

cmanske, is this your beat?

Comment 10

[Charles Manske]

Nope, not in my court!

Comment 11

[José Jeria]

So... if not BiDi Hebrew & Arabic(!??), what then?

Comment 12

[Simon Montagu :smontagu]

This seems to have been fixed, maybe by bug 103266? Marking WORKSFORME after
testing today's build on W2K and Linux.

Comment 13

[José Jeria]

Attachment: Testcase showing that bug is not fixed?

See this new attachment. Nothing happends. Dont know if it is this bug that is
not fixed or if it is bug 103533 causing this.

Comment 14

[José Jeria]

Attachment: new working testcase

Same as testase 58348, but now it works. What i changed was the script. 
obj.dir = (obj.dir=="Ltr")? "rtl":"ltr";
Now i check for "Ltr" instead of "ltr". Why is it so? Why capital L? Bug?

Comment 15

[José Jeria]

I guess this one has to be reopened because it is not working as expected. See
comment 14. 

Comment 16

[Simon Montagu :smontagu]

According to dbaron in http://bugzilla.mozilla.org/show_bug.cgi?id=10173#c6,
this behaviour is perfectly correct according to the DOM standard.