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.

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

fix(carousel): Ensure correct autoplay display and animation #11338

Merged
merged 16 commits into from
Jan 23, 2025

Conversation

macandcheese
Copy link
Contributor

@macandcheese macandcheese commented Jan 18, 2025

Related Issue: #11317

Summary

Ensures the animation is correctly applied when autoplay advances through Carousel Items.
Also updates the Progress component to adhere to the new value format / breaking change for 3.x: #10622 / logged in internal tracking discussion

@github-actions github-actions bot added the bug Bug reports for broken functionality. Issues should include a reproduction of the bug. label Jan 18, 2025
@macandcheese macandcheese added the pr ready for visual snapshots Adding this label will run visual snapshot testing. label Jan 18, 2025
Copy link
Member

@driskull driskull left a comment

Choose a reason for hiding this comment

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

👍

@driskull
Copy link
Member

Is there a test we could add to ensure this is working properly?

@macandcheese
Copy link
Contributor Author

Is there a test we could add to ensure this is working properly?

Added a variant of an existing test to cover the autoplay case. I used a custom, shorter autoplay-duration for the autoplay tests - which should speed the Carousel suite up quite a bit.

@macandcheese macandcheese merged commit a5626fa into dev Jan 23, 2025
9 checks passed
@macandcheese macandcheese deleted the macandcheese/11317-carousel-autoplay-animation branch January 23, 2025 17:57
benelan added a commit that referenced this pull request Jan 24, 2025
* origin/dev: (34 commits)
  build: update browserslist db (#11339)
  build(deps): update dependency lint-staged to v15.4.1 (#11343)
  chore: release next
  feat(graph): add component tokens (#11355)
  chore: release next
  fix(popover, tooltip): drop relative-positioning to reduce risk of clipping (#11373)
  chore: release next
  fix(date-picker): no longer disable min/max value month in select menu (#11350)
  chore(icon): improve icon load error message (#11367)
  fix(date-picker): remove outline for header actions (#11369)
  chore: release next
  fix(carousel): Ensure correct `autoplay` display and animation (#11338)
  chore: release next
  fix(flow): process items on loaded (#11364)
  chore: release next
  fix(combobox, stepper, table): respect user hidden attribute (#10983)
  refactor(action-pad): restore rounded styling (#11358)
  test(combobox): avoid emitting change event for value property update (#11281)
  chore: release next
  refactor(action-pad): remove unnecessary overflow css styling. (#11349)
  ...
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