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

Minor improvement to test coverage of CSS transition polyfill #246

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

necolas
Copy link
Contributor

@necolas necolas commented Dec 16, 2024

No description provided.

Copy link

workflow: benchmarks/size

Comparison of minified (terser) and compressed (brotli) size results, measured in bytes. Smaller is better.

Results Base Patch Ratio
react-strict-dom/dist/dom/index.js
· compressed 2,515 2,515 1.00
· minified 8,688 8,688 1.00
react-strict-dom/dist/dom/runtime.js
· compressed 855 855 1.00
· minified 2,435 2,435 1.00
react-strict-dom/dist/native/index.js
· compressed 15,824 15,824 1.00
· minified 59,086 59,086 1.00

Copy link

workflow: benchmarks/perf (native)

Comparison of performance test results, measured in operations per second. Larger is better.

Results Base Patch Ratio
css.create
· small 1,139,700 1,190,815 1.04 +
· small with units 442,606 448,546 1.01 +
· small with variables 674,068 689,604 1.02 +
· several small 329,804 329,424 1.00 -
· large 215,503 211,304 0.98 -
· large with polyfills 147,583 150,839 1.02 +
· complex 101,367 102,196 1.01 +
· unsupported 226,960 225,593 0.99 -
css.createTheme
· simple theme 225,494 225,537 1.00 +
· polyfill theme 214,785 213,303 0.99 -
css.props
· small 245,848 246,178 1.00 +
· small with units 192,428 192,381 1.00 -
· small with variables 106,706 106,637 1.00 -
· small with variables of units 76,136 75,534 0.99 -
· large 104,760 104,515 1.00 -
· large with polyfills 37,685 37,572 1.00 -
· complex 24,023 24,066 1.00 +
· unsupported 154,296 153,776 1.00 -
· simple merge 165,181 164,490 1.00 -
· wide merge 18,236 18,161 1.00 -
· deep merge 17,896 17,390 0.97 -
· themed merge 32,453 32,549 1.00 +

@necolas necolas merged commit 7d567a9 into main Dec 16, 2024
8 checks passed
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