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

fix: #1888 drawer now animates out #1988

Merged
merged 3 commits into from
Jun 20, 2023
Merged

Conversation

megagames-me
Copy link
Contributor

Fixes bug #1888, drawer-overlay's dimensions no longer collapse and drawer-side no longer disappears when the drawer is closed.

Screen.Recording.2023-06-12.at.10.03.02.PM.mov

@saadeghi saadeghi self-assigned this Jun 12, 2023
@megagames-me
Copy link
Contributor Author

I added a slight offset to the drawer-content like v2 had. I can remove this if needed though.

@saadeghi
Copy link
Owner

saadeghi commented Jun 12, 2023

I added a slight offset to the drawer-content like v2 had. I can remove this if needed though.

Yes please remove that translate because that causes other issues

If anyone wants that small effect, they can have it by adding a few class names like

transition-transform translate-x-0 [.drawer-toggle:checked~&]:translate-x-2

@megagames-me
Copy link
Contributor Author

Yes please remove that translate because that causes other issues

Ok, removed. Can you make sure the transition timings are correct though? They look a little off to me when it closes. I can change them if needed.

@saadeghi saadeghi merged commit b5b55c5 into saadeghi:master Jun 20, 2023
@saadeghi
Copy link
Owner

Thank you

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.

2 participants