Skip to content

remove height and with from rx video#5210

Merged
adhami3310 merged 1 commit intomainfrom
remove-height-and-with-from-rx-video
May 1, 2025
Merged

remove height and with from rx video#5210
adhami3310 merged 1 commit intomainfrom
remove-height-and-with-from-rx-video

Conversation

@adhami3310
Copy link
Copy Markdown
Member

fixes #2363

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Apr 30, 2025

CodSpeed Performance Report

Merging #5210 will improve performances by 6.21%

Comparing remove-height-and-with-from-rx-video (7b5b79c) with main (26f1a7b)

Summary

⚡ 1 improvements
✅ 11 untouched benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
test_compile_stateful[_simple_page] 256.5 µs 241.5 µs +6.21%

@adhami3310 adhami3310 merged commit 1a8d33d into main May 1, 2025
41 of 42 checks passed
@adhami3310 adhami3310 deleted the remove-height-and-with-from-rx-video branch May 1, 2025 05:53
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.

rx.video does not support responsiveness in its width and height properties

2 participants