IgniteUI / igniteui-angular / 39801780696

94%

push

github

fix(focus-trap): skip non-tabbable elements, clean up listener, reach 100% coverage (#17777)

* fix(focus-trap): skip non-tabbable elements and clean up listener

- Skip hidden, invisible, inert, negative tabindex and fieldset-disabled
  elements so focus no longer gets stuck on Tab
- Focus details summary and contenteditable elements
- Navigate relative to the focused element in DOM order, incl. shadow hosts
- Replace fromEvent/takeUntil with a host listener removed on destroy
- Remove dead code and cover the directive with tests

* fix(focus-trap): skip contenteditable elements nested in editing host

* fix(focus-trap): use isContentEditable to detect editing hosts

25373 of 27970 branches covered (90.72%)

Branch coverage included in aggregate %.

21 of 21 new or added lines in 1 file covered. (100.0%)

36894 of 38781 relevant lines covered (95.13%)

29965.39 hits per line