Closed Bug 835169 Opened 7 years ago Closed 7 years ago
Do we need support for the table[cols] attribute?
We have some code to support this attribute (e.g. <http://mxr.mozilla.org/mozilla-central/source/layout/style/nsRuleNode.cpp?rev=037363fa0258#6615>), but it's not clear to me it's good for anything.
It seems the result of the style mapping is unused, though I'm currently compiling the full tree to check. (This code might be mapping from either table or td, though. See the comments in HTMLTableElement.cpp about how its attribute mapping is weird.) (BTW, do any other browsers support the layout attribute?)
Apparently Bernd thought so too.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 587458
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Comment on attachment 707232 [details] [diff] [review] Remove unused mapping into style of table cols attribute. r=me
Attachment #707232 - Flags: review?(bzbarsky) → review+
Attachment #707232 - Attachment is obsolete: true
The revised patch adds the changes to nsCSSPropList.h and makes no other changes.
Comment on attachment 707426 [details] [diff] [review] Remove unused mapping into style of table cols attribute. r=me
Attachment #707426 - Flags: review?(bzbarsky) → review+
Status: REOPENED → RESOLVED
Closed: 7 years ago → 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
Yoshino-san updated: https://developer.mozilla.org/en-US/docs/Site_Compatibility_for_Firefox_21 As it wasn't documented elsewhere, and it was non-standard and quite cryptic, I don't think we need more doc. Switch back to dev-doc-needed if you disagree.
You need to log in before you can comment on or make changes to this bug.