Fix TCF overlay issues on mobile sized screens #5157
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
gilluminate
force-pushed
the
fix/mobile-tcf-overlay
branch
from
0478536
to
887702b
Compare
| @@ -537,7 +537,7 @@ div#fides-banner-inner .fides-privacy-policy { | |||
| } | |||
|
|
|||
| div#fides-banner-inner .fides-privacy-policy, | |||
| button.fides-banner-button.fides-manage-preferences-button, | |||
| button.fides-banner-button.fides-banner-button-tertiary, | |||
There was a problem hiding this comment.
fixes the Manage Preferences button size on mobile
| @media screen and (max-width: 768px) { | ||
| .fides-tcf-banner-container #fides-banner { | ||
| padding: 24px; | ||
| } | ||
| } |
There was a problem hiding this comment.
since the TCF specific styles are below the mobile media query styles, the mobile adjustment was being overridden and not being applied. This fix does the same thing, but specific to TCF padding.
|
Test summaryRun details
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Cloud |
|
Test summaryRun details
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Cloud |
Description Of Changes
Fixes a newly introduced issue with overlay banner layout, specifically with the positioning of the buttons.
Includes some added polish for noticed issues while testing the mobile version.
Code Changes
Steps to Confirm
Pre-Merge Checklist
CHANGELOG.md