Open Bug 1789166 Opened 2 years ago Updated 2 years ago

Update <details>/<summary> to use "content-visibility:hidden" for hiding

Categories

(Core :: Layout, defect)

defect

Tracking

()

People

(Reporter: dholbert, Unassigned)

References

(Blocks 2 open bugs)

Details

The html spec on the <details> and <summary> elements changed a bit last year, in https://github.com/whatwg/html/pull/6466 , this commit in particular:
https://github.com/whatwg/html/commit/0050553a9170f6bd00b8ff0085cefe7f2295c591

We have bug 1724299 filed on updates to the find backend in response to this change; but there's some prerequisite work that needs to happen in the rendering implementation, I think.

https://github.com/whatwg/html/pull/6466#issuecomment-842489712 has some notes on specific steps that Chromium took to implement the spec-change. One main change in particular is that the spec now requires that the hidden slot should be hidden (when "collapsed") via content-visibility:hidden rather than via display:none. The spec specifically requires that now.

(There's also a bit more magic that needs to happen in order for the hidden content to also be findable via find-in-page, too (see more notes in that pull-request comment), since content-visibility:hidden is not-findable by default. Not sure if that makes sense to handle here vs. in bug 1724299 vs. elsewhere; we can sort that out when someone starts working on this.)

Severity: -- → S3
You need to log in before you can comment on or make changes to this bug.