Repository navigation
Aditya Dhayapulay - Add smart search to on-site member table - #5718
Conversation
Add column suggestions, literal match highlighting, combined 350ms-debounced filters and immediate reset. Preserve keyboard focus and readable dark-mode styles while avoiding table-body updates during typing. Implements the medium-priority Jae/Divanshu on-site member search task. Use the existing Project Details table as confirmed; no routing or worker-data API changes.
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Replace generic status/group containers with output and a fieldset with legend, preserving the responsive grid and accessible filter name. Add regression coverage for their native semantics. Resolve both SonarCloud javascript:S6819 findings on PR #5718.
Replace generic status/group containers with output and a fieldset with legend, preserving the responsive grid and accessible filter name. Add regression coverage for their native semantics. Resolve both SonarCloud javascript:S6819 findings on PR #5718.
Include the complete Google Doc request so reviewers can compare the implementation with all five requested enhancements. PR #5718
rahulrubugunday
left a comment
There was a problem hiding this comment.
I have reviewed your PR locally (102f0ef, backend on development) and it works as per requirement.
- Typing "team" in Team shows the "Team A" suggestion, filters to 2 of 3 members and highlights the matching text.
- Filters combine, and Arrow Down + Enter picks a suggestion (First Name "ca" then Cailin shows 1 of 3). Escape closes the suggestions.
- A search with no match shows "No members match these filters."
- Clear All Filters empties every search and restores all rows right away, even while typing is pending.
- At 375px the table scrolls sideways without the page overflowing. Checked in both light and dark mode, and all 24 tests in the ProjectDetails test folder pass.
AaditTrivedi
left a comment
There was a problem hiding this comment.
Reviewed by Aadit Trivedi.
Checked out the PR (no conflicts with development) and ran its tests: 23/23 passing. Adds smart search to the on-site member table, with the new search logic covered by tests.
Approving.
iAbhi001
left a comment
There was a problem hiding this comment.
I tested frontend PR #5718 locally at commit 102f0ef paired with the backend on development in both light and dark mode. Navigating to the Project Details page (/bmdashboard/projects/) confirms the enhanced smart search and UX improvements on the "Members working on site today" table function cleanly. All 24 Vitest tests in ProjectDetails/tests pass without regressions. Tested column search filtering across First Name, Role, and Team, confirming that substring matches highlight properly and table filtering responds cleanly after the 350ms debounce. Keyboard navigation via ArrowDown and Enter selects suggestions, while Escape and blur dismiss the menu without clearing the typed input. Combining filters narrows rows accurately, and searching non-existent terms triggers the "No members match these filters" empty state. The Clear All Filters button immediately restores the full table. Tested responsive behavior at 375px width and verified horizontal scrolling without page overflow, with clear legibility and contrast across both themes. Approving these changes.
d503a9b
102f0ef to
d503a9b
Compare
|
|
Moved all three pictures to direct attachments in the PR description and removed them from the PR commits. I verified that the attachments render and match the originals. Application and test files are byte-for-byte unchanged from the previously tested head. Fresh GitHub tests, preview and Sonar pass. The history cleanup reset the earlier approvals, so I requested renewed reviews. |
|
Thank you all, merging! |



Description
Original task request
Original task in Google Docs
Complete task captured directly from the original document; the requested wording is unchanged.
Implement the medium-priority Jae/Divanshu: Enhanced Smart Search + UX Improvements for the Members Working on Site Today table task.
Add unique suggestions for First Name, Last Name, Role and Team, literal matching-text highlights, a Clear All Filters button, 350ms debounced filtering, and scoped light/dark styles.
Related PRS (if any)
Frontend only; no paired backend PR or API changes.
Main changes explained
How to test
feat/site-members-smart-searchand use Node20 with the repository's existing dependencies.npx vitest run src/components/BMDashboard/Projects/ProjectDetails/__tests__(24 tests including the existing ProjectDetails test)./bmdashboard/projects/<projectId>).Validation already performed
Screenshots or videos of changes
Nine code/UI screenshots and a short compressed walkthrough are staged in the developer's Week5 media folder. They are not attached to this PR yet.
Note
The task listed
/bmdashboard/tools/day, but that URL resolves to the tool-detail route in current development. Aditya confirmed using the existing Project Details member table instead; this PR does not change routing.The existing table currently displays sample rows. Those are preserved; the new optional
membersprop supports supplied records, but this PR does not introduce live on-site worker-data integration or claim authenticated end-to-end API testing. No task/issue number was supplied, so no unrelated GitHub issue is auto-closed.Documentation cleanup: screenshots are attached directly to this description and removed from the PR commit range. Application and test files are byte-for-byte unchanged. Previously tested head
102f0ef700corresponds to cleaned headd503a9bf7b; all fresh GitHub test, preview and Sonar checks pass.