Skip to content

Meta: drop the now-unused toggleStatus() - #12865

Open
zcorpan wants to merge 2 commits into
mainfrom
mdn-anno-drop-togglestatus
Open

Meta: drop the now-unused toggleStatus()#12865
zcorpan wants to merge 2 commits into
mainfrom
mdn-anno-drop-togglestatus

Conversation

@zcorpan

@zcorpan zcorpan commented Aug 31, 2026

Copy link
Copy Markdown
Member

MDN annotation panels are <details> now, so nothing calls this. Note that
archived commit snapshots have their own copy baked in and are unaffected.

DO NOT LAND until the wattsi change that emits <details> panels has shipped
and html.spec.whatwg.org has been rebuilt with it; until then this function
is what makes the panels open.


This is step 4 of whatwg/wattsi#169

Generated by Claude.

@zcorpan zcorpan added spec tooling do not merge yet Pull request must not be merged per rationale in comment labels Aug 31, 2026
@annevk

annevk commented Aug 31, 2026

Copy link
Copy Markdown
Member

This needs a "Meta:" prefix (and then lowercase "drop").

@zcorpan zcorpan mentioned this pull request Aug 31, 2026
7 tasks
@zcorpan zcorpan changed the title Drop the now-unused toggleStatus() Meta: drop the now-unused toggleStatus() Aug 31, 2026
wattsi is moving the MDN compatibility tables out of the markup and into a
"data-mdn" attribute per panel, built on first open by this script. Part of
#12782.

The inline `toggleStatus()` stays for now: it is what drives the current
`<button>`-based panels, and it has to keep working until the wattsi change
has shipped. Removing it is a follow-up.
MDN annotation panels are `<details>` now, so nothing calls this. Note that
archived commit snapshots have their own copy baked in and are unaffected.

DO NOT LAND until the wattsi change that emits `<details>` panels has shipped
and html.spec.whatwg.org has been rebuilt with it; until then this function
is what makes the panels open.
@zcorpan
zcorpan force-pushed the mdn-anno-drop-togglestatus branch from 1985c25 to 675e5ff Compare August 31, 2026 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge yet Pull request must not be merged per rationale in comment spec tooling

Development

Successfully merging this pull request may close these issues.

2 participants