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

Bug: onTransitionStart #26065

Closed
m1gu3l opened this issue Jan 27, 2023 · 5 comments · Fixed by #27345
Closed

Bug: onTransitionStart #26065

m1gu3l opened this issue Jan 27, 2023 · 5 comments · Fixed by #27345

Comments

@m1gu3l
Copy link

m1gu3l commented Jan 27, 2023

React version: any

Steps To Reproduce

Add onTransitionStart and onTransitionEnd handlers to element with styles that contain transition declaration

Link to code:

export function registerSimpleEvents() {

Link to docs:
https://developer.mozilla.org/en-US/docs/Web/API/Element/transitionstart_event

The current behavior

onTransitionEnd fires, onTransitionStart doesn't

The expected behavior

Both handlers should fire

@m1gu3l m1gu3l added the Status: Unconfirmed A potential issue that we haven't yet confirmed as a bug label Jan 27, 2023
@m1gu3l m1gu3l changed the title Bug: t Bug: onTransitionStart Jan 27, 2023
@eps1lon
Copy link
Collaborator

eps1lon commented Feb 7, 2023

It doesn't look like this bug report has enough info for one of us to reproduce it.

Please provide a CodeSandbox (https://react.new), or a link to a repository on GitHub.

Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve

@m1gu3l
Copy link
Author

m1gu3l commented Feb 7, 2023

@eps1lon
Copy link
Collaborator

eps1lon commented Feb 8, 2023

When we do this, we should also add cancel and run for consistency. Seems like these weren't added because they didn't exist when that particular code was written.

Not sure how much work is required to get this into the current synthetic event system. But then there are also plans to drop the synthetic event system at which point onTransitionStart will just start working automatically.

@gaearon
Copy link
Collaborator

gaearon commented Feb 24, 2023

Should be easy to add to the list, I think.

@pauldraper
Copy link

pauldraper commented Aug 7, 2023

This has been supported by every major browser since March 2020. https://caniuse.com/mdn-api_element_transitionstart_event

(Is this React thing still active?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants