Skip to content

Conversation

@Kwestman1
Copy link

@Kwestman1 Kwestman1 commented Apr 13, 2025

Resolves:

front page sections show two rows when you click next multiple times #9253

Changes:

Fixed carousel formatting on homepage. The issue was that, given the "padding-right", originally set to 30 px, on the individual images, the box would spill over into the next slide and push the last image to the front of the carousel. I changed the padding to be 24 px so that all 5 images appear on a given slide.

Test Coverage:

Testing: running npm test

Screenshot 2025-04-13 at 4 02 56 PM

Locally: running npm start

Before change -->

Screenshot 2025-04-13 at 4 27 56 PM

Issue seen here, once you click to the next slide, the last image doesn't appear until you click back to the beginning slide

Screenshot 2025-04-13 at 4 28 04 PM Screenshot 2025-04-13 at 4 28 09 PM

After Change -->

Screenshot 2025-04-13 at 4 28 25 PM Screenshot 2025-04-13 at 4 28 30 PM

@github-actions
Copy link

github-actions bot commented Apr 13, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@Kwestman1
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

Copy link

@DDCcarr1 DDCcarr1 left a comment

Choose a reason for hiding this comment

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

Seems about good. Fairly simple fix.

@Kwestman1
Copy link
Author

Kwestman1 commented Apr 25, 2025

Seems about good. Fairly simple fix.

Can I request that a maintainer looks at this? To approve the workflow.

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