Skip to content

fix: allow withEditItems to work with lazyScroll#496

Merged
ImCoolNowRight merged 2 commits into
developfrom
fix/with-edit-items-lazy-scroll
Oct 30, 2024
Merged

fix: allow withEditItems to work with lazyScroll#496
ImCoolNowRight merged 2 commits into
developfrom
fix/with-edit-items-lazy-scroll

Conversation

@joshhowenstine

@joshhowenstine joshhowenstine commented Mar 27, 2024

Copy link
Copy Markdown
Contributor

Description

An issue was discovered where withEditItems does not integrate smoothly with lazyScroll. This update ensures that the scrolling functionality will pause until the initial item swap transition is complete, thereby preventing any conflict or unexpected behavior during additional logic execution.

Ticket

Testing Instructions

  1. Navigate to the withEditItems story.
  2. Scroll through the row, it should operate as expected.
  3. Press "Enter" to enable edit mode and move the current item. The functionality should remain seamless.
  4. Enable the lazyScroll option in Storybook.
  5. Repeat the above actions. After swapping items, the scrolling animation should now proceed as expected without issues.

@svc-lightning-ui-components

Copy link
Copy Markdown
Collaborator

Test Execution Passed.

1 similar comment
@svc-lightning-ui-components

Copy link
Copy Markdown
Collaborator

Test Execution Passed.

@ImCoolNowRight ImCoolNowRight left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great!

@ImCoolNowRight
ImCoolNowRight force-pushed the fix/with-edit-items-lazy-scroll branch from 22eb8ab to 6942063 Compare October 30, 2024 14:04
@ImCoolNowRight
ImCoolNowRight merged commit 0cf0522 into develop Oct 30, 2024
@ImCoolNowRight
ImCoolNowRight deleted the fix/with-edit-items-lazy-scroll branch October 30, 2024 14:17
svc-lightning-ui-components pushed a commit that referenced this pull request Oct 30, 2024
# [@lightningjs/ui-components-v2.25.0](https://github.com/rdkcentral/Lightning-UI-Components/compare/@lightningjs/ui-components-v2.24.0...@lightningjs/ui-components-v2.25.0) (2024-10-30)

### Bug Fixes

* allow withEditItems to work with lazyScroll ([#496](#496)) ([0cf0522](0cf0522))

### Features

* allow surface class radius to be overwritten by parent ([#548](#548)) ([3db0c5c](3db0c5c))
@svc-lightning-ui-components

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version @lightningjs/ui-components-v2.25.0 🎉

The release is available on:

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.

4 participants