Skip to content

Remove use of Animated.delay and sequence for CSS transition polyfill#334

Merged
necolas merged 1 commit into
mainfrom
native/remove-animated-delay-sequence
Jul 8, 2025
Merged

Remove use of Animated.delay and sequence for CSS transition polyfill#334
necolas merged 1 commit into
mainfrom
native/remove-animated-delay-sequence

Conversation

@necolas

@necolas necolas commented Jul 8, 2025

Copy link
Copy Markdown
Contributor

We can use the delay field in the config for timing and spring rather than using Animated.delay and Animated.sequence. This should help with animation sequencing and avoid depending on onAnimationComplete events firing.

We can use the `delay` field in the config for `timing` and `spring`
rather than using `Animated.delay` and `Animated.sequence`. This should
help with animation sequencing and avoid depending on
`onAnimationComplete` events firing.
@github-actions

github-actions Bot commented Jul 8, 2025

Copy link
Copy Markdown

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,514 2,514 1.00
· minified 8,691 8,691 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 16,022 16,065 1.00 +
· minified 59,923 59,912 1.00 -

@github-actions

github-actions Bot commented Jul 8, 2025

Copy link
Copy Markdown

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,142,718 1,133,708 0.99 -
· small with units 427,033 426,159 1.00 -
· small with variables 665,158 660,011 0.99 -
· several small 317,556 317,950 1.00 +
· large 205,287 203,844 0.99 -
· large with polyfills 144,961 144,983 1.00 +
· complex 99,930 99,384 0.99 -
· unsupported 216,336 217,777 1.01 +
css.createTheme
· simple theme 233,299 232,213 1.00 -
· polyfill theme 218,740 218,242 1.00 -
css.props
· small 292,999 292,988 1.00 -
· small with units 221,640 222,379 1.00 +
· small with variables 115,256 113,990 0.99 -
· small with variables of units 78,897 78,893 1.00 -
· large 126,721 126,520 1.00 -
· large with polyfills 41,993 41,757 0.99 -
· complex 25,416 25,446 1.00 +
· unsupported 181,291 181,074 1.00 -
· simple merge 185,054 184,978 1.00 -
· wide merge 19,171 19,419 1.01 +
· deep merge 18,766 19,047 1.01 +
· themed merge 35,930 36,505 1.02 +

@necolas necolas merged commit 55aa8d2 into main Jul 8, 2025
8 checks passed
@necolas necolas deleted the native/remove-animated-delay-sequence branch July 25, 2025 21:08
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.

3 participants