Skip to content

Conversation

@sbruel
Copy link
Contributor

@sbruel sbruel commented Sep 12, 2025

Summary

Infinite scrolling has agents marketplace has a few issues:

  • if the viewport is tall enough by default to show more than 6 items (and there is no scrollbar), there is no way to get more items to show
  • when the window is resized to become bigger, and could potentially show more agents, the agents are not shown
  • when switching between category tabs before scrolling, he infinite scroll does not work on the other categories.

This PR fixes all these issues. Check the videos for before / after.

Before

2025-09-12-infinite-scroll-before-s.mov

After

2025-09-12-infinite-scroll-after-s.mov

Change Type

  • Bug fix (non-breaking change which fixes an issue)

Testing

Follow the 3 different scenarios described in the summary or in the video.

Checklist

Please delete any irrelevant options.

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my own code
  • I have commented in any complex areas of my code
  • My changes do not introduce new warnings
  • I have written tests demonstrating that my changes are effective or that my feature works
  • Local unit tests pass with my changes

@sbruel sbruel force-pushed the fix-agents-marketplace-infinite-scroll branch from 60d92b0 to 9916c06 Compare September 12, 2025 08:59
@sbruel sbruel marked this pull request as ready for review September 12, 2025 09:02
@danny-avila danny-avila changed the title fix: Auto-fetch agents to fill viewport in marketplace infinite scroll 🪟 fix: Auto-fetch agents to fill Viewport in Marketplace Scroll Sep 12, 2025
@danny-avila danny-avila changed the base branch from main to dev September 12, 2025 14:57
@danny-avila danny-avila merged commit 30c24a6 into danny-avila:dev Sep 12, 2025
6 checks passed
patricksn3ll pushed a commit to patricksn3ll/LibreChat that referenced this pull request Dec 11, 2025
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