Page MenuHomePhabricator

Add support to TOC for heading elements that have (or child of element with) display: none;
Closed, ResolvedPublicFeature

Description

Author: number24602

Description:
If a page has a section-header that is in a collapsed part of the page, the table-of-contents link to it does not work. It doesn't even jump to the collapsed chunk. That's confusing--there's a TOC entry that's a hyperlink that's generated by the WM server pointing to other content on that same page, it should be able to resolve itself. If the chunk is un-collapsed, the TOC link works correctly. A better behavior would be to have the link to the header in the collapsed content cause it to un-collapse (i.e., the link in the TOC go to the section to which it points).


Version: unspecified
Severity: enhancement

Details

Reference
bz33937

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 12:05 AM
bzimport set Reference to bz33937.
bzimport added a subscriber: Unknown Object (MLST).

I *think* you're talking about the mobile version of the site, right?

number24602 wrote:

Nope, regular version; Vector skin, firefox (consistent on several versions and OS/platforms). See https://en.wikipedia.org/wiki/User:DMacks/transclude-test and click on any of the TOC entries. For example, "1.1 liquid nitrogen in an electric kettle" is a link to https://en.wikipedia.org/wiki/User:DMacks/transclude-test#liquid_nitrogen_in_an_electric_kettle (note this is a live-updated page, so titles may change but effect is same for any of them) and that section/link-target is in the collapsed-content box below.

Thanks for the clear demonstration. It seems to affect all skins (which makes sense). Some sort of JS modification is likely the solution.

matmarex set Security to None.
matmarex removed a subscriber: Unknown Object (MLST).
Aklapper lowered the priority of this task from Medium to Low.Dec 9 2015, 6:13 PM
Aklapper subscribed.
Aklapper changed the subtype of this task from "Task" to "Feature Request".Feb 4 2022, 11:14 AM
Aklapper removed subscribers: Tfinc, TrevorParscal.