Skip to content

fix(ui5-side-navigation): prevent text squeezing during expand animation - #13774

Merged
s-todorova merged 16 commits into
mainfrom
sidenav_animation
Jul 23, 2026
Merged

fix(ui5-side-navigation): prevent text squeezing during expand animation#13774
s-todorova merged 16 commits into
mainfrom
sidenav_animation

Conversation

@s-todorova

Copy link
Copy Markdown
Contributor

fixes: #13546

@s-todorova
s-todorova requested a review from a team July 2, 2026 06:34
@s-todorova
s-todorova temporarily deployed to netlify-preview July 2, 2026 06:34 — with GitHub Actions Inactive
@sap-ui5-webcomponents-release

Copy link
Copy Markdown

Comment thread packages/fiori/src/themes/SideNavigationItemBase.css
@s-todorova
s-todorova temporarily deployed to netlify-preview July 6, 2026 19:07 — with GitHub Actions Inactive
@s-todorova
s-todorova temporarily deployed to netlify-preview July 7, 2026 06:29 — with GitHub Actions Inactive
@s-todorova
s-todorova temporarily deployed to netlify-preview July 8, 2026 09:53 — with GitHub Actions Inactive
@s-todorova
s-todorova temporarily deployed to netlify-preview July 8, 2026 10:46 — with GitHub Actions Inactive
@s-todorova
s-todorova temporarily deployed to netlify-preview July 8, 2026 12:59 — with GitHub Actions Inactive
@s-todorova
s-todorova temporarily deployed to netlify-preview July 8, 2026 13:09 — with GitHub Actions Inactive
@s-todorova
s-todorova temporarily deployed to netlify-preview July 13, 2026 06:49 — with GitHub Actions Inactive
@s-todorova
s-todorova requested a review from a team July 13, 2026 06:49

@TeodorTaushanov TeodorTaushanov left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AI review:

  • Add a fallback timeout (e.g., 500ms) to reset _bAnimating if transitionend doesn't fire
  • Reset _bAnimating = false in onExitDOM
  • Check getAnimationMode() !== AnimationMode.None before activating animation machinery

@s-todorova
s-todorova temporarily deployed to netlify-preview July 21, 2026 05:56 — with GitHub Actions Inactive
@s-todorova
s-todorova requested a review from a team July 21, 2026 05:57
@s-todorova
s-todorova temporarily deployed to netlify-preview July 21, 2026 08:18 — with GitHub Actions Inactive
@s-todorova
s-todorova temporarily deployed to netlify-preview July 22, 2026 11:14 — with GitHub Actions Inactive
@s-todorova
s-todorova temporarily deployed to netlify-preview July 22, 2026 11:17 — with GitHub Actions Inactive
@s-todorova
s-todorova temporarily deployed to netlify-preview July 23, 2026 07:41 — with GitHub Actions Inactive
@s-todorova
s-todorova temporarily deployed to netlify-preview July 23, 2026 12:28 — with GitHub Actions Inactive
@s-todorova
s-todorova merged commit 242aeec into main Jul 23, 2026
13 checks passed
@s-todorova
s-todorova deleted the sidenav_animation branch July 23, 2026 12:56
@s-todorova
s-todorova temporarily deployed to netlify-preview July 23, 2026 12:56 — with GitHub Actions Inactive
@sap-ui5-webcomponents-release

Copy link
Copy Markdown

🧹 Preview deployment cleaned up: https://pr-13774--ui5-webcomponents.netlify.app

@sap-ui5-webcomponents-release

Copy link
Copy Markdown

🎉 This PR is included in version v2.25.0 🎉

The release is available on v2.25.0

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ui5-side-navigation]: Visual glitch during the expand animation

2 participants