HTML5 expects us to support left, right, justify, center and middle for align on td, th, tr, col, thead, tbody and tfoot, both in no quirks and quirks mode. Patch upcoming.
Created attachment 452534 [details] [diff] [review] Patch v1
So no more absmiddle? Is that not supported by other UAs?
Created attachment 452578 [details] [diff] [review] Patch v2 Actually, absmiddle is hidden in the CSS block above the prose, where all the other values are defined.
I assume comment 3 is talking about the spec, not our code? Spec link, please?
Yeah. <http://www.whatwg.org/html/#alignment>, absmiddle is in the second CSS block, the other values are in the prose below.
OK. So why did the code use to have "center" twice in there, and why is it ok to no longer do that?
As can be seen at <http://software.hixie.ch/utilities/js/live-dom-viewer/saved/533>, getAttribute("align") currently returns "center" instead of "absmiddle", even though those values have different effects. I don't see the point in having the reverse mapping from text-align:center (rather than text-align:-moz-center) to "center".
Comment on attachment 452578 [details] [diff] [review] Patch v2 r=bzbarsky. Sorry for the lag here.... Ask for approval?
Attachment #452578 - Flags: review?(bzbarsky) → review+
Created attachment 461297 [details] [diff] [review] Patch for checkin
Attachment #452578 - Attachment is obsolete: true
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b3
You need to log in before you can comment on or make changes to this bug.