Ok! Makes sense :)
The behavior is unchanged, the module already handled this way before. Apart from that, I completely agree with you - but it goes beyond the scope ...
Added link. Thank you!
Added "extra" wrappers.
Fixed in commit.
There are more countries and regions than the EU that have data protection laws, so I have kept this general. See also https://unctad.org/page/data...
This could be debatable. I am a big fan of having a select or checkbox in place to actively enable cdn, instead of it being a fallback. But since w...
Let's add a link on how to set asset-packagist up: https://www.drupal.org/docs/develop/using-composer/manage-dependencies#third-party-libraries
Let's wrap this in "extra" so the user doesn't think this needs to go in the bottom level.
"web" doesn't have to be on the same level as "themes" and "modules" and "libraries" can also have a different location.
"[...] and respect the General Data Protection Regulation." Would be better here I think....
Oh, I see it is used in hook.
How does the directory
property works? I didn't find in the documentation. And it should be /libraries/js-cookie
not cookies.
I think this sentence can be improved to clarify it's currently loading from CDN already?...
Perhaps better "from the local libraries
directory"?
/libraries/js-cookies vs. /libraries/js-cookie/dist/js.cookie.min.js
(below)
Great suggestion @RoSk0, I added the one-liner.
Why not add this instructions curl --create-dirs -o libraries/js-cookie/dist/js.cookie.min.js https://cdn.jsdelivr.net/npm/js-cookie@3.0.5/dist/js....