Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(button): convert to css modules #1030

Merged
merged 4 commits into from
Oct 6, 2023

Conversation

Niznikr
Copy link
Contributor

@Niznikr Niznikr commented Oct 6, 2023

Summary

We need to tweak some variables button uses for the refresh. So to allow us to ff those styles we need to convert them to css modules.

@Niznikr Niznikr requested review from a team, apucacao, lwonsower and tvarney13 October 6, 2023 13:35
@changeset-bot
Copy link

changeset-bot bot commented Oct 6, 2023

🦋 Changeset detected

Latest commit: 32efa64

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 19 packages
Name Type
@launchpad-ui/split-button Patch
@launchpad-ui/inline-edit Patch
@launchpad-ui/snackbar Patch
@launchpad-ui/button Patch
@launchpad-ui/alert Patch
@launchpad-ui/form Patch
@launchpad-ui/core Patch
@launchpad-ui/banner Patch
@launchpad-ui/collapsible Patch
@launchpad-ui/drawer Patch
@launchpad-ui/dropdown Patch
@launchpad-ui/filter Patch
@launchpad-ui/modal Patch
@launchpad-ui/pagination Patch
@launchpad-ui/select Patch
@launchpad-ui/tag Patch
@launchpad-ui/card Patch
@launchpad-ui/menu Patch
@launchpad-ui/navigation Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

margin-left: -0.1rem;
}

.SplitButtonDrop:global([class*='_Button--small_']) {
Copy link
Contributor

Choose a reason for hiding this comment

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

What are those selectors with underscores?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Targeting parent selectors whose class names get hashed from css modules

@github-actions
Copy link
Contributor

github-actions bot commented Oct 6, 2023

Size Change: +781 B (0%)

Total Size: 168 kB

Filename Size Change
packages/alert/dist/index.es.js 1.37 kB +1 B (0%)
packages/alert/dist/style.css 1.62 kB +19 B (+1%)
packages/button/dist/index.es.js 1.87 kB +242 B (+15%) ⚠️
packages/button/dist/index.js 1.95 kB +242 B (+14%) ⚠️
packages/button/dist/style.css 3.81 kB +93 B (+3%)
packages/form/dist/style.css 2.76 kB +6 B (0%)
packages/inline-edit/dist/style.css 352 B +14 B (+4%)
packages/snackbar/dist/index.es.js 1.18 kB -1 B (0%)
packages/snackbar/dist/index.js 1.73 kB -2 B (0%)
packages/snackbar/dist/style.css 597 B +17 B (+3%)
packages/split-button/dist/index.es.js 944 B +57 B (+6%) 🔍
packages/split-button/dist/index.js 1.01 kB +54 B (+6%) 🔍
packages/split-button/dist/style.css 534 B +39 B (+8%) 🔍
ℹ️ View Unchanged
Filename Size
packages/alert/dist/index.js 1.44 kB
packages/avatar/dist/index.es.js 1.16 kB
packages/avatar/dist/index.js 1.23 kB
packages/avatar/dist/style.css 466 B
packages/banner/dist/index.es.js 644 B
packages/banner/dist/index.js 714 B
packages/banner/dist/style.css 545 B
packages/card/dist/index.es.js 706 B
packages/card/dist/index.js 775 B
packages/card/dist/style.css 754 B
packages/chip/dist/index.es.js 679 B
packages/chip/dist/index.js 749 B
packages/chip/dist/style.css 895 B
packages/clipboard/dist/index.es.js 1.51 kB
packages/clipboard/dist/index.js 1.59 kB
packages/clipboard/dist/style.css 829 B
packages/collapsible/dist/index.es.js 856 B
packages/collapsible/dist/index.js 926 B
packages/collapsible/dist/style.css 94 B
packages/columns/dist/index.es.js 619 B
packages/columns/dist/index.js 692 B
packages/columns/dist/style.css 354 B
packages/core/dist/index.es.js 1.13 kB
packages/core/dist/index.js 1.51 kB
packages/counter/dist/index.es.js 333 B
packages/counter/dist/index.js 396 B
packages/counter/dist/style.css 256 B
packages/data-table/dist/index.es.js 2.47 kB
packages/data-table/dist/index.js 2.53 kB
packages/data-table/dist/style.css 385 B
packages/drawer/dist/index.es.js 1.73 kB
packages/drawer/dist/index.js 2.29 kB
packages/drawer/dist/style.css 570 B
packages/dropdown/dist/index.es.js 1.15 kB
packages/dropdown/dist/index.js 1.21 kB
packages/filter/dist/index.es.js 2.3 kB
packages/filter/dist/index.js 2.38 kB
packages/filter/dist/style.css 1 kB
packages/focus-trap/dist/index.es.js 270 B
packages/focus-trap/dist/index.js 333 B
packages/form/dist/index.es.js 4.23 kB
packages/form/dist/index.js 4.34 kB
packages/icons/dist/index.es.js 1.35 kB
packages/icons/dist/index.js 1.42 kB
packages/icons/dist/style.css 611 B
packages/inline-edit/dist/index.es.js 1.58 kB
packages/inline-edit/dist/index.js 1.66 kB
packages/inline/dist/index.es.js 565 B
packages/inline/dist/index.js 637 B
packages/inline/dist/style.css 299 B
packages/markdown/dist/index.es.js 960 B
packages/markdown/dist/index.js 1.03 kB
packages/markdown/dist/style.css 234 B
packages/menu/dist/index.es.js 3.55 kB
packages/menu/dist/index.js 3.64 kB
packages/menu/dist/style.css 1.22 kB
packages/modal/dist/index.es.js 3.03 kB
packages/modal/dist/index.js 3.59 kB
packages/modal/dist/style.css 1.03 kB
packages/navigation/dist/index.es.js 2.79 kB
packages/navigation/dist/index.js 2.86 kB
packages/navigation/dist/style.css 1.25 kB
packages/overlay/dist/index.es.js 1 kB
packages/overlay/dist/index.js 1.06 kB
packages/pagination/dist/index.es.js 1.17 kB
packages/pagination/dist/index.js 1.24 kB
packages/pagination/dist/style.css 356 B
packages/popover/dist/index.es.js 3.07 kB
packages/popover/dist/index.js 3.58 kB
packages/popover/dist/style.css 629 B
packages/portal/dist/index.es.js 393 B
packages/portal/dist/index.js 453 B
packages/progress-bubbles/dist/index.es.js 1.76 kB
packages/progress-bubbles/dist/index.js 1.83 kB
packages/progress-bubbles/dist/style.css 961 B
packages/progress/dist/index.es.js 1.02 kB
packages/progress/dist/index.js 1.09 kB
packages/progress/dist/style.css 278 B
packages/select/dist/index.es.js 5.91 kB
packages/select/dist/index.js 6 kB
packages/select/dist/style.css 1.34 kB
packages/slider/dist/index.es.js 579 B
packages/slider/dist/index.js 644 B
packages/slider/dist/style.css 672 B
packages/stack/dist/index.es.js 494 B
packages/stack/dist/index.js 565 B
packages/stack/dist/style.css 226 B
packages/tab-list/dist/index.es.js 737 B
packages/tab-list/dist/index.js 809 B
packages/tab-list/dist/style.css 455 B
packages/table/dist/index.es.js 1.02 kB
packages/table/dist/index.js 1.1 kB
packages/table/dist/style.css 905 B
packages/tag/dist/index.es.js 2.85 kB
packages/tag/dist/index.js 2.92 kB
packages/tag/dist/style.css 945 B
packages/toast/dist/index.es.js 979 B
packages/toast/dist/index.js 1.53 kB
packages/toast/dist/style.css 544 B
packages/toggle/dist/index.es.js 764 B
packages/toggle/dist/index.js 844 B
packages/toggle/dist/style.css 1.52 kB
packages/tokens/dist/index.css 620 B
packages/tokens/dist/index.es.js 2.17 kB
packages/tokens/dist/index.js 2.18 kB
packages/tokens/dist/media-queries.css 112 B
packages/tokens/dist/themes.css 2.2 kB
packages/tooltip/dist/index.es.js 517 B
packages/tooltip/dist/index.js 591 B
packages/tooltip/dist/style.css 366 B
packages/vars/dist/index.es.js 1.65 kB
packages/vars/dist/index.js 1.72 kB

compressed-size-action

@Niznikr Niznikr merged commit e48d0df into release/color-refresh Oct 6, 2023
@Niznikr Niznikr deleted the refactor/button-css-modules branch October 6, 2023 14:37
Niznikr added a commit that referenced this pull request Nov 17, 2023
* chore: enter prerelease mode

* feat(tokens)!: move color primitives to themes.css (#1020)

* feat(tokens)!: move color primitives to themes.css

* feat(tokens): build with TS

* feat(tokens): generate types

* feat(tokens): transform dark values

* chore: add changeset

* fix: update value to hsl

* chore: fix lockfile

* refactor(button): convert to css modules (#1030)

* refactor(button): convert to css modules

* chore: add changeset

* fix: disabled icon

* fix: test

* refactor(tokens)!: update color primitives and aliases for product refresh (#1029)

* feat(tokens)!: update color primitives

* feat(tokens): update color aliases

* chore: add changeset

* fix: add opacity to aliases

* feat: test wcag 2.2

* feat(tokens): add new aliases

* refactor: update button styles

* chore: update tokens story

* refactor: simplify contract

* chore: merge branch 'main' into release/color-refresh (#1036)

* chore(deps): update all non-major dependencies (#1032)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* refactor!: use base 16 font size (#1033)

* refactor: use base 16 font size

* chore: add changeset

* fix: ve styles

* fix: inline style

* chore: version packages (#1034)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* chore(deps): update nrwl/nx-set-shas action to v4 (#1035)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: update initialVersions

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: pd-sa-github-launchpad-ui <[email protected]>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat(tokens): add typography tokens (#1037)

* feat(tokens): add typography tokens

* refactor: use vars

* chore: add changeset

* feat: add display

* chore: version packages (alpha) (#1028)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: refresh menu component (#1043)

* feat: refresh menu styles

* chore: use medium radius on popovers

* chore: update menu stories

* chore: add changeset

* fix: update custom css so tests don't fail

* refactor(menu): use CSS modules (#1046)

* refactor(menu): use css modules

* chore: add changeset

* chore: fix classes

* chore: version packages (alpha) (#1045)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* chore: merge branch 'main' into release/color-refresh (#1049)

* chore(deps): update all non-major dependencies (#1032)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* refactor!: use base 16 font size (#1033)

* refactor: use base 16 font size

* chore: add changeset

* fix: ve styles

* fix: inline style

* chore: version packages (#1034)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* chore(deps): update nrwl/nx-set-shas action to v4 (#1035)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update all non-major dependencies (#1039)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency lint-staged to v15 (#1040)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* feat(tokens, icons, alert): add new tokens and icons (#1038)

* feat(tokens, icons): add new tokens and icons

* feat(tokens, icons): update token names and stories, update icon svgs

* feat(tokens, icons): update changelog

* feat(icons): update css to use gradient tokens

* feat(alert): update CSS to use new gradient tokens

* Update packages/tokens/src/color-aliases.yaml

Co-authored-by: Robert Niznik <[email protected]>

---------

Co-authored-by: Robert Niznik <[email protected]>

* chore: version packages (#1041)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat(tokens): add primitive size tokens (#1031)

* feat: add primitive size tokens

* fix: use size variable tokens in storybook

* fix: add whitespace at bottom of size token file

* chore: update sizes to use base 16

* chore: use sizing tokens in borders

* fix: add 3 and 1200 sizes

* chore: update fonts to use size tokens

* chore: use size tokens in spacing

* chore: se size tokens in viewport

* fix: space between constants in size story

* chore: add changeset

* fix: make slice const name more explicit

* fix: use size 1 for border 200

Co-authored-by: Robert Niznik <[email protected]>

---------

Co-authored-by: Robert Niznik <[email protected]>

* feat: add building and door icons (#1048)

* feat: adds building icons

* feat: add door-exit icon

* chore: add changeset

* chore: version packages (#1042)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* chore: fix token build

* chore: update initialVersions

* chore: fix size story

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: pd-sa-github-launchpad-ui <[email protected]>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Valerie Roske <[email protected]>
Co-authored-by: Matthew Ferry <[email protected]>

* fix: correct tokens version (#1051)

* chore: version packages (alpha) (#1050)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* fix(tokens): keep css color keywords (#1060)

* fix(tokens): keep css color keywords

* chore: bump ts-node

* chore: add changeset

* feat: Update color steps from 9 to 12 (#1068)

* feat: add 3 new grays and black and white to 12

* feat: purple remap from 9 to 12 steps

* fix: update flair base dark color

* feat: pink remap from 9 to 12 steps

* feat: cyan remap from 9 to 12 steps

* feat: yellow remap from 9 to 12 steps

* fix: adjust yellow 300 chroma

* feat: blue remap from 9 to 12 steps

* fix: reorder white and black tokens in ascending order

* feat: red map 9 to 12 steps

* feat: green map 9 to 12 steps

* feat: yellow map 9 to 12 steps

* fix: a11y tertiary text in dark

* fix: a11y button hover state

* fix: update chip tokens for a11y

* fix: update fills for info and error in dark

* chore: add changeset

* chore: merge branch 'main' into release/color-refresh (#1073)

* chore(deps): update all non-major dependencies (#1032)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* refactor!: use base 16 font size (#1033)

* refactor: use base 16 font size

* chore: add changeset

* fix: ve styles

* fix: inline style

* chore: version packages (#1034)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* chore(deps): update nrwl/nx-set-shas action to v4 (#1035)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update all non-major dependencies (#1039)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency lint-staged to v15 (#1040)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* feat(tokens, icons, alert): add new tokens and icons (#1038)

* feat(tokens, icons): add new tokens and icons

* feat(tokens, icons): update token names and stories, update icon svgs

* feat(tokens, icons): update changelog

* feat(icons): update css to use gradient tokens

* feat(alert): update CSS to use new gradient tokens

* Update packages/tokens/src/color-aliases.yaml

Co-authored-by: Robert Niznik <[email protected]>

---------

Co-authored-by: Robert Niznik <[email protected]>

* chore: version packages (#1041)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat(tokens): add primitive size tokens (#1031)

* feat: add primitive size tokens

* fix: use size variable tokens in storybook

* fix: add whitespace at bottom of size token file

* chore: update sizes to use base 16

* chore: use sizing tokens in borders

* fix: add 3 and 1200 sizes

* chore: update fonts to use size tokens

* chore: use size tokens in spacing

* chore: se size tokens in viewport

* fix: space between constants in size story

* chore: add changeset

* fix: make slice const name more explicit

* fix: use size 1 for border 200

Co-authored-by: Robert Niznik <[email protected]>

---------

Co-authored-by: Robert Niznik <[email protected]>

* feat: add building and door icons (#1048)

* feat: adds building icons

* feat: add door-exit icon

* chore: add changeset

* chore: version packages (#1042)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* chore(deps): update commitlint monorepo to v18 (#1053)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency nx to v17 (#1054)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(deps): update all non-major dependencies (#1052)

* fix(deps): update all non-major dependencies

* chore: add changeset

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Robb Niznik <[email protected]>

* chore(deps): lock file maintenance (#1056)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update actions/setup-node action to v4 (#1058)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency node to v20 (#1059)

* chore(deps): update dependency node to v20

* ci: use node 20

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Robb Niznik <[email protected]>

* docs: add getting started page to SB (#1062)

* docs: add getting started page to SB

* chore: clean up url

* feat(icons): org settings icons and updates (#1063)

* feat: add new org settings icons

* chore: add changeset

* fix: add person-off back

* chore: version packages (#1055)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* chore(deps): update all non-major dependencies (#1065)

* chore(deps): update all non-major dependencies

* fix: pin style-dictionary

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Robb Niznik <[email protected]>

* chore(deps): lock file maintenance (#1069)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* feat: add diamond and diamond-outline icons (#1070)

* chore: version packages (#1071)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* chore(deps): update dependency html-react-parser to v5 (#1072)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: update initialVersions

* fix: adjust build script

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: pd-sa-github-launchpad-ui <[email protected]>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Valerie Roske <[email protected]>
Co-authored-by: Matthew Ferry <[email protected]>

* chore: version packages (alpha) (#1061)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* chore: move primitives back to index.css

* chore: exit prerelease

* fix: update box colors

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: pd-sa-github-launchpad-ui <[email protected]>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Matthew Ferry <[email protected]>
Co-authored-by: Valerie Roske <[email protected]>
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.

2 participants