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.

Sign up for GitHub

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

Jump to bottom

Replace wrapper--cookieconsent with tailwind classes #4301

Merged
merged 4 commits into from
Dec 31, 2024
Merged

Replace wrapper--cookieconsent with tailwind classes #4301

merged 4 commits into from
Dec 31, 2024

Conversation

theskumar
Copy link
Member

@theskumar theskumar commented Dec 20, 2024 •
edited
Loading

  • One less scss rule, removes extra px to allow for better spacing in mobile view + html wrapping
  • Remove unused wrapper--relative
  • Remove unused wrapper--media-boxes
  • Remove unused card--reviewers

@theskumar theskumar self-assigned this Dec 20, 2024
@theskumar theskumar requested a review from frjo December 20, 2024 11:41
@theskumar theskumar changed the title Replace wrapper--cookieconsent with twailwind Replace wrapper--cookieconsent with tailwind classes Dec 20, 2024
@frjo frjo added Type: Maintenance Type: Patch Mini change, used in release drafter labels Dec 31, 2024
@frjo frjo merged commit 0846f65 into main Dec 31, 2024
7 checks passed
@theskumar theskumar deleted the css/cookieconsent branch January 2, 2025 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@frjo frjo frjo approved these changes

Assignees

@theskumar theskumar

Labels
Type: Maintenance Type: Patch Mini change, used in release drafter
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@theskumar @frjo