|
3.46 KB,
patch
|
bz
:
review+
|
Details | Diff | Splinter Review |
Created attachment 595720 [details] [diff] [review] Patch v1 These attributes are not supported at all in IE and Opera, and only in quirks mode in Gecko. Chrome supports them in all modes. I think we can get away with removing support entirely.
Is it possible to get a more data driven answer than "we think we can get away with it"?
|
||
Comment 2•6 years ago
|
||
This seems to be a perfect question to direct to Blekko, who have a neat "grep the web" tool we can use. Essentially we would ask them "what percentage of <table> tags use hspace or vspace? Coincidentally there is a brownbag discussion being held today with them and I will follow up to see if this is possible.
|
||
Comment 3•6 years ago
|
||
Comment on attachment 595720 [details] [diff] [review] Patch v1 r=me on the code changes, but I'd love to see the data here. Does IE really not support this even in their compat quirks mode?
|
(Assignee) | |
Comment 4•6 years ago
|
||
My IE8 on WinXP doesn't seem to support it, with or without doctype.
|
|
(Assignee) | |
Comment 5•6 years ago
|
||
John, do you know when we could get the data?
|
|
||
Comment 6•6 years ago
|
||
I've followed up with them and will update this ASAP.
|
|
(Assignee) | |
Comment 7•6 years ago
|
||
Thanks!
|
|
(Assignee) | |
Comment 8•6 years ago
|
||
John, you would get to me ASAP over a month ago. Do you have an ETA? I'd rather not wait indefinitely to finish this bug.
|
|
||
Comment 9•6 years ago
|
||
Hi Gerv, Sorry, perhaps some miscommunication here. I saw you were cc'd on the emails from Blekko and thought that was sufficient. I'll dive back into that thread and get it sorted out.
|
|
||
Comment 10•6 years ago
|
||
Hi Ms2ger, Blekko poated the results of two crawls here: http://blekko.com/webgrep?page=view&id=2bf3a3f2189ce6142e124c5dc78a00b8 http://blekko.com/webgrep?page=view&id=736e28936d117ea69349c0e92f7b5994
|
|
(Assignee) | |
Comment 11•5 years ago
|
||
So, what's the conclusion here?
|
|
||
Comment 12•5 years ago
|
||
What does the data look like?
|
||
Comment 13•4 years ago
|
||
Someone needs to draw conclusions from comment 10 here. It's a big number, but not clear to me how big. And it wouldn't surprise me if most (or even nearly all) of the values were 0, which I think should be the default for both.
|
||
Comment 14•4 years ago
|
||
One of the crawls looked for non-zero values if I understand the grep correctly. So looking at just number of URLs: hspace 3.16M of which 2.21M (70%) have non-zero value vspace 3.29M of which 2.28M (69%) have non-zero value One thing I wonder is, in <table[^>]*hspace\s*=\s*['"]?[ ^0][^>]*> there's a space before the ^, is that intended to be there? Also, since the " or ' before it is optional in the regexp, it could instead match the [^0], so these results might be bogus. I did a different search, data set 18/06/2013 from http://webdevdata.org/ (53,000 pages): $ grep -riE "<table\s[^>]*(v|h)space\s*=\s*[\"']?[1-9]" . > ~/Desktop/vhspace.txt $ wc -l ~/Desktop/vhspace.txt 13 /Users/zcorpan/Desktop/vhspace.txt The matches were from 9 different sites. So it's 0.017% of the sites in this data set that use non-zero vspace/hspace.
|
|
||
Comment 15•4 years ago
|
||
Analysis of visual impact of dropping vspace/hspace on those 9 pages: stihi.ru - an image would get rid of 10px vertical spacing and two items in the footer would lose 3px spacing. Seems acceptable. seiya-saiga.com - no visible difference. www.clipmoon.com - the column widths differ by a few pixels. Seems acceptable. proza.ru - same or very similar site as stihi.ru gossiplankanews.com - no visible difference. gossiplanka.lk - same site as above. www.localharvest.org - header would lose 4px vertical spacing. Seems acceptable. akipress.org - no visible difference. paparasinewslanka.com - 404
|
|
||
Comment 16•4 years ago
|
||
I filed bugs on Blink and WebKit: https://code.google.com/p/chromium/issues/detail?id=277080 https://bugs.webkit.org/show_bug.cgi?id=120133
|
|
||
Comment 17•4 years ago
|
||
Spec bug: https://www.w3.org/Bugs/Public/show_bug.cgi?id=23039
|
||
Updated•4 years ago
|
||
|
|
(Assignee) | |
Comment 19•3 years ago
|
||
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/1ZjuiWEj-Ww
|
||
Comment 20•2 years ago
|
||
The Blink bug is fixed since February 2015 https://code.google.com/p/chromium/issues/detail?id=277080
|
|
(Assignee) | |
Comment 21•2 years ago
|
||
https://github.com/whatwg/html/pull/13
|
|
||
Comment 22•2 years ago
|
||
Posted the site compatibility doc: https://www.fxsitecompat.com/en-US/docs/2015/hspace-vspace-attributes-on-table-will-no-longer-be-supported/
Description
•