-
-
Notifications
You must be signed in to change notification settings - Fork 329
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
[Shape] Fix 3D flip starting zoom, fix nested shapes, ignore child elements with class 'active' #464
[Shape] Fix 3D flip starting zoom, fix nested shapes, ignore child elements with class 'active' #464
Conversation
…ggered, also centralized flip method to shorten code
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
🤔 Not completely satisfied with the transition between different sizes. |
Fixed the transition if shape sides differ in size.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
This PR fixes the following issues:
Whenever a shape had child elements which also had the
active
class set (either from another element likemenu
or a nested shape but not the "own"active side
), the whole animation was messed up (see screenshots/testcase)Therefore nested shape elements were not working
The Flip animations always started with an hard increased size to gain the 3d-Cube effect. But it still looked misleading instead of a smooth transition. Reason for this was, how shape dealt with translateZ. The fix also reduced the amount of needed code 🙂
Testcase
Fixed
http://jsfiddle.net/qtL0dkr3/
Broken
http://jsfiddle.net/mpkjave0/1/
Screenshot
Nested shapes
Flip Animation
Closes
Semantic-Org/Semantic-UI#4704
Semantic-Org/Semantic-UI#4699
Semantic-Org/Semantic-UI#5127