Ensure <radiogroup> and <radio> comply with spec.

RESOLVED FIXED in mozilla0.9.6

Status

()

defect
RESOLVED FIXED
18 years ago
11 years ago

People

(Reporter: hyatt, Assigned: bugzilla)

Tracking

Trunk
mozilla0.9.6
x86
Windows 2000
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [xul1.0-widgets-radio])

Attachments

(5 attachments)

Reporter

Description

18 years ago
Radio group and radio are fully specified in section 3.8 of the XUL spec now.  
This bug is for you to go through and ensure all the properties mentioned are 
defined and working.  

Note the deprecation of the "checked" property in favor of "selected", the need 
for <radiogroup> to respond to onselect, the default vertical orientation, and 
the ability for radio groups to work without an id (covered in a separate bug).

While doing this, it will probably be useful to construct the radio groups test 
page.  I can get it checked into gila with the other tests.
Assignee

Comment 1

18 years ago
--> me
Assignee: hewitt → blakeross
Reporter

Updated

18 years ago
Blocks: 70753
Whiteboard: [xul1.0-widgets-radio]
Assignee

Comment 2

18 years ago
Posted patch patchSplinter Review
Assignee

Comment 3

18 years ago
Reporter

Comment 4

18 years ago
seems like if (i in children && ...) can just be if (...).

what is the bug with onselect again?  you only fire it in the false case.  is
that because it isn't working in the true case?
Assignee

Comment 5

18 years ago
Posted patch wahSplinter Review

Comment 6

18 years ago
r=hewitt
I apply this patch ( the wah patch ) and with modern I get no radio buttons and
in classic I get radio buttons with no state ( no visible dot in them ). This is
with a tree from today (9/6) with no other changes. 

Is there other files needed to go with this patch? ( like .idl files? )
Assignee

Comment 9

18 years ago
I forgot to attach the skin css files.

Updated

18 years ago
Blocks: 98598
so can/will you post them? I need this patch for XUL radio accessibility work.
thnx
Hello? Blake, you said you were going to post those files....
Assignee

Comment 12

18 years ago
Posted patch skins patchesSplinter Review
Blake: what's up with this one? Is this complete? I need this stuff in the tree
to land my patches. The patch has been in the bug for over a month now with no
more progress.
Assignee

Comment 14

18 years ago
There has been plenty of progress on my end.  Many other changes are landing
with this one.  I'm finishing them up.  Landing very very soon.

Updated

18 years ago
Blocks: 104661
Assignee

Updated

18 years ago
Target Milestone: --- → mozilla0.9.6
Assignee

Comment 15

18 years ago
Fixed.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED

Updated

11 years ago
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.