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

feat: Number of notes in folders #616

Closed
wants to merge 7 commits into from

Conversation

laudep
Copy link
Contributor

@laudep laudep commented Sep 5, 2020

Add numbers next to folders & tags displaying the amount of notes.

Display count for:

  • total notes in storage
  • notes in (sub)folders
  • notes for tag
  • attachments in storage
  • trashed notes in storage

Solves #602

@Flexo013 Flexo013 added the awaiting review ❇️ Pull request is awaiting a review. label Sep 7, 2020
@Rokt33r
Copy link
Member

Rokt33r commented Nov 3, 2020

@laudep I've made quite lots of changes from v0.10. Do you think you can resolve the conflicts? If you don't have time, I probably start working on this from Nov 11th.

@laudep
Copy link
Contributor Author

laudep commented Nov 4, 2020

@laudep I've made quite lots of changes from v0.10. Do you think you can resolve the conflicts? If you don't have time, I probably start working on this from Nov 11th.

Done 👍

@@ -50,6 +50,13 @@ const FolderListFragment = ({
)
}, [folderPathnames, storageId, sideNavOpenedItemSet])

const getFolderNoteCount = (folderPathname: string): number =>
Copy link
Member

Choose a reason for hiding this comment

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

We cannot use this method everywhere, this will cause each folder nav item component to iterate every note for every change. But populated folders in our hook state are having folderIdSet props. This should be useful to show number of notes in a folder.

@Rokt33r
Copy link
Member

Rokt33r commented Aug 24, 2021

@Rokt33r Rokt33r closed this Aug 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review ❇️ Pull request is awaiting a review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants