-
Notifications
You must be signed in to change notification settings - Fork 14
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
DDFLSBP-651 - Implement cookie blocking placeholder #1220
DDFLSBP-651 - Implement cookie blocking placeholder #1220
Conversation
…custom cookie functionality, that are not configurable in either the cookieinformation module or cookieinformation dashboard. For now I have added a single preprocessing function, that modifies all video media iFrames, adding new attributes that cookieinformation is used for handling cookies.
…ceholder to all video media elements.
…nt-cookie-blocking-placeholder
…ia fields than video
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hvis jeg afviser alle cookies ser jeg dette på en side hvor der er indsat et Vimeo klip:
Det er muligvis mit lokalmiljø som driller med stylingen, men under alle omstændigheder kunne det måske være en forbedring at lave <div class="video-embed__wrapper">
elementet lidt lavere, når cookie-placeholder
vises, således at der ikke er så meget "tom" plads?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great feature, just a few minor suggestions...
web/themes/custom/novel/templates/fields/field--media--video.html.twig
Outdated
Show resolved
Hide resolved
…slatable aria-label text
…m by adding a pagefold div
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🎉
Link to issue
https://reload.atlassian.net/browse/DDFLSBP-651
Description
This PR introduces the following:
A new custom module (dpl_cookies), used for handling all custom cookie functionality, that is not handled in either the cookieinformation module or cookieinformation dashboard. For now this includes a preprocess hook that adds 3 new data attributes to all video media elements. These attributes are need by the cookieinformation script when handling user cookie consent.
A new template specifically targeting field--media--video. In this template we have added our custom cookie placeholder, which gets shown to user who have not yet consented to using cookies, in place of the original content.
Screenshot of the result
Screenrecording showing the result:
Screen.Recording.2024-06-06.at.14.13.01.mov
Additional comments or questions
This PR has a sister PR in the dpl-design-system: danskernesdigitalebibliotek/dpl-design-system#650