[Feature] Mark as read for articles#267
[Feature] Mark as read for articles#267imhariprakash wants to merge 10 commits intomedyo:developfrom
Conversation
medyo
left a comment
There was a problem hiding this comment.
Well done @imhariprakash 👌
I have two suggestions:
First, the current implementation relies on the source to track read posts, which means you need to modify each source card. Ideally, we should store read posts by their unique ID and simply check if the ID exists. This avoids depending on the source and allows us to have a consistent limit across all sources.
Another small improvement: some users like to see which posts they already read. It would be nice to add a setting where users can choose how read posts appear.
For example:
- Dim read posts (default)
- Hide read posts
- Show normally
79c4e4b to
c13feae
Compare
c13feae to
2184806
Compare
|
Change 1:
Change 2:
Kindly review the changes. |
…nent to handle 'all' tag click
Allow to reset selected tag from the header tags
…sh/hackertab.dev into imhariprakash-feature_mark_as_read2
|
@imhariprakash I did a few optimizations and improved the UI, please give it a try and let me know what you think |
|
Hi, @medyo |


Description
Changes
src/stores/readPosts.ts):readPosts(per-source arrays of read post IDs) andmarkAsRead(source, postId).src/components/List/ListComponent.tsx):readPosts[source].src/components/Elements/CardWithActions/CardItemWithActions.tsx):source={meta.value}into the list (read posts per source).src/assets/App.css):Behaviour