Page MenuHomePhabricator

Verify table of contents fix in production
Closed, ResolvedPublic

Description

T126836 got merged last sprint and will ride the train from the 26th April.
Let's verify there are no problems in production.

Event Timeline

@dr0ptp4kt Just want you to be aware of this ticket so you can weigh in on whether it should be in 71 or 72.

The description says it will ride the train 26th May. Is it actually 26th April 2016? It it's 26th April, please move this to Ready for Signoff for Sprint 71.

@Catrope originally reported the issue. @Catrope, would you be willing to confirm the fix works?

@phuedx any reason you suggested verifying this in sprint 72 not 71?

None.

Verified on https://m.mediawiki.org/wiki/Reading/Web/Team and various other pages as anon and logged in user that everything is a-okay.
Leaving open in case anyone wants to run any tests of their own.

Let's wait until a320a9c has been deployed to group2 (Thursday, 28th April).

bmansurov claimed this task.

Verified that it's working as expected on https://en.m.wikipedia.org/wiki/Book. If someone else wants to verify it, just visit the page and scroll down until you see the ToC and refresh the page, you'll see the space is reserved for the ToC to appear, and it does appear after a little while. On a smaller screen sizes the ToC doesn't appear at all, and that's the expected behavior.