Skip to content

feat(frontend): simplify results page layout+styling & bump pnpm with deps #1018

Merged
gildesmarais merged 5 commits into
mainfrom
ui-fixes-corrected
Jun 13, 2026
Merged

feat(frontend): simplify results page layout+styling & bump pnpm with deps #1018
gildesmarais merged 5 commits into
mainfrom
ui-fixes-corrected

Conversation

@gildesmarais

@gildesmarais gildesmarais commented Jun 13, 2026

Copy link
Copy Markdown
Member

This pull request introduces several improvements and refactorings to the frontend codebase, focusing on UI consistency, user guidance, dependency updates, and test reliability. The most notable changes include the introduction of a reusable Notice component to standardize notification UI, enhancements to the bookmarklet feature with contextual help, updates to dependencies and build configuration, and refinements to test expectations and API URLs.

UI Refactoring and Consistency:

  • Introduced a new reusable Notice component and refactored various notification and alert UIs throughout the app (App.tsx, AppPanels.tsx) to use this component for consistency and maintainability. [1] [2] [3] [4]
  • Updated the DominantField component to render an arrow icon for action buttons when the text is '>', improving visual clarity. [1] [2]

Bookmarklet Feature Enhancements:

  • Improved the bookmarklet feature by adding a help notice that explains how to use it, which can be triggered and dismissed by the user. This includes passing an onClick handler to the Bookmarklet component and displaying contextual guidance in the UI. [1] [2] [3] [4] [5] [6] [7]

Dependency and Build Configuration Updates:

  • Updated frontend dependencies in package.json (including pnpm, eslint, prettier, and others) and added pnpm-workspace.yaml to the Docker build context to ensure correct dependency resolution and workspace configuration. [1] [2] [3] [4]
  • Added additional frontend build and cache directories to .dockerignore for cleaner Docker images.

Test and Mock Improvements:

  • Updated test and mock files to use https for the OpenAPI URL instead of http, aligning with production usage and improving test reliability. [1] [2] [3] [4] [5] [6]
  • Adjusted test expectations for UI text to match new UI content, such as changing button and heading text assertions. [1] [2]

Minor UI and Accessibility Adjustments:

  • Added and adjusted class names and ARIA attributes for improved layout and accessibility, such as layout-rail-reading and aria-label/ariaLive props in form and notice components. [1] [2] [3]

These changes collectively improve the frontend's maintainability, user experience, and reliability in both development and production environments.

@gildesmarais gildesmarais force-pushed the ui-fixes-corrected branch 2 times, most recently from 07ff0e6 to 7af3e31 Compare June 13, 2026 07:03
@gildesmarais gildesmarais changed the title refactor: simplify and unify frontend results page layout and styling feat: simplify frontend results page layout+styling & bump pnpm with deps Jun 13, 2026
@gildesmarais gildesmarais changed the title feat: simplify frontend results page layout+styling & bump pnpm with deps feat(frontend): simplify results page layout+styling & bump pnpm with deps Jun 13, 2026
@gildesmarais gildesmarais marked this pull request as ready for review June 13, 2026 07:10
@gildesmarais gildesmarais merged commit 0758f1a into main Jun 13, 2026
15 checks passed
@gildesmarais gildesmarais deleted the ui-fixes-corrected branch June 13, 2026 07:11
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.

1 participant