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

Changed styles cookies #975

Merged
merged 6 commits into from
Nov 27, 2024
Merged

Changed styles cookies #975

merged 6 commits into from
Nov 27, 2024

Conversation

Chelakhovl
Copy link
Collaborator

Updated styles for Buttons and added z index

@Chelakhovl Chelakhovl self-assigned this Nov 26, 2024
@Chelakhovl
Copy link
Collaborator Author

Removed Modal Logic:

Eliminated the modal functionality.
Retained only the initial cookie banner with "Allow" and "Decline" buttons.
Updated Styles:

Adjusted styles to center content properly and ensure responsive design.
Removed unnecessary modal-related styles.
Updated z-index:

Increased z-index for the cookie banner to 2000 to prevent overlapping issues with tooltips and other elements.
Updated Tests:

Removed tests related to the modal.
Updated unit tests to verify the correct behavior of the cookie banner:
Ensures buttons and links render correctly.
Verifies that cookies are saved on "Allow."
Confirms the banner disappears on "Decline."
Changes Made:
Component:

Removed modal-related code from CookieMod.jsx.
Updated logic to handle only the initial banner state.
Styles:

Cleaned up unused modal styles in CookieMod.module.css.
Adjusted banner layout and responsive behavior.
Tests:

Updated CookieMod tests to reflect simplified functionality.
Removed unnecessary modal-related test cases.

@Chelakhovl Chelakhovl merged commit 13a0a6d into develop Nov 27, 2024
4 checks passed
@Chelakhovl Chelakhovl deleted the 974-update-styles-cookies branch December 7, 2024 15:12
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.

2 participants