OCD-5295 | Refactors to the filter and search components#1511
Open
mattystank wants to merge 51 commits into
Open
OCD-5295 | Refactors to the filter and search components#1511mattystank wants to merge 51 commits into
mattystank wants to merge 51 commits into
Conversation
…aying results - Replaced table structures with card components in inactive certificates, real world testing, SED, and SVAP search views. - Updated sorting functionality to use ChplSortControls. - Enhanced loading indicators with CircularProgress. - Improved layout and styling for better responsiveness and user experience. - Added a new background color to the theme palette for better visual distinction. [#OCD-5295]
…y components for consistent layout - Removed redundant header and body styles from various search views (Corrective Action, Decertified Products, Decision Support Interventions, Inactive Certificates, Real World Testing, SED, SVAP). - Integrated ChplPageHeader and ChplPageBody components to standardize the presentation of page titles and subtitles across search pages. - Updated subtitles to provide detailed descriptions and context for each search page. - Adjusted layout and styling for better responsiveness and visual consistency. [#OCD-5295]
- Added informational headers and descriptive content to all 9 collection/search pages - Maintained card-based result display using ChplSearchResultCard - Updated component props to use cardTitle/cardTitleValue naming convention [#OCD-5295]
andlar
reviewed
Jun 1, 2026
andlar
left a comment
Contributor
There was a problem hiding this comment.
I know this is a draft, and I only got about 40% of the way through it while waiting on other things to happen, but wanted to put my comments out there while I was thinking of them
…search bar functionality [#OCD-5295]
… with card view [#OCD-5295]
…omponents for consistent layout across all search pages [#OCD-5295]
…rch result display across various views [#OCD-5295]
…raphy - Removed makeStyles and associated styles from multiple search view components. - Updated text elements to use Typography for better consistency and readability. - Simplified JSX structure by removing unnecessary divs and classes. - Ensured all components maintain functionality while enhancing visual presentation. [#OCD-5295]
…s search views [#OCD-5295]
…iews for enhanced certification status display [#OCD-5295]
…out in ChplSearchResultCard and ChplSearchResultControls [#OCD-5295]
…text handling [#OCD-5295]
… improve button padding in listing service [#OCD-5295]
…verse defaults in corrective action view [#OCD-5295]
improved layout and alignment on smaller screens [#OCD-5295]
…ntrols [#OCD-5295]
…rove styling [#OCD-5295]
…ed responsiveness and consistency [#OCD-5295]
…s for improved user experience [#OCD-5295]
…onsistency [#OCD-5295]
…layout consistency [#OCD-5295]
…proved layout consistency [#OCD-5295]
[#OCD-5295]
[#OCD-5295]
…eadability [#OCD-5295]
andlar
requested changes
Jun 26, 2026
andlar
reviewed
Jun 26, 2026
…ling for consistency [#OCD-5295]
…ers view layout [#OCD-5295]
andlar
reviewed
Jul 1, 2026
…update main content structure in various views [#OCD-5295]
andlar
reviewed
Jul 1, 2026
…dy; enhance flex styling in ChplSearchResultCard [#OCD-5295]
… for cleaner layout [#OCD-5295]
andlar
reviewed
Jul 2, 2026
andlar
previously approved these changes
Jul 2, 2026
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
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.
This pull request focuses on standardizing the use of the
paletteandthemeobjects from thethemesmodule for consistent styling across the application. It also introduces several UI and usability improvements, especially around filter panels and download buttons, and adds better support for responsive layouts and embedded views. Below are the most important changes grouped by theme:Theming and Styling Consistency
Replaced hardcoded color values with references to
paletteandthemethroughout multiple components, includingapi-key-registration.jsx,change-request.jsx,change-requests-view.jsx,developer-view.jsx,filter-chips.jsx, andfilter-panel.jsx, ensuring consistent color usage and easier theme management. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14]Updated filter panel and filter chips components to use
palettefor backgrounds and borders, and improved their responsive layout and z-index handling. [1] [2] [3] [4] [5] [6]Filter Panel and Search Bar Enhancements
Improved the filter panel's anchor and width calculations for better alignment with the search bar, changed the advanced search button color, and adjusted the panel's responsive behavior and border logic. [1] [2] [3] [4]
Enhanced the filter search bar's responsive styles to improve usability on small screens. [1] [2]
Download Listings and Button Group Improvements
Change Requests Component Updates
embeddedprop toChplChangeRequestsViewandChplChangeRequestsfor better control over layout and sticky behavior when used in different contexts. Updated prop types accordingly. [1] [2] [3] [4] [5] [6] [7] [8]Minor UI Tweaks
These changes collectively improve the application's maintainability, visual consistency, and user experience across devices and embedded contexts.