Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Skripte, die nach Ablehnung geladen werden #204

Closed
MC-PMOE opened this issue Nov 29, 2021 · 2 comments
Closed

Skripte, die nach Ablehnung geladen werden #204

MC-PMOE opened this issue Nov 29, 2021 · 2 comments

Comments

@MC-PMOE
Copy link
Contributor

MC-PMOE commented Nov 29, 2021

Es wäre cool wenn es neben den Skripten die nach Einverständnis geladen werden auch die Möglichkeit gäbe Skripte nach Ablehnung zu laden (als clean up z.B.).
Beispiel wäre bei mir aktuell, dass ich Mapbox erst nach Zustimmung laden lasse. Wenn zugestimmt wird packt Mapbox 2 Einträge in den localStorage. Wird jetzt im nachhinein die Cookie Präferenz geändert und externe Medien ausgeschaltet, würde ich gerne via localStorage.removeItem diese Einträge wieder entfernen.

@IngoWinter
Copy link
Member

Das Löschen von Cookies oder Webstorage Einträgen ist nicht notwendig (und bringt auch keinem was). Jede zusätzliche Zeile Code macht das Addon komplexer, fehleranfälliger und pflegeintensiver. Deswegen bin ich gegen eine solche Erweiterung.

@MC-PMOE
Copy link
Contributor Author

MC-PMOE commented Nov 30, 2021

Das Löschen von Cookies oder Webstorage Einträgen ist nicht notwendig (und bringt auch keinem was)

War mir so nicht bewusst, dachte es könnte Probleme geben wenn trotz Ablehnung derartiges zurückbleibt

Jede zusätzliche Zeile Code macht das Addon komplexer, fehleranfälliger und pflegeintensiver

Das stimmt wohl, könnte dann natürlich um Komplexität zu vermeiden auch hiermit gelöst werden #156 falls sich andere spezielle Anwendungsfälle ergeben und ist vermutlich auch besser als den auszuführenden Code mit in die Cookie Definition zu packen.

@MC-PMOE MC-PMOE closed this as completed Nov 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants