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

Transition no longer maintaining class specified in enterTo at end of transition. #2991

Closed
tomelliot opened this issue Feb 16, 2024 · 5 comments · Fixed by #3074
Closed

Transition no longer maintaining class specified in enterTo at end of transition. #2991

tomelliot opened this issue Feb 16, 2024 · 5 comments · Fixed by #3074

Comments

@tomelliot
Copy link

tomelliot commented Feb 16, 2024

What package within Headless UI are you using?
@headlessui/react

What version of that package are you using?

v1.7.18

What browser are you using?

Chrome Version 121.0.6167.160 (Official Build) (arm64)

Reproduction URL

repro

Describe your issue

A <Transition> that would previously fade out and stay at opacity-0 no longer stays at opacity-0 at the end of the transition. After fading out it pops back to full visibility (the opacity-0 class is removed).

        <Transition
          appear={true}
          show={true}
          enter="transition-opacity ease-in-out duration-1000"
          enterFrom="opacity-100"
          enterTo="opacity-0"
          aria-hidden="true"
        >
          <div>
            <CheckCircleIcon className={clsx("h-5 w-5 text-green-500")} />
          </div>
        </Transition>
@mrajaei76
Copy link

i have the same problem after upgrading to version 1.7.18

@noynek
Copy link

noynek commented Mar 12, 2024

I've got the same problem using 2.0.0-alpha.4 which is required for Catalyst.

@areriklq
Copy link

This seems similar to this issue (?): #2914
According to my bisect it broke in v1.7.18 (which seems to echo others here)

@RobinMalfait
Copy link
Member

Hey!

This should be fixed by #3074, and will be available in the next release.

You can already try it using:

  • npm install @headlessui/react@insiders.

@areriklq
Copy link

Hi @RobinMalfait ,

Sorry for the late response, but I just now tried to update to 1.7.19 and it does not solve the transition regression that was introduced in 1.7.18 (but I don't see it mentioned in the release notes either so it might be expected?).

However, v2.0.2 does solve the issue I was seeing. ✅ Thanks!

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 a pull request may close this issue.

5 participants