Skip to content

fix(carousel): animate items with the same direction#9335

Merged
benelan merged 2 commits intomainfrom
benelan/cherry-pick-carousel-hotfix
May 14, 2024
Merged

fix(carousel): animate items with the same direction#9335
benelan merged 2 commits intomainfrom
benelan/cherry-pick-carousel-hotfix

Conversation

@benelan
Copy link
Copy Markdown
Contributor

@benelan benelan commented May 14, 2024

Related Issue: #9232

Summary

Updates carousel to wait for item animations before each slide.

Note: I'll submit a follow-up PR to refactor openCloseComponent.onToggleOpenCloseComponent to use the same DOM util as carousel.

**Related Issue:** #9232

## Summary

Updates carousel to wait for item animations before each slide.

**Note**: I'll submit a follow-up PR to refactor
[`openCloseComponent.onToggleOpenCloseComponent`](https://github.com/Esri/calcite-design-system/blob/main/packages/calcite-components/src/utils/openCloseComponent.ts#L91)
to use the same DOM util as carousel.
@benelan benelan requested a review from a team as a code owner May 14, 2024 19:31
@github-actions github-actions Bot added the bug Bug reports for broken functionality. Issues should include a reproduction of the bug. label May 14, 2024
@benelan benelan changed the title fix(carousel): animate items with the same direction (#9325) fix(carousel): animate items with the same direction May 14, 2024
@benelan benelan added the pr ready for visual snapshots Adding this label will run visual snapshot testing. label May 14, 2024
@benelan benelan merged commit af43d15 into main May 14, 2024
@benelan benelan deleted the benelan/cherry-pick-carousel-hotfix branch May 14, 2024 21:52
benelan added a commit that referenced this pull request May 14, 2024
…fix-release

* origin/main:
  ci: setup hotfix releasing scripts and settings (#9337)
  fix(carousel): animate items with the same direction (#9335)
  chore: release next
  build(deps): update dependency @esri/calcite-ui-icons to v3.28.1 (#9336)
  fix: add type-fest as dependency due to build error (#9334)
  chore: release next
  fix(block): update text hierarchy and spacing (#9329)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bug reports for broken functionality. Issues should include a reproduction of the bug. pr ready for visual snapshots Adding this label will run visual snapshot testing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants