create HTML table caption accessible unconditionally

NEW
Unassigned

Status

()

Core
Disability Access APIs
P2
normal
3 months ago
a month ago

People

(Reporter: surkov, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

3 months ago
spun off bug 1321960 comment #38

we have GetRect().IsEmpty() check [1] when deciding to create or not create HTML:caption accessible, which makes a caption inaccessible in the example

<table>
  <caption style="width: 0; height: 0">Some text</caption>
</table>

It seems the behavior goes from 377783 with no explanation. Any objections for reverting the change? What do other browsers do?

[1] https://hg.mozilla.org/mozilla-central/annotate/6fec4855b5345eb63fef57089e61829b88f5f4eb/layout/generic/nsBlockFrame.cpp#l6852
(Reporter)

Updated

3 months ago
Flags: needinfo?(mzehe)

Comment 1

3 months ago
Inconclusive. Chrome renders it as a table caption, IE does render it as a regular div (or so it seems to NVDA), and Edge renders it as plain text as child of the table accessible.

I'd say we follow Chrome's example and just render the thing. Caption is an exotic enough element that we can assume author intent if they bother using it, even if they style it so that it is invisible.
Flags: needinfo?(mzehe)
Priority: -- → P5
Summary: create HTML table caption accessible uncondionally → create HTML table caption accessible unconditionally
(Reporter)

Comment 2

a month ago
It's definitely not p5, we have to fix to depend on layout update less than we do now, which is tightly related with our performance. Ideally we should get a fix around a next release or so, so bumping up to p2 for now to not get it lost in the backlog.
Priority: P5 → P2
You need to log in before you can comment on or make changes to this bug.