-
Notifications
You must be signed in to change notification settings - Fork 905
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
feat: replaced overflow hidden with clip in styles panel (#4203) #4914
base: main
Are you sure you want to change the base?
feat: replaced overflow hidden with clip in styles panel (#4203) #4914
Conversation
fda715b
to
ccf30e1
Compare
Super, last 2 weeks I had 2 serious issues with hidden vs clip!!! |
The only issue is the builder and clipped content. |
@istarkov ohh, we actually have a use case with the slider where we use overflow hidden and scroll to the right image from the navigator thanks to overflow hidden. |
Same usecase ruined some animations if hidden is set ;) |
What if only in the builder if element is selected we temporary replace clip on hidden for all ancestors. In case of no animations are in progress. |
will probably sooner or later lead to a situation where something is broken in that particular state, e.g. if something relies on clip but hidden is used instead |
Description
closes ( #4203 )
Steps for reproduction
Code Review
Before requesting a review
Before merging
.env
file