Bug 18196

Summary: IE-only supports <comment> tag
Product: WebKit Reporter: jasneet <jasneet>
Component: DOMAssignee: Nobody <webkit-unassigned>
Status: RESOLVED INVALID    
Severity: Normal CC: ahmad.saleem792, ap, bfulgham, jasneet, rniwa, webkit
Priority: P2 Keywords: HasReduction
Version: 525.x (Safari 3.1)   
Hardware: PC   
OS: Windows XP   
URL: http://classadnew.sina.com.cn/user/info_fix.php?f_city=556&f_id=1624659
Attachments:
Description Flags
screenshot
none
reduction none

Description jasneet 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
Comment 1 jasneet 2008-03-28 16:34:23 PDT
Created attachment 20170 [details]
screenshot
Comment 2 jasneet 2008-03-28 16:35:09 PDT
Created attachment 20171 [details]
reduction
Comment 3 Robert Blaut 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.
Comment 4 Peter Kasting 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.
Comment 5 Robert Blaut 2008-03-31 13:32:33 PDT
Confirmed as an evangelism bug.
Comment 6 Dave Hyatt 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.
Comment 7 Dave Hyatt 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.

Comment 8 Darin Fisher (:fishd, Google) 2008-04-02 23:17:06 PDT
hyatt:  note the domain of the problem site.  no browser except IE has marketshare in china.
Comment 9 Ahmad Saleem 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!