-
Notifications
You must be signed in to change notification settings - Fork 12
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
feat(ui): persist all data to localStorage #144
Conversation
…rsist pinia notifications store to local storage
…eed component import
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The toasts at the bottom are only showing for less than a second. I suggest to increase that to 2-3 seconds.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another likely unrelated issue, when I'm on the feed page (route "/") and refresh the page, I'm rerouted to "/discover".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Storing and caching looks good to me.
…ntions pages -- cache not working
Idk I feel like that's too long for success messages -- maybe just for errors. |
With less than a second you need to be expecting where the message appears and read for your life =D |
yeah but the success messages are all 1 word, just confirming success the action you already took (and already know the outcome to). Idk maybe we can get some more feedback and see what people think. |
…ll w/ caching on mention, hashtag, cashtags lists
…ge using @tanstack/query, refactor notifications store to only hold read status
…re label aligns with value
…d component to resolve rendering issues in webkitgtk
…navigating via keyboard or hovering
Feat: pagination + infinite scroll
Fix/minor quirks
pinia-plugin-persistedstate