Skip to content

Fix: limit unvisited feed items with native details collapse (closes #28)#99

Open
RealBhupesh wants to merge 2 commits intocofeed-19:masterfrom
RealBhupesh:fix/rss-unvisited-limit
Open

Fix: limit unvisited feed items with native details collapse (closes #28)#99
RealBhupesh wants to merge 2 commits intocofeed-19:masterfrom
RealBhupesh:fix/rss-unvisited-limit

Conversation

@RealBhupesh
Copy link
Copy Markdown

What this PR does

This PR limits the default number of visible unvisited items per feed and collapses the remainder behind a native HTML5 <details> section.

Why

Some feeds can have 50+ unvisited items, which creates a very long page and makes scanning multiple feeds difficult.

Changes

  • show first 20 unvisited items by default
  • show remaining items under:
    • See other N unvisited
  • keep existing behavior for:
    • marking one item as visited
    • favoriting items
    • “Mark all as visited” (applies to all unvisited items, not just visible ones)

Tests

  • added e2e test for collapsing behavior:
    • verifies 20 visible by default
    • verifies summary count for hidden items
    • verifies expanding shows all items
  • full e2e suite passes locally

Closes #28

@RealBhupesh
Copy link
Copy Markdown
Author

Quick UX question: after expanding several “See other N unvisited” sections, getting back to the top is a bit tedious.
Would you accept a tiny follow-up PR for a “Back to top” button (or “Collapse all” action), keeping everything native/simple?

Copy link
Copy Markdown
Member

@strdr4605 strdr4605 left a comment

Choose a reason for hiding this comment

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

“Back to top”

Where should it navigate to? top of page?


Please clean up unnecessary file changes in this PR. There should be only /e2e and /src file changes

@RealBhupesh RealBhupesh requested a review from strdr4605 April 1, 2026 10:34
@RealBhupesh
Copy link
Copy Markdown
Author

Where should it navigate to? top of page?

something like that, if someone wants to scroll back to any page which sits on top or in the middle it would be hassle for him to scroll and scroll (if he has opened all the titles)

Copy link
Copy Markdown
Member

@strdr4605 strdr4605 left a comment

Choose a reason for hiding this comment

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

image

We need to add similar spacing, check if we can use flexbox, and gap. Also maybe reduce the spacing between details and first items, this way details are like a continuation of the list and users can scan it easier.

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.

Limit number of unvisited per RSS Feed in the feeds list

2 participants