Closed Bug 835169 Opened 7 years ago Closed 7 years ago

Do we need support for the table[cols] attribute?

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla21

People

(Reporter: Ms2ger, Assigned: dbaron)

References

Details

(Keywords: dev-doc-complete)

Attachments

(1 file, 1 obsolete file)

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
Assignee: nobody → dbaron
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Duplicate of this bug: 587458
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+
https://hg.mozilla.org/mozilla-central/rev/0ebed8dbb21c
Status: REOPENED → RESOLVED
Closed: 7 years ago7 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.