Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 686913
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Kyle, bug 686913 is about to be fixed and will take into account progress elements only so this is the meter part of it.
Status: REOPENED → NEW
Assignee: nobody → mounir
Status: NEW → ASSIGNED
Comment on attachment 630559 [details] [diff] [review] Patch Doesn't this make the meter non-labelable? Is that desired?
Oh... The specs tricked me. Before (and that's how our code behave) only form controls were able to be labeled but since <progress> and <meter> are not form controls, Hixie changed the specs to allow any element to be labeled... I was assuming that given those elements were no longer form controls they were not labelable. Patches will come.
That should fix it, using bug 762434. BTW, I could have IsLabelable() defined in each element that is labelable with simply |return true;|. I just kept the nsIFormControl logic here so I did the same kind of thing for meter and progress. If you think having IsLabelable defined for each element would be better, I can do that.
Comment on attachment 630921 [details] [diff] [review] Patch v2 r=me
Attachment #630921 - Flags: review?(bzbarsky) → review+
Target Milestone: --- → mozilla16
Status: ASSIGNED → RESOLVED
Closed: 7 years ago → 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.