6511 - label on <checkbox/>

Status: VERIFIED FIXED

(Core :: XUL, defect, P3)

Description

[Alistair Vining]

It seems that in a .xul file:

<html:label><html:input type="checkbox" name="check1"
checked="checked"/>checkbox 1</html:label>

works, but

<html:label><checkbox/>Dual state</html:label>

doesn't.  Is this a deliberate decision?  The advantage of the first syntax was
that clicking on the label selcted the checkbox.

Comment 1

[Peter Trudelle]

reassigning to pinkerton

Comment 2

[Peter Trudelle]

reassigning to pinkerton

Comment 3

[Mike Pinkerton (not reading bugmail)]

i'll take a look. not sure what could be different. marking assigned.

Comment 4

[Mike Pinkerton (not reading bugmail)]

labels only know about things inside them that are nsIFormControlFrames. the xul
checkbox is currently not one of these. Not sure how to get around this since I
don't want to make it an nsIFormControlFrame.

Comment 5

[Mike Pinkerton (not reading bugmail)]

ccing hyatt, as promised.

the more we talk about this, the uglier it gets.

Comment 6

[Mike Pinkerton (not reading bugmail)]

all platforms.

Comment 7

[Mike Pinkerton (not reading bugmail)]

this should go away when we merge with the html checkbox

Comment 8

[Mike Pinkerton (not reading bugmail)]

7546 is targeted for m8, so targeting this for m9.

Comment 9

[Mike Pinkerton (not reading bugmail)]

moving to M10 because i have no idea when the dependant bug will get fixed.

Comment 10

[Mike Pinkerton (not reading bugmail)]

<checkbox> no longer exists. use <html:input> for tri-state checkboxes, where it
works inside of a label.

Comment 11

[phillip]

code level fix, marking verified. reopen if i'm wrong here...