hideOthers hides everything when target is in Shadow DOM and paren is not explicitly provided

[harunurhan]

Traversal to mark all parent chain of as "elementToKeep" gets stuck at shadow root since it's using element.parentNode, so no top level parent (direct child of body) is reached.

aria-hidden/src/index.ts

Line 50 in 95843f2

keep(el.parentNode!);

And since it's using target.ownerDocument.body to start setting aria-hidden: "true", it just stops after hiding all of its children, which is pretty much everything on the page :(

aria-hidden/src/index.ts

Line 10 in 95843f2

return sampleTarget.ownerDocument.body;

Consumer can indeed address this issue manually, but maybe we could change the default getDefaultParent to return shadow root to make this easier for everyone, since not all will notice this is broken.

aria-hidden/src/index.ts

Line 3 in 95843f2

const getDefaultParent = (originalTarget: Element | Element[]): HTMLElement | null => {

Maybe something like that could work?

const rootNode = element.getRootNode();
const isShadowRoot = rootNode instanceof ShadowRoot;
return isShadowRoot ? rootNode : element.owerDocument.body;

However in some cases if not most, I don't think hiding everything only within a shadow DOM is going to work, so it could be better to fix parent traversal.

keep(el.parentNode ?? el.host);

[theKashey]

• check if parent contains target
• if not
  • if there is owner document - move target there
    • repeat
  • display warning. If parent equal to the body - do nothing (we really don't want to "hide" everything)

This should "bubble up" target to current document, po

[theKashey]

Resolved in v1.2.2