sindresorhus/srcset

Match the latest spec #1

zcorpan posted onGitHub

  • h descriptor is not allowed anymore
  • w and x on the same candidate is not allowed
  • Just splitting on commas does not parse correctly (e.g. URLs can contain commas).

http://www.whatwg.org/specs/web-apps/current-work/multipage/embedded-content.html#parse-a-srcset-attribute


@issuehunt has funded $60.00 to this issue.


posted by issuehunt-app[bot] over 5 years ago

can I work on this

posted by fisker over 5 years ago

@fisker Do you want to start working at that?

posted by evilebottnawi about 5 years ago

@evilebottnawi #3 already started working on this, but have not fix the changes requested for long time.

posted by fisker about 5 years ago

@fisker Go ahead :)

posted by sindresorhus about 5 years ago

After reading the spec, and I found that parse-srcset project do exactly the spec says. I don't think I can do better. So, I'm not going to work on this issue any more, unless we want the similar implementation. I feel I made a mistake replacing parse-srcset with srcset on prettier project.

posted by fisker about 5 years ago

@sindresorhus has rewarded $54.00 to @willianveiga. See it on IssueHunt

  • :moneybag: Total deposit: $60.00
  • :tada: Repository reward(0%): $0.00
  • :wrench: Service fee(10%): $6.00
posted by issuehunt-app[bot] over 4 years ago

Fund this Issue

$60.00
Rewarded

Rewarded pull request

Other pull requests

Recent activities

willianveiga was rewarded by sindresorhus for sindresorhus/srcset# 1
over 4 years ago
sindresorhus submitted an output to  sindresorhus/ srcset# 1
over 4 years ago