Refine some of the JS/CSS for the feature:
- Ensure that loading pages with anchor links pointing to deeply nested sections works properly
- Ensure that collapsing sections properly triggers scroll to relevant areas
- other things TBD
Minor bugs so far:
- The Parsoid version of the Sticky Headers styles may benefit from z-index: 1 to ensure content elements like images never overlap them. The legacy parser styles already do this. (Patch)
-
section headers with more than one line peak through the sticky header - see T410441 -
Our autoscroll requirement assumes that the reader wants to go down, not up. I think we should remove it(This patch should address this) -
Our experiment won't run on pages referred from "Related articles" e.g.(out of scope / expected behavior) - Closing sections jumps to random points in the section below Patch
- Bullet points "peak out" from beside the scrolling headers
- now fixed for unordered lists (bullet points) and short ordered lists
- could still occur with long ordered lists (numbers with multiple digits)
- we could be even more proof against this sort of problem with bigger changes to the Minerva skin's body content and heading and section margin/padding but this may require tweaking HTML to avoid changing the visible layout
