Skip to content

Michael 020/bug/fix load more button #111

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

michael-020
Copy link
Contributor

Related Tickets & Documents

Fixes: #2654

Description

  • Improved the Load More button UX by making it consistently visible and adding better loading states
  • Implemented prefetching for posts when users scroll near the button
  • Added visual feedback during loading states

Changes

  • Modified showLoadMore condition to keep button visible during loading states
  • Added loading text for better UX feedback
  • Implemented intersection observer for prefetching posts when scrolling near the button
  • Implemented buffer system for smooth content loading
  • Added proper cleanup for prefetching states

Type of Change

  • Chore (maintenance, refactoring, tooling updates)
  • Bug fix (non-breaking change that fixes an issue)
  • New feature (change that adds functionality)
  • Breaking Change (may require updates in existing code)
  • UI improvement (visual or design changes)
  • Performance improvement (optimization or efficiency enhancements)
  • Documentation update (changes to README, guides, etc.)
  • CI (updates to continuous integration workflows)
  • Revert (undo a previous commit or merge)

Testing

  • Verified Load More button remains visible during post loading
  • Tested loading indicator appears correctly when fetching posts
  • Confirmed prefetching triggers only once when Load More button comes into view
  • Verified prefetching state management prevents multiple simultaneous calls

Demo

Screen.Recording.2025-04-20.at.11.03.14.PM.1.mov

Prefetching demo:

Screen.Recording.2025-04-20.at.11.40.14.PM.mov

Environment and Dependencies

  • New Dependencies: None
  • Configuration Changes: None required

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added corresponding tests
  • I have run the build command to ensure there are no build errors
  • My changes have been tested across relevant browsers/devices
  • For UI changes, I've included visual evidence of my changes

Signed-off-by: Michael Hosamani <michaelhosamani26@gmail.com>
Signed-off-by: Michael Hosamani <michaelhosamani26@gmail.com>
Signed-off-by: Michael Hosamani <michaelhosamani26@gmail.com>
Copy link

keploy bot commented Apr 20, 2025

Nice Pr 😄. We currently support Unit Test Generation only for Go projects.
Stay tuned – support for more languages is coming 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.

1 participant