Skip to content

Conversation

@GodKratos
Copy link

📝 Description

The recently requested carousel uses hard coded display settings to adjust how item cards are displayed. These settings do not produce a good display and the width override of the other carousel views causes the scrolling of the items to either cut off early or too late depending on the width of the screen.

This PR adjusts the way the recently requested carousel is displayed by doing the following:

  • dynamically sets the numVisible and numScroll parameters of the Recently Requested carousel based on the window size and the item card size (when adjusted for mobile view)
  • removes the width override of the carousel items so that the scrolling size of items matches the item size (this fixes the issue of scrolling not matching the contents)
  • centres the contents of the carousel items to look better

📸 Screenshots (if applicable)

Before the Change:

Carousel scrolling would stop early cutting off the last item:
image

After the Change:

Carousel items scroll evenly no matter the window size:
image

@tidusjar
Copy link
Member

tidusjar commented Oct 20, 2025

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@sonarqubecloud
Copy link

@tidusjar
Copy link
Member

Thanks for this, please ignore the failing status checks, everything is passing but it doesn't have permissions to update the PR (i'll sort that).

I'll review soon

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