36515 - XUL Textfield does not display value attribute after 'setAttribute("value","fuba")'

Status: VERIFIED FIXED

(Core :: XUL, defect, P3)

Description

[gb]

If u change the value of a XUL textfield with
widget.setAttribute("value","fuba")
the new value is only displayed after left-mouse-click on the textfield.
If u use 
widget.value="fuba"
everything is ok.

tested with version: 2000041811 and 2000041915 (Linux)
	             2000041805 (Windows)

Example :
<?xml version="1.0"?> 
<?xml-stylesheet href="chrome://global/skin/" type="text/css"?>
<window
	xmlns:html="http://www.w3.org/TR/REC-html40"
	xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">

	<titledbox id="bugOverlay" flex="1">
		<button value="help" oncommand="document.getElementById('field').value='button
1'"/>
		<textfield id="field" value="this is not a bug" flex="1"/>
		<button value="help" oncommand="document.getElementById('field').value='button
2'"/>
		<box>
			<text value="click button to change text"/>
		</box>
	</titledbox>

	<titledbox id="bugOverlay" flex="1">
		<button value="help"
oncommand="document.getElementById('field2').setAttribute('value','button 1')"/>
		<textfield id="field2" value="this is a bug" flex="1"/>
		<button value="help"
oncommand="document.getElementById('field2').setAttribute('value','button 2')"/>
		<box orient="vertical">
		<text value="click button to change text"/>
			<text value="after this click on textfield to display change  (BUG!?!?!)"
style="color:red"/>
		<spring/>
		</box>
	</titledbox>

</window>

Comment 1

[Kathleen :Brade]

I think that this is a known issue; hyatt knows for sure.
I think you can't use setAttribute and instead are supposed to set the value 
directly.

Hyatt--did I get this right?

Comment 2

[Peter Trudelle]

can't find an existing bug on this, iving to hyatt

Comment 3

[Asa Dotzler [:asa]]

setting status to new

Comment 4

[John Morrison]

In 20000608nn mac/linux/win32 M17 trunk bits ... this is fixed. It works for 
either the .value property, or by setAttribute('value', ...)   --> FIXED.

[There is a box layout bug in the testcase, but that's another bug]

Comment 5

[John Morrison]

verified fixed.