Match spec for col/colgroup.span, textarea.rows/cols

RESOLVED FIXED in Firefox 51

Status

()

P3
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: ayg, Assigned: ayg)

Tracking

unspecified
mozilla51
Points:
---

Firefox Tracking Flags

(firefox51 fixed)

Details

Attachments

(1 attachment)

https://github.com/whatwg/html/pull/1693

In brief, we and Edge throw for textarea.rows/cols, which is what the spec used to say.  The spec changed to match Blink and WebKit, who don't throw, because it's not nice to ask anyone to start throwing.  There were a bunch of other inconsistencies between UAs here, and I reviewed what the new spec says against implementations and it seems to be reasonable, so I suggest we change to match it.  (This will also affect col/colgroup.span, which we already didn't throw for.)

Updated

2 years ago
Priority: -- → P3
Created attachment 8783488 [details] [diff] [review]
0001-Bug-1296929-Match-spec-for-col-colgroup.span-textare.patch
Assignee: nobody → ayg
Status: NEW → ASSIGNED
Attachment #8783488 - Flags: review?(jst)

Updated

2 years ago
Attachment #8783488 - Flags: review?(jst) → review+

Comment 4

2 years ago
Pushed by ayg@aryeh.name:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1442a6ab4296
Match spec for col/colgroup.span, textarea.rows/cols; r=jst

Comment 5

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/1442a6ab4296
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox51: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in before you can comment on or make changes to this bug.