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

💄 [#1677] Making cookiebanner margins and modal same on mobile #737

Merged
merged 1 commit into from
Aug 22, 2023

Conversation

jiromaykin
Copy link
Contributor

@jiromaykin jiromaykin commented Aug 21, 2023

in order to test/review this: if you have previously clicked on the cookiebanner (making it invisible), you can make it appear in your browser again by looking in your Dev Tools for an >> Application >> Cookies/Local Storage tab and delete the "cookieBannerAccepted" cookie, or in >> Storage Tab >> Cookies >> localhost and remove "cookieBannerAccepted" (usually delete cookies via right-click)

https://taiga.maykinmedia.nl/project/open-inwoner/task/1677

@jiromaykin jiromaykin changed the title 🎨 [#1677] Corrected cookiebanner margins and modal 🎨 [#1677] Making cookiebanner margins and modal same on mobile Aug 21, 2023
@jiromaykin jiromaykin marked this pull request as ready for review August 21, 2023 10:22
@jiromaykin jiromaykin changed the title 🎨 [#1677] Making cookiebanner margins and modal same on mobile 💄 [#1677] Making cookiebanner margins and modal same on mobile Aug 21, 2023
@jiromaykin jiromaykin force-pushed the feature/1677-make-mobile-cookiebanner-modal branch 2 times, most recently from 60fc70f to 02ee1a5 Compare August 21, 2023 10:33
@codecov-commenter
Copy link

codecov-commenter commented Aug 21, 2023

Codecov Report

Merging #737 (e0285cb) into develop (ae69cd9) will not change coverage.
The diff coverage is n/a.

❗ Current head e0285cb differs from pull request most recent head 95eaaac. Consider uploading reports for the commit 95eaaac to get more accurate results

@@           Coverage Diff            @@
##           develop     #737   +/-   ##
========================================
  Coverage    96.27%   96.27%           
========================================
  Files          674      674           
  Lines        24007    24007           
========================================
  Hits         23113    23113           
  Misses         894      894           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@jiromaykin jiromaykin force-pushed the feature/1677-make-mobile-cookiebanner-modal branch from f64cd7b to 4f2b54d Compare August 21, 2023 13:24
@jiromaykin jiromaykin force-pushed the feature/1677-make-mobile-cookiebanner-modal branch from e0285cb to 95eaaac Compare August 21, 2023 13:25
@alextreme alextreme merged commit 5d1233b into develop Aug 22, 2023
10 checks passed
@alextreme alextreme deleted the feature/1677-make-mobile-cookiebanner-modal branch August 22, 2023 17:54
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

Successfully merging this pull request may close these issues.

4 participants