Skip to content

Refactor main css file#381

Merged
DinneK merged 5 commits into
devfrom
Refactor-main-CSS-file
May 9, 2025
Merged

Refactor main css file#381
DinneK merged 5 commits into
devfrom
Refactor-main-CSS-file

Conversation

@DinneK
Copy link
Copy Markdown
Contributor

@DinneK DinneK commented May 9, 2025

module-name: Tokenize FE

Problem

The existing CSS does not fully leverage USWDS components or design tokens. As a result styles are bloated, inconsistent and difficult to maintain.

Solution

  • Refactored CSS with tokens where applicable
  • Introduced custom tokens for consistency and reusability
  • Begin to organize and clean up CSS for clarity and maintainability

Result

No visual or functional changes to UI. The page looks and behaves the same, but CSS is now maintainable.

Test Plan

Check in browser

Notes and Next Steps

This is part I

  • Fix all tags

DinneK added 5 commits May 5, 2025 12:20
Signed-off-by: Dinne Kopelevich <dinne.kopelevich@gsa.gov>
Signed-off-by: Dinne Kopelevich <dinne.kopelevich@gsa.gov>
Signed-off-by: Dinne Kopelevich <dinne.kopelevich@gsa.gov>
Signed-off-by: Dinne Kopelevich <dinne.kopelevich@gsa.gov>
Signed-off-by: Dinne Kopelevich <dinne.kopelevich@gsa.gov>
@DinneK DinneK self-assigned this May 9, 2025
@DinneK DinneK added enhancement New feature or request front-end labels May 9, 2025
Comment thread app/src/css/tokens.css
Comment thread app/src/css/tokens.css
Comment thread app/src/css/tokens.css
Comment thread app/src/css/tokens.css
Copy link
Copy Markdown
Contributor

@decause-gov decause-gov left a comment

Choose a reason for hiding this comment

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

Excellent work! LGTM +1 🚢

Comment thread app/src/css/tokens.css
@DinneK DinneK merged commit 563a21f into dev May 9, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request front-end

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants