467815 - XForms range controls do not work with FireFox 3

Status: RESOLVED FIXED

(Core Graveyard :: XForms, defect)

Description

[Dan McCreary]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.0.4) Gecko/2008102920 Firefox/3.0.4
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.0.4) Gecko/2008102920 Firefox/3.0.4

XForms range controls (sliders) do not work in FireFox 3.  This has been verified by Aaron Reed.  To test just run the URL above.  You will note that in initial values of the range controls are being set correctly so it appears they are binding to the elements in the model.  But when you change the value by dragging the range knob the values do not change.

Thanks for you work on this.

Reproducible: Always

Steps to Reproduce:
1.Run the URL above in FF 3 with the XForms extension
2.Drag the knob
3.Note that the values do not change
Actual Results:  
The values do not change.

Expected Results:  
The values should be updated as you drag the knob.

Thanks for your work on this.  The XForms extension is a very valuable component for us.

Comment 1

[Dion Sole]

Attachment: Patch to fix XHTML range widget.

The issue here was that the control element for the XHTML range wasn't actually a proper DOM element, so addEventListener wouldn't work on it. Similarly, get/set value() referred to this.control.value instead of this.control.slider.value.

With the attached patch, the widget works correctly again.

Comment 2

[Dion Sole]

Just adding you guys since this wasn't filed under the XForms component.

Comment 3

[alexander :surkov (:asurkov)]

Ok, bug 417051 should fix this but I missed this actually.

Comment 4

[alexander :surkov (:asurkov)]

Comment on attachment 382439 [details] [diff] [review]
Patch to fix XHTML range widget.

r=me

please get one more review before checking in.

Comment 5

[alexander :surkov (:asurkov)]

checked in on trunk, http://hg.mozilla.org/xforms/rev/80fbf731d56e

Comment 6

[aaronr]

checked into ff3 (cvs trunk)