WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED INVALID
18196
IE-only supports <comment> tag
https://bugs.webkit.org/show_bug.cgi?id=18196
Summary
IE-only supports <comment> tag
jasneet
Reported
2008-03-28 16:34:06 PDT
I Steps: Go to
http://classadnew.sina.com.cn/user/info_fix.php?f_city=556&f_id=1624659
Scroll to the end of the page. II Issue: Extra content is showing up in center column before footer. III Conclusion: Only IE supports (non-standard) <comment> tag. IV Other browsers: IE7: ok FF3: not ok Opera9.24: not ok V Nightly tested: 31386
Attachments
screenshot
(159.96 KB, image/jpeg)
2008-03-28 16:34 PDT
,
jasneet
no flags
Details
reduction
(107 bytes, text/html)
2008-03-28 16:35 PDT
,
jasneet
no flags
Details
View All
Add attachment
proposed patch, testcase, etc.
jasneet
Comment 1
2008-03-28 16:34:23 PDT
Created
attachment 20170
[details]
screenshot
jasneet
Comment 2
2008-03-28 16:35:09 PDT
Created
attachment 20171
[details]
reduction
Robert Blaut
Comment 3
2008-03-29 00:48:20 PDT
Since it's non-standard element and also not supported by other browsers (except IE) there is no reason to implement it.
Peter Kasting
Comment 4
2008-03-31 12:49:54 PDT
Sure there is: site compat. A good reason _not_ to implement would be "doing this will break x, y, and z". Obviously there's an opportunity to evangelize the site to only use standards-compliant tags, but that doesn't also preclude consideration of support for this IE-ism if it increases real-world compat and doesn't break things.
Robert Blaut
Comment 5
2008-03-31 13:32:33 PDT
Confirmed as an evangelism bug.
Dave Hyatt
Comment 6
2008-03-31 13:34:29 PDT
This is trivial to support. The danger in supporting it would be with tag misnesting, since an error in closing the tag could result in a pretty catastrophic rendering failure. We'd have to make sure we understood IE's error resolution for the tag before turning it on. comment { display: none } in the CSS file is all that is required to implement this.
Dave Hyatt
Comment 7
2008-03-31 13:35:10 PDT
Given that Firefox and Opera don't support it, it's hard to imagine that this is that important though.
Darin Fisher (:fishd, Google)
Comment 8
2008-04-02 23:17:06 PDT
hyatt: note the domain of the problem site. no browser except IE has marketshare in china.
Ahmad Saleem
Comment 9
2022-07-25 17:00:58 PDT
No the browser support it event today (Safari 15.6, Chrome Canary 106 and Firefox Nightly 104). I think this can be marked as "RESOLVED WONTFIX"? Thanks!
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug