Skip to content

Offload styles from main application stylesheet#10285

Merged
aduth merged 8 commits intomainfrom
aduth-reduce-stylesheet
Mar 25, 2024
Merged

Offload styles from main application stylesheet#10285
aduth merged 8 commits intomainfrom
aduth-reduce-stylesheet

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Mar 22, 2024

🛠 Summary of changes

Removes several items from the main application stylesheet to reduce its size:

  1. Remove unused usa-step-indicator styles
    • IdV step indicator is fully custom and does not use design system styles
  2. Split account layout navigation styles to separate stylesheet
  3. Move usa-collection styles to document capture as the only place it's used

Performance Impact:

NODE_ENV=production yarn build:css && brotli-size app/assets/builds/application.css

Before: 22.7 kB
After: 21.6 kB
Diff: -1.1 kB (-4.8%)

📜 Testing Plan

Verify no regressions in affected styles, particularly account page header and navigation, and address search functionality for in-person proofing during document capture.

@aduth aduth merged commit 977a5de into main Mar 25, 2024
@aduth aduth deleted the aduth-reduce-stylesheet branch March 25, 2024 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants