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

Add Unsorted tab to dashboard #1036

Closed
wants to merge 1 commit into from

Conversation

garygcchiu
Copy link

Add a new 'Unsorted' tab to the dashboard to include all items not associated with a list.

  • Dashboard Layout:

    • Import Inbox icon from lucide-react.
    • Add 'Unsorted' tab to the items function with the Inbox icon and path /dashboard/unsorted.
  • Unsorted Page:

    • Create apps/web/app/dashboard/unsorted/page.tsx.
    • Import and use the Bookmarks component to display items not associated with any list.
    • Set the query prop to filter items not associated with any list.
    • Add a header with the title 'Unsorted'.

Add a new 'Unsorted' tab to the dashboard to include all items not associated with a list.

* **Dashboard Layout**:
  - Import `Inbox` icon from `lucide-react`.
  - Add 'Unsorted' tab to the `items` function with the `Inbox` icon and path `/dashboard/unsorted`.

* **Unsorted Page**:
  - Create `apps/web/app/dashboard/unsorted/page.tsx`.
  - Import and use the `Bookmarks` component to display items not associated with any list.
  - Set the `query` prop to filter items not associated with any list.
  - Add a header with the title 'Unsorted'.
@MohamedBassem
Copy link
Collaborator

Hey, thanks for the PR. But I don't think this does what you think it does, and it probably doesn't work.

You can achieve what you want by creating a smart list with the filtering being -is:inlist instead.

@garygcchiu
Copy link
Author

Sorry, this was meant to be opened as a draft and was WIP. Thanks for the very quick review and guidance.

@garygcchiu garygcchiu deleted the add-unsorted-tab branch February 16, 2025 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants