IgniteUI / igniteui-webcomponents / 32253227988

98%

Build Type

push

github

Committed by web-flow

Commit Message

fix(popover): repair anchor resolution, open state and middleware order (#2343)

The target was resolved once per `anchor` change, and the platform popover was
only ever shown from the `open` property path, so an anchor that entered the DOM
after the first render left the component permanently invisible. The anchor is
now resolved every time the popover opens, and every visibility change is
committed through a single path.

- An anchor assigned while the popover is already open - by property or through
  the `anchor` slot - now shows it instead of only positioning it.
- Unsetting `anchor` detaches the popover, and an anchor removed from the DOM
  hides it instead of leaving it parked in the viewport corner. An id matching
  no element keeps the current anchor, as `igc-dropdown` does.
- `flip` runs before `shift` for aligned placements and after it for base ones,
  and `arrow` runs last, per the floating-ui ordering guidance.
- A reposition resolving after a close or an anchor swap no longer commits its
  stale coordinates, and `autoUpdate` is released when the platform closes the
  floating element.
- The arrow is inset by its own measured size instead of a fixed `-4px` and no
  longer discards parts set on it from the outside, which moves the tooltip
  arrow selectors to `[part~='...']`.
- The middleware chain is built once per update rather than per reposition, and
  `hasStickyAncestor`/`getVisibleAncestor` share an `iterAncestors` walker.

Coverage Stats

6966 of 7308 branches covered (95.32%)

Branch coverage included in aggregate %.

189 of 190 new or added lines in 2 files covered. (99.47%)

5 existing lines in 1 file now uncovered.

48606 of 49189 relevant lines covered (98.81%)

1449.96 hits per line