backpack icon indicating copy to clipboard operation
backpack copied to clipboard

[BD-8573][BpkCarousel][BpkMobileScrollContainer] Resolve force reflow issues causing by carousel utils and scroll container

Open yifeng-ruan opened this issue 7 months ago • 4 comments

After upgrading bpk, performance issues related to force reflow have emerged, particularly with updates to the carousel component. To fix the drop in INP Score caused by reflow, I added IntersectionObserver monitoring and requestAnimationFrame to address the reflow caused by scroll and height/width calculations. image

Remember to include the following changes:

  • [x] Ensure the PR title includes the name of the component you are changing so it's clear in the release notes for consumers of the changes in the version e.g [KOA-123][BpkButton] Updating the colour
  • [ ] README.md (If you have created a new component)
  • [ ] Component README.md
  • [ ] Tests
  • [ ] Accessibility tests
    • The following checks were performed:
      • [ ] Ability to navigate using a keyboard only
      • [ ] Zoom functionality (Deque University explanation):
        • [ ] The page SHOULD be functional AND readable when only the text is magnified to 200% of its initial size
        • [ ] Pages must reflow as zoom increases up to 400% so that content continues to be presented in only one column i.e. Content MUST NOT require scrolling in two directions (both vertically and horizontally)
      • [ ] Ability to navigate using a screen reader only
  • [ ] Storybook examples created/updated
  • [ ] For breaking changes or deprecating components/properties, migration guides added to the description of the PR. If the guide has large changes, consider creating a new Markdown page inside the component's docs folder and link it here

yifeng-ruan avatar Jul 02 '24 09:07 yifeng-ruan