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

795-fix: Fix slider button #797

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

sergiozeppo
Copy link
Collaborator

What type of PR is this? (select all that apply)

  • πŸ• Feature
  • πŸ› Bug Fix
  • 🚧 Breaking Change
  • πŸ§‘β€πŸ’» Code Refactor
  • πŸ“ Documentation Update

Description

Fix alignment of arrow controls of the "RS in pictures" carousel.
I had to use the !important marker because need to overwrite the external react-responsive-carousel styles.

Related Tickets & Documents

Screenshots, Recordings

Before:
image

After:
image

Added/updated tests?

  • πŸ‘Œ Yes
  • πŸ™…β€β™‚οΈ No, because they aren't needed
  • πŸ™‹β€β™‚οΈ No, because I need help

[optional] Are there any post deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

image

@github-actions github-actions bot changed the title 795-fix: Fix alignment of arrow controls 795-fix: Fix slider button Feb 18, 2025
@sergiozeppo sergiozeppo self-assigned this Feb 18, 2025
@KristiBo KristiBo added bug Something isn't working preview and removed preview labels Feb 21, 2025
Copy link

Lighthouse Report:

  • Performance: 97 / 100
  • Accessibility: 100 / 100
  • Best Practices: 100 / 100
  • SEO: 100 / 100

View detailed report

@@ -118,18 +118,18 @@ figure {
.control-prev {
--slider-arrow-btn-height: 45px;

top: calc(50% - var(--slider-arrow-btn-height) / 2);
top: calc(50% - var(--slider-arrow-btn-height) / 2) !important;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we avoid using !important and increase css specificity some other way?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working preview
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix community slider buttons misalignement
4 participants