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

chore: refactor at rules to be a separate param in functions #556

Merged
merged 1 commit into from
May 16, 2024

Conversation

layershifter
Copy link
Member

@layershifter layershifter commented May 16, 2024

Performs internals refactor to reduce number of arguments in resolveStyleRules().

Copy link

github-actions bot commented May 16, 2024

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
core
makeStyles (runtime)
25.905 kB
9.238 kB
25.85 kB
9.242 kB
-55 B
4 B
react
makeStyles (runtime)
28.41 kB
10.227 kB
28.355 kB
10.228 kB
-55 B
1 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
core
__resetStyles (makeResetStyles)
273 B
197 B
core
__styles (makeStyles)
1.727 kB
832 B
core
makeResetStyles (runtime)
17.14 kB
6.569 kB
core
mergeClasses
1.865 kB
900 B
core
shorthands.padding()
4.781 kB
1.537 kB
react
__css
1.723 kB
812 B
react
__styles
4.187 kB
1.832 kB
react
makeResetStyles (runtime)
19.64 kB
7.562 kB
react
makeStaticStyles (runtime)
9.576 kB
4.182 kB
shadow-dom
createShadowDOMRenderer
4.085 kB
1.724 kB
🤖 This report was generated against e3bf466e36464585ecff0ca3492760f6ada76dcc

@layershifter layershifter marked this pull request as ready for review May 16, 2024 12:51
@layershifter layershifter requested a review from a team as a code owner May 16, 2024 12:51
@layershifter layershifter merged commit 3cac9f0 into microsoft:main May 16, 2024
4 checks passed
@layershifter layershifter deleted the feat/class-hash-salt branch May 16, 2024 13:03
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.

None yet

2 participants