Skip to content

feat: navigation update for results page#9000

Open
bookwormsuf wants to merge 6 commits intodevelopfrom
feat/navigation-updates
Open

feat: navigation update for results page#9000
bookwormsuf wants to merge 6 commits intodevelopfrom
feat/navigation-updates

Conversation

@bookwormsuf
Copy link
Copy Markdown
Contributor

Problem

Navigation pattern in the responses section was inconsistent and lacked clear visual hierarchy, making the page navigation bloated and difficult to use.

Closes [linear ticket TBD]

Solution

Refactored navigation architecture by moving from horizontal top navigation to a vertical side navigation pattern within the responses section, providing clearer information hierarchy and improved UX.

Breaking Changes

  • Yes - this PR contains breaking changes
    • Details ...
  • No - this PR is backwards compatible

Features:

  • Implemented side navigation panel for responses section

Improvements:

  • Moved navigation from top horizontal tabs to left-side vertical navigation

Bug Fixes:

  • N/A

Before & After Screenshots

Description Before After
Responses empty state image image
Responses locked state image image
Responses unlocked state image image
Feedback empty state image image
Feedback unlocked state image image
Charts empty state image image
Charts locked state image image
Charts unlocked state Screenshot 2025-12-17 at 7 42 24 PM Screenshot 2025-12-17 at 7 52 07 PM

Tests

Navigation Flow Tests:

  1. Navigate to the responses section
  2. Verify side navigation appears on the left
  3. Click through each navigation item in the side nav
  4. Verify responsive behavior
  5. Test navigation state persistence when switching between pages

Visual Tests:

  1. Confirm navigation hierarchy is clear and intuitive
  2. Verify no UI elements are overlapping or misaligned
  3. Check that the layout appears less cluttered than before

Note: I wasn't sure how to make payment forms on local, so wasn't able to test the views with payments

Deploy Notes

New environment variables:

  • None

New scripts:

  • None

New dependencies:

  • None

New dev dependencies:

  • None

@bookwormsuf bookwormsuf requested a review from a team as a code owner December 17, 2025 11:54
Comment thread frontend/src/features/admin-form/responses/FormResultsLayout.tsx Fixed
@bookwormsuf bookwormsuf changed the title navigation update for results page feat/navigation update for results page Dec 17, 2025
@bookwormsuf bookwormsuf changed the title feat/navigation update for results page feat:navigation update for results page Dec 17, 2025
@bookwormsuf bookwormsuf changed the title feat:navigation update for results page feat: navigation update for results page Dec 17, 2025
@datadog-opengovsg
Copy link
Copy Markdown

datadog-opengovsg Bot commented Dec 17, 2025

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: c1773e7 | Docs | Was this helpful? Give us feedback!

…tion or class'

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
bookwormsuf and others added 2 commits December 18, 2025 14:09
…tion or class'

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
@eliotlim eliotlim added the queue for merge Priority for merge and release (especially before monorepo refactoring) label Feb 23, 2026
Copy link
Copy Markdown
Contributor

@eliotlim eliotlim left a comment

Choose a reason for hiding this comment

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

looks okay to me -- there's a stray comment, and I'd wanna test the results page:

  • on desktop
  • on mobile

Comment on lines +40 to +46
// Temporary debug - remove after checking
console.log('Charts Debug:', {
isChartsEnabled,
isFormEncryptMode,
formResponseMode: form?.responseMode,
shouldShowCharts,
})
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.

console log to be removed?

@linear
Copy link
Copy Markdown

linear Bot commented Feb 23, 2026

@kevin9foong kevin9foong removed the queue for merge Priority for merge and release (especially before monorepo refactoring) label Feb 23, 2026
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.

3 participants