Skip to content
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

Favorite pages #1545

Merged
merged 3 commits into from
Oct 28, 2024
Merged

Favorite pages #1545

merged 3 commits into from
Oct 28, 2024

Conversation

mejo-
Copy link
Member

@mejo- mejo- commented Oct 22, 2024

๐Ÿ“ Summary

Allow to mark pages as favorites. Favorites will get additionally listed in a collapsible "Favorites" section at the top of the page list.

Resolves: #300

๐Ÿ–ผ๏ธ Screenshots

No Favorites Favorites expanded Favorites collapsed Favorites actions
image image image image
Screencast
recording1

๐Ÿ Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests

@mejo- mejo- added enhancement New feature or request 3. to review labels Oct 22, 2024
@mejo- mejo- self-assigned this Oct 22, 2024
@mejo- mejo- requested a review from max-nextcloud as a code owner October 22, 2024 15:27
@mejo- mejo- requested a review from juliusknorr October 22, 2024 15:27
Copy link
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

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

Looks really really nice! :) I have no change requests, and am looking forward to using it in action!

Copy link
Member

@nimishavijay nimishavijay left a comment

Choose a reason for hiding this comment

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

Awesome! That looks super nice! ๐Ÿš€
Only one small change: the chevron arrow on the "Favorites" item should be on the right, so that it's center aligned with the + button of the other pages :) Other than that it's perfect!
Approving to not block for another day ๐Ÿ˜… Looking forward to seeing this in our handbook!

mejo- added 2 commits October 23, 2024 10:57
Signed-off-by: Jonas <jonas@freesources.org>
Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- force-pushed the feat/favourite_pages branch from de29f8d to 8f21a37 Compare October 23, 2024 09:29
@mejo-
Copy link
Member Author

mejo- commented Oct 23, 2024

Only one small change: the chevron arrow on the "Favorites" item should be on the right, so that it's center aligned with the + button of the other pages :) Other than that it's perfect!

Good point, changed now:

image

@mejo- mejo- force-pushed the feat/favourite_pages branch 2 times, most recently from 9e2b590 to bc286fd Compare October 23, 2024 09:59
Fixes: #300

Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- force-pushed the feat/favourite_pages branch from bc286fd to 6c6e8da Compare October 23, 2024 10:04
Copy link
Member

@juliusknorr juliusknorr left a comment

Choose a reason for hiding this comment

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

Very nice ๐Ÿ‘

@mejo- mejo- merged commit 0882cb3 into main Oct 28, 2024
51 of 53 checks passed
@mejo- mejo- deleted the feat/favourite_pages branch October 28, 2024 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review enhancement New feature or request
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Favourite pages
4 participants