Summary: | Drop support for <isindex> | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ian 'Hixie' Hickson <ian> | ||||
Component: | DOM | Assignee: | Chris Dumez <cdumez> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | ap, bugs-webkit, cdumez, commit-queue, darin, ddkilzer, d, eoconnor, gavin.sharp, hyatt, ian, kennyluck, koivisto, nickshanks, rniwa, sam, webkit, webkit, zcorpan | ||||
Priority: | P2 | ||||||
Version: | 420+ | ||||||
Hardware: | PC | ||||||
OS: | Linux | ||||||
Attachments: |
|
Description
Ian 'Hixie' Hickson
2006-02-07 22:20:00 PST
(m326369, o195531) HTML5 was changed by the way to support <isindex> in the parser. (In reply to comment #2) > HTML5 was changed by the way to support <isindex> in the parser. > So shouldn't the bug to be resolved as WONTFIX? (In reply to comment #3) > (In reply to comment #2) > > HTML5 was changed by the way to support <isindex> in the parser. > So shouldn't the bug to be resolved as WONTFIX? Yes. (In reply to comment #0) > <isindex> has been a pain (see e.g. bug 4574 or bug 4828), and Safari's support > for it differs from Mozilla's, IE's, and Opera's. And Bug 18415. Since we last looked at this, Chrome and IE have both said they will drop support for this tag. Blink thread: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/14q_I06gwg8 I can't find a citation for isindex being dropped in Project Spartan, but Travis from MSFT confirmed this in a conversation. Right now we're at 2 vs. 2 rendering engines with regard to isindex support. Is WebKit interested in following along and removing it? https://github.com/whatwg/html/issues/1088 It has been removed from the spec. Tests added to web-platform-tests in https://github.com/w3c/web-platform-tests/pull/3684 and html5lib-tests in https://github.com/html5lib/html5lib-tests/pull/74 Given that this was dropped from the HTML spec, Chrome and Edge, I think we should at least try to drop <isindex>. Created attachment 288667 [details]
Patch
Corresponding Mozilla bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1266495 Comment on attachment 288667 [details] Patch Clearing flags on attachment: 288667 Committed r205858: <http://trac.webkit.org/changeset/205858> All reviewed patches have been landed. Closing bug. |