Skip to content

#3909 gantt page filtering#4035

Merged
chpy04 merged 13 commits intodevelopfrom
#3909-Gantt-Page-Filtering
Apr 2, 2026
Merged

#3909 gantt page filtering#4035
chpy04 merged 13 commits intodevelopfrom
#3909-Gantt-Page-Filtering

Conversation

@EdgarButtner
Copy link
Copy Markdown
Contributor

@EdgarButtner EdgarButtner commented Mar 12, 2026

Changes

Reworked the frontend of the Filters popup for the Gantt page. Shifted from buttons to checkboxes and used accordions to expand collapse rather than listing all possible filters simultaneously.

I also changed the rendering to only render sections if they had more than one task. This is done in the GanttChartCollectionSection where we render a section.

Notes

No other notes

Screenshots

Img of filters:
Screenshot 2026-03-12 at 5 02 17 PM

Img of full page:
Screenshot 2026-03-12 at 5 06 51 PM

Img of smallest page:
Screenshot 2026-03-12 at 5 07 25 PM

Img of Revised Filtering
Screenshot 2026-03-27 at 4 44 51 PM

To Do

Everything should be done unless revisions or future changes are nessesary.

Checklist

It can be helpful to check the Checks and Files changed tabs.
Please review the contributor guide and reach out to your Tech Lead if anything is unclear.
Please request reviewers and ping on slack only after you've gone through this whole checklist.

  • [ ✓ ] All commits are tagged with the ticket number
  • [ ✓ ] No linting errors / newline at end of file warnings
  • [ ✓ ] All code follows repository-configured prettier formatting
  • [ ✓ ] No merge conflicts
  • [ ✓ ] All checks passing
  • [ ✓ ] Screenshots of UI changes (see Screenshots section)
  • [ ✓ ] Remove any non-applicable sections of this template
  • [ ✓ ] Assign the PR to yourself
  • [ ✓ ] No yarn.lock changes (unless dependencies have changed)
  • [ ✓ ] Request reviewers & ping on Slack
  • [ ✓ ] PR is linked to the ticket (fill in the closes line below)

Closes #3909

@staysgt staysgt self-requested a review March 12, 2026 23:15
@staysgt staysgt self-requested a review March 29, 2026 17:45
Copy link
Copy Markdown
Contributor

@staysgt staysgt left a comment

Choose a reason for hiding this comment

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

One really small comment, this looks really great!

return (
<Box display="flex" alignItems="center">
<GanttTaskBar
key={task.id}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Really small thing - the key should be in the outermost element inside of the mapping, so delete the key={task.id} from GanttTaskBar and put it in the Box element like before

@staysgt staysgt self-requested a review April 1, 2026 00:43
Copy link
Copy Markdown
Contributor

@staysgt staysgt left a comment

Choose a reason for hiding this comment

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

Great job!

Copy link
Copy Markdown
Contributor

@chpy04 chpy04 left a comment

Choose a reason for hiding this comment

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

lit

@chpy04 chpy04 merged commit 4db880e into develop Apr 2, 2026
4 checks passed
@chpy04 chpy04 deleted the #3909-Gantt-Page-Filtering branch April 2, 2026 03:46
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.

[Guest Page Improvements] - Gantt Page Filtering

3 participants