Change Details

Spun out of T297611 When determining whether or not to show the new ToC, the same rules that govern the current ToC should be applied. If the ToC is currently hidden via magic words like `__NOTOC__` or by other means, it should remain hidden in this update. # Developer notes - If __NOTOC__ is present in the page, SkinMustache still gets the data needed to render the table of contents. We'll need to change the core code, to provide some indication that the table of contents was disabled to support this change. - `Skin::getSectionsData` should return null when __NOTOC__ is used. Attached POC patch uses `OutputPage::isTOCEnabled` in core (could be limited to Vector in `SkinVector::getTemplateData()` to set data variable `is-vector-table-of-contents-visible`?) - returns false when NOTOC magic word is present.

Spun out of T297611 When determining whether or not to show the new ToC, the same rules that govern the current ToC should be applied. If the ToC is currently hidden via magic words like `__NOTOC__` or by other means, it should remain hidden in this update. # Developer notes - If __NOTOC__ is present in the page, SkinMustache still gets the data needed to render the table of contents. We'll need to change the core code, to provide some indication that the table of contents was disabled to support this change. - `Skin::getSectionsData` should return null when __NOTOC__ is used. Attached POC patch uses `OutputPage::isTOCEnabled` in core (could be limited to Vector in `SkinVector::getTemplateData()` to set data variable `is-vector-table-of-contents-visible`?) - returns false when NOTOC magic word is present. See thread in #content-transformers channel re: `OutputPageisTOCEnabled` usage https://wikimedia.slack.com/archives/C024Z8K9CAU/p1642090580148800

Spun out of T297611 When determining whether or not to show the new ToC, the same rules that govern the current ToC should be applied. If the ToC is currently hidden via magic words like `__NOTOC__` or by other means, it should remain hidden in this update. # Developer notes - If __NOTOC__ is present in the page, SkinMustache still gets the data needed to render the table of contents. We'll need to change the core code, to provide some indication that the table of contents was disabled to support this change. - `Skin::getSectionsData` should return null when __NOTOC__ is used. Attached POC patch uses `OutputPage::isTOCEnabled` in core (could be limited to Vector in `SkinVector::getTemplateData()` to set data variable `is-vector-table-of-contents-visible`?) - returns false when NOTOC magic word is present. See thread in #content-transformers channel re: `OutputPageisTOCEnabled` usage https://wikimedia.slack.com/archives/C024Z8K9CAU/p1642090580148800