On 11/7/13, 4:27 PM, Tab Atkins Jr. wrote:
On Thu, Nov 7, 2013 at 3:39 PM, Timothy Hatcher <timo...@apple.com> wrote:
On Nov 7, 2013, at 2:22 PM, Tab Atkins Jr. <jackalm...@gmail.com> wrote:
srcset's parsing algorithm *cannot* be extended in the future.  I gave
an example of how it would fail over on blink-dev; I can reproduce it
here if necessary.

I don't subscribe to blink-dev. The WebKit community are the ones you need to 
convince.

A simple "Yes" would have sufficed.  I was just asking if I needed to
demonstrate it.  Take this markup:

<img srcset="foo 1x, bar 2x || baz 1x, qux 2x">

You expect this to break into two lists, "foo 1x, bar 2x" and "baz 1x,
qux 2x", which are then each split as currently proposed.

However, by the current parsing algorithm, || is read as an
unsupported descriptor, so it just breaks them into "foo 1x", "bar 2x
|| baz 1x", and "qux 2x".  The middle one is thrown away, because of
the duplicate x descriptor, and so the whole thing is just parsed to
the same value as "foo 1x, qux 2x", which is completely wrong.

This just sounds like a bad excuse. As far as I know, no browser has shipped srcset.

Benjamin
_______________________________________________
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev

Reply via email to