1563275 - dynamically inserted slotted elements may be inaccessible

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect, P5)

Description

[alexander :surkov (:asurkov)]

if an element slotted under a shadow DOM of a custom element is inserted, then accessible service fails to create an accessible for it and add into hierarchy.

Comment 1

[alexander :surkov (:asurkov)]

only an XBL element under a shadow DOM gives this bug, which is triggerred by arrowscrollbox in bug 1555497, see [1]. So if arrowscrollbox CE conversion is done in bug 1514926, then we might not care about this bug.

[1] https://searchfox.org/mozilla-central/source/accessible/base/TreeWalker.cpp#104

Comment 2

[James Teh [:Jamie]]

Just to be extra clear, are you saying it isn't possible to trigger this bug in web content, only with XBL?

Comment 3

[alexander :surkov (:asurkov)]

(In reply to James Teh [:Jamie] from comment #2)

Just to be extra clear, are you saying it isn't possible to trigger this bug in web content, only with XBL?

that's right.

Comment 4

[James Teh [:Jamie]]

(In reply to alexander :surkov (:asurkov) from comment #1)

So if arrowscrollbox CE conversion is done in bug 1514926, then we might not care about this bug.

I see that bug has p5. What's the likelihood of that being worked on? Just trying to figure out priority for this bug.

Comment 5

[alexander :surkov (:asurkov)]

not yet sure, it might turn out to be a blocker of bug 1555497 if no workarounds found

Comment 6

[James Teh [:Jamie]]

Given that bug 1555497 has no priority, assigning p5 for now. Happy to re-triage once more is known.

Comment 7

[alexander :surkov (:asurkov)]

Attachment: Bug 1563275 - make dynamically inserted slotted elements accessible

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a3aa13e0e10b
make dynamically inserted slotted elements accessible r=Jamie

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/10e5f2b2940c
make dynamically inserted slotted elements accessible: take new Prettier style into account. a=eslint-fix CLOSED TREE

Comment 10

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/a3aa13e0e10b
https://hg.mozilla.org/mozilla-central/rev/10e5f2b2940c