search

rodneyrehm / viewport-units-buggyfill

Making viewport units (vh|vw|vmin|vmax) work properly in Mobile Safari.
MIT License
1.59k stars 151 forks source link

What's up with viewport-units-buggyfill2 on npm? #94

Open ryanburnette opened 6 years ago

ryanburnette commented 6 years ago

@rodneyrehm Have you noticed viewport-units-buggyfill2 advertising v0.6.3 on npm? Not sure if it's something malicious or erroneous or what. If it contains something of value perhaps it should be incorporated into viewport-units-buggyfill.

rodneyrehm commented 6 years ago

Thanks for pointing that out! I was not aware of this fork and it being published on npm. I'm not sure why @springuper didn't send PRs for those changes, but instead felt it necessary to publish a clone…

ryanburnette commented 6 years ago

There is a lot of malicious stuff on NPM to watch out for, but it's probably just someone who doesn't understand etiquette.

springuper commented 6 years ago

It's my bad, sorry.

I published it to npm registry to be able to handle presto based opera quickly (commit), but i should publish it in our private registry.

Sorry for the inconvenience, i'll remove it from npm registry. i'd like to contribute too if it's useful for others.