Skip to content

refactor: convert menu.less to menu.module.css#1007

Merged
pawelgrimm merged 5 commits intomainfrom
pawel/refactor/less-to-modules
Feb 6, 2026
Merged

refactor: convert menu.less to menu.module.css#1007
pawelgrimm merged 5 commits intomainfrom
pawel/refactor/less-to-modules

Conversation

@pawelgrimm
Copy link
Contributor

@pawelgrimm pawelgrimm commented Feb 5, 2026

Short description

Migrate menu styling from Less to CSS modules, and convert story style files from Less to plain CSS.

  • Converts src/menu/menu.less to src/menu/menu.module.css
  • Updates story imports: .less.css
  • Keeps existing (global) class names to preserve behavior / avoid breaking changes.

Note that this is the only non-legacy, non-deprecated component that was using .less.

PR Checklist

  • Reviewed and approved Chromatic visual regression tests in CI

pawelgrimm and others added 2 commits February 5, 2026 16:39
@pawelgrimm pawelgrimm marked this pull request as draft February 5, 2026 23:51
@pawelgrimm pawelgrimm force-pushed the pawel/refactor/less-to-modules branch 2 times, most recently from 4efd65d to f9c8441 Compare February 6, 2026 21:15
@pawelgrimm pawelgrimm force-pushed the pawel/refactor/less-to-modules branch from 2b3c491 to 4f60671 Compare February 6, 2026 21:24
@pawelgrimm pawelgrimm self-assigned this Feb 6, 2026
@pawelgrimm pawelgrimm added the 👀 Show PR Used for PRs that need a review, but can be merged when CI is green. label Feb 6, 2026
@pawelgrimm pawelgrimm requested review from a team and craigcarlyle and removed request for a team February 6, 2026 21:34
@pawelgrimm pawelgrimm marked this pull request as ready for review February 6, 2026 21:35
@pawelgrimm pawelgrimm merged commit b63ce3a into main Feb 6, 2026
10 checks passed
@pawelgrimm pawelgrimm deleted the pawel/refactor/less-to-modules branch February 6, 2026 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

👀 Show PR Used for PRs that need a review, but can be merged when CI is green.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant