Replace inline style by hidden attribute...

[robsonsobral]

...in a retro compatible way

closes #48

[robsonsobral]

Rethinking... My code is retro-compatible only in case the style is set inline. I guess I should change it.

[robsonsobral]

FIXED.

[Alex-D]

I think I can drop IE8 support now and use classList

Please remove yarn.lock since I now prefer package-lock.json :)

[robsonsobral]

I think I can drop IE8 support now and use classList

Are you sure? Move to v3 just because of that? No adding tests? I guess we could, at least, add code linting. Of course, keeping the simplicity and the small size.

---

dataset is also IE10+

[Alex-D]

I will take some time to clean that with my eslint ruleset

IE8 is not supported in v2 since I already use classList and dataset, so what is wrong to keep them? I just need to update the README to remove IE8 from the list and go with IE10+ or did I miss something?

[robsonsobral]

I will take some time to clean that with my eslint ruleset

Let's use Airbnb! It's a standard.

IE8 is not supported in v2 since I already use classList and dataset, so what is wrong to keep them? I just need to update the README to remove IE8 from the list and go with IE10+ or did I miss something?

Changes of support are MAJOR, but as that's already done on v2, I guess we can just update the README.

I wouldn't reduce support for things so simple to resolve, but you're the boss!

[robsonsobral]

We could move to ESBuild as well.

[robsonsobral]

The constant of the world is to change, @Alex-D .

https://github.com/WICG/display-locking/blob/main/explainers/hidden-content-explainer.md