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.

Sign up for GitHub

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

Jump to bottom

[Accordion] expand/close animation is not smooth #2479

Closed
mvorisek opened this issue Oct 3, 2022 · 2 comments
Closed

[Accordion] expand/close animation is not smooth #2479

mvorisek opened this issue Oct 3, 2022 · 2 comments
Labels
type/usage Any support issues asking for help

Comments

@mvorisek
Copy link
Contributor

mvorisek commented Oct 3, 2022

Bug Report

Steps to reproduce

  1. open https://dev.agiletoolkit.org/demos/form/form-section.php
  2. navigate to Accordion Section in Form
  3. click Section 1 accordion title
  4. click it again and notice the close animation is very jerky

Expected result

both accordion section open/close must be reasonably smooth

Actual result

especially the close animation is very noticeably jerky

Version

2.8.8 and also the latest 2.9.0 beta
@mvorisek mvorisek added state/awaiting-investigation Anything which needs more investigation state/awaiting-triage Any issues or pull requests which haven't yet been triaged type/bug Any issue which is a bug or PR which fixes a bug labels Oct 3, 2022
@lubber-de
Copy link
Member

lubber-de commented Oct 3, 2022 •
edited

The forced min-height of your content node is causing this. You should either remove it or add the min-height style to your custom id node (first child of your content) instead to fix it

@lubber-de lubber-de added type/usage Any support issues asking for help and removed type/bug Any issue which is a bug or PR which fixes a bug state/awaiting-investigation Anything which needs more investigation state/awaiting-triage Any issues or pull requests which haven't yet been triaged labels Oct 3, 2022
@mvorisek mvorisek mentioned this issue Oct 3, 2022
Merged
@mvorisek
Copy link
Contributor Author

mvorisek commented Oct 3, 2022

thank you for the hint, super smooth then :)

@mvorisek mvorisek closed this as completed Oct 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type/usage Any support issues asking for help
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mvorisek @lubber-de