fix: hide arrow buttons #408
Merged
+58
−55
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📌 Purpose
Fixes the arrow navigation buttons on the home page that were visible but non-functional when there wasn't enough content to scroll through. Previously, the arrows would show up even with only 1-2 subjects, giving users a poor experience with non-working buttons.
Corresponding issue: closes #399
🖼️ Showcase
The arrows disappear when no scrolling available
Since the data wasn't seeded , the ss are not available
🔧 Changes
Modified the conditional rendering logic in PapersCarousel.tsx to only show arrow buttons when there's actually content to scroll through
Added check for displayPapers.length > chunkSize to ensure arrows appear only when necessary
Improved user experience by hiding non-functional UI elements
Additional Notes
The fix maintains responsive behavior across all screen sizes (mobile, tablet, desktop)
No breaking changes to existing functionality
Arrows will automatically show/hide based on content availability
Testing can be done once database connection is properly configured with test data