[task-board] all tasks modal - #1
Open
jayporta wants to merge 3 commits into
Open
Conversation
A flat, cross-column list of every task behind a "Show all" button in the toolbar: title, created date, due date, and status, with status filter chips, a sort control, and single or bulk delete. Filter and sort are local to the modal — the board underneath is untouched. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Replace the status filter chips with a single-select "Filter by" dropdown matching the sort control, defaulting to All. Changing it clears the row selection, so a task the filter hides can never stay ticked out of sight — which lets the selection hold only visible ids and drops a per-render scan. Deleting one row no longer clears a batch the user has ticked up. Move both dialog close buttons out of DialogTitle into a shared DialogCloseButton: MUI names a dialog after its whole title element, so a button in there was being read out as part of that name. The MuiDialogTitle override existed only for that layout and is no longer needed. Also: memoise the filtered and sorted rows, stick the table headings to the scroll box, build the sort menu from SORT_KEYS rather than a cast, share the delete confirmation copy from lib, and cover the sort tie-break with a test. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Compare select-all against the rows themselves rather than a length match, which was only correct while the selection stayed a subset of what the filter shows. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds a Show all button beside the toolbar search that opens a modal listing every task across all columns as one flat list. List can be filtered and sorted. Tasks can be deleted individually or in bulk.