search

OverZealous / cdnizer

Node module for replacing local links with CDN links, includes fallbacks and customization
MIT License
52 stars 24 forks source link

Improve: doesn't work with srcset #22

Open bars3s opened 8 years ago

bars3s commented 8 years ago

Hi @OverZealous!

Found that cdnizer doesn't process srcset attributes. I think it will be very useful feature.

Thx!

OverZealous commented 8 years ago

Yeah, that definitely looks useful. That being said, that's a lot of work to integrate: the current design uses a RegExp-based replacement strategy that expects a (before)(url)(after) format, while srcsets are a comma-separated list of values, each of which has to be processed individually.

I'll see if I can find some time to add it in the near future.