enact icon indicating copy to clipboard operation
enact copied to clipboard

WRQ-6229: Add config `scrollTargetOnDescendantsFocus` to container

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
  • [x] 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

The container needs additional config to define whether the container will be the scroll target when its child items get focused.

Resolution

Add private container config scrollTargetOnDescendantsFocus. Add private method getScrollTargetOnDescendantsFocus to find the scroll target container when the element is focused.

Additional Considerations

Links

WRQ-6299

Comments

juwonjeong avatar Feb 26 '24 05:02 juwonjeong

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.38%. Comparing base (d89daae) to head (338caf8). Report is 1 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3215      +/-   ##
===========================================
+ Coverage    82.29%   82.38%   +0.09%     
===========================================
  Files          155      155              
  Lines         7149     7231      +82     
  Branches      1885     1921      +36     
===========================================
+ Hits          5883     5957      +74     
- Misses         995     1002       +7     
- Partials       271      272       +1     

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

codecov[bot] avatar Feb 26 '24 05:02 codecov[bot]