-
-
Notifications
You must be signed in to change notification settings - Fork 185
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
#445-favourite section #447
Conversation
Hey @narayan954 do verify this |
Yes, sure allow me some time. |
@Soulpurr , can you please match it against the current implementation and adapt/improve it? |
The sidebar one right? |
yes |
hii @narayan954 i have updated this according to #437 as it doesn't have fav section for smaller screen i have added it so pls verify and approve it. |
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.
looks good to me!
So i have added a separate favourite post section.
Right now in order to navigate favourite section its link is located at the profile bar which is fine as of now and for mobile devices.
But after merging #437 we can directly navigate from sidebar for larger screens.
You just need to add a link to this page in sidebar favourite item.
fixes #445
scrnli_08_06_2023_13-33-16.webm