IgniteUI / igniteui-webcomponents / 33503665428

98%

push

github

fix(avatar, badge): repair icon-only badge padding and ARIA semantics (#2363)

The badge `icon` part activated for any slotted `igc-icon`, while the padding
rule matched `[part='base']` exactly - so an icon badge that also had text lost
its inline padding and clipped the text under `overflow: hidden`. Avatar
exposed the invalid role `image` and inverted its ARIA: `alt`/`initials` became
the role description, while the accessible name was pinned to the literal
"avatar".

- The `icon` part now requires the icon to be the badge's only content, as the
  documentation already claimed, and the padding rule excludes that part
  explicitly instead of relying on an exact attribute match.
- Badge reports a static `badge` role description, so the visual `variant` no
  longer reaches assistive technology. `role="status"` is unchanged.
- Avatar uses `role="img"`, derives its accessible name from `alt` then
  `initials`, and keeps `avatar` as the role description.
- Avatar renders `alt=""` when no alt text is given, rather than an `<img>`
  carrying no `alt` attribute at all.
- Removed the avatar `icon` part, documented but never rendered.
- The avatar fallback test asserted pre-update DOM and passed for the wrong
  reason; it now exercises the load error, next to new coverage for the image
  alt, both components' ARIA and the badge icon part.

7283 of 7636 branches covered (95.38%)

Branch coverage included in aggregate %.

20 of 20 new or added lines in 2 files covered. (100.0%)

51113 of 51679 relevant lines covered (98.9%)

1465.79 hits per line