Skip to content

Commit

Permalink
Merge pull request #5720 from nextcloud-libraries/fix/NcAppSidebar--r…
Browse files Browse the repository at this point in the history
…emove-toggle-animation-on-appear

fix(NcAppSidebar): apply toggle offset transition only on sidebar transition
  • Loading branch information
ShGKme committed Jun 19, 2024
2 parents 0365599 + d5a2a7a commit 6dcce53
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions src/components/NcAppSidebar/NcAppSidebar.vue
Original file line number Diff line number Diff line change
Expand Up @@ -1126,8 +1126,15 @@ export default {
--app-sidebar-padding: #{$app-navigation-padding};
// A padding between the toggle button and the page border
--app-sidebar-offset: 0;
transition-duration: var(--animation-quick);
transition-property: --app-sidebar-offset;
// Explicitly disable transition by default to enable it only when sidebar animation is active
// !important to override styles from an older version, because it's global non-scoped styles
transition: --app-sidebar-offset 0ms !important;
}
// When AppSidebar is animation is active - also apply transition for the toggle button offset
.content:has(.app-sidebar.slide-right-enter-active),
.content:has(.app-sidebar.slide-right-leave-active) {
transition: --app-sidebar-offset var(--animation-quick);
}
.content:has(.app-sidebar__toggle) {
Expand Down

0 comments on commit 6dcce53

Please sign in to comment.