enact icon indicating copy to clipboard operation
enact copied to clipboard

WRQ-342: Implement useMarqueeController

Open juwonjeong opened this issue 1 year ago • 1 comments

Enact-DCO-1.0-Signed-off-by: Juwon Jeong ([email protected])

Checklist

  • [x] I have read and understand the contribution guide
  • [ ] A CHANGELOG entry is included
  • [ ] At least one test case is included for this feature or bug fix
  • [x] Documentation was added or is not needed
  • [ ] This is an API breaking change

Issue Resolved / Feature Added

Refactor MarqueeController to be hook-based implementation.

Resolution

Add useMarqueeController custom hook that returns the MarqueeControllerContext provider and handlers related to Marquee synchronization.

Additional Considerations

Links

WRQ-342

Comments

juwonjeong avatar Feb 21 '24 01:02 juwonjeong

Codecov Report

Attention: Patch coverage is 89.89899% with 10 lines in your changes are missing coverage. Please review.

Project coverage is 82.36%. Comparing base (441bfab) to head (296540b).

Files Patch % Lines
packages/ui/Marquee/useMarqueeController.js 89.24% 7 Missing and 3 partials :warning:
Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #3211   +/-   ##
========================================
  Coverage    82.36%   82.36%           
========================================
  Files          155      156    +1     
  Lines         7208     7209    +1     
  Branches      1907     1907           
========================================
+ Hits          5937     5938    +1     
  Misses         999      999           
  Partials       272      272           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Feb 21 '24 01:02 codecov[bot]