Page MenuHomePhabricator

Talk overlay "issues" are not visible in A/B test
Closed, ResolvedPublic2 Estimated Story Points

Description

Ambox templates are used on pages other than the main namespace - notably template, categories and talk to provide information to editors.

For example talk pages:
https://en.m.wikipedia.org/wiki/Talk:Sandwich#/issues

Screen Shot 2018-07-06 at 3.16.44 PM.png (603×350 px, 62 KB)

When bucketed with the new treatment
e.g.http://reading-web-staging.wmflabs.org/wiki/Talk:Sandwich
The issues are not accessible.

Screen Shot 2018-07-06 at 3.17.50 PM.png (594×350 px, 50 KB)

It also seems our recent changes have caused a regression with the display of these issues. Note well that this regression was fixed in T199005#4409931.

Screen Shot 2018-07-06 at 3.14.25 PM.png (597×339 px, 69 KB)

acceptance criteria

  • Restore the "about this page" link for talk and category namespace. A/B test should not apply here.
  • Do not render issues if their html is an empty string.
  • Make it clear in the code that this code should be retained post A/B test

qa steps

  • Ensure both A and B groups are accessible for http://reading-web-staging.wmflabs.org/wiki/Pharmacovigilance. In one treatment the "page issues" link renders under the heading, and in the other the banner should be visible and clickable
  • Check that on "Category:Use_American_English_from_January_2014", "Talk:Sandwich" and "Talk:Pharmacovigilance" show a "About this page" link under the title regardless of whether you are in groups A or B of the A/B test.

sign off notes

As part of sign off, we'll want to work out the long term plan for dealing with these type of amboxes. Options involve keeping both versions around; redesigning these amboxes or displaying them (despite the impact on appearance)

Event Timeline

Jdlrobson updated the task description. (Show Details)
Jdlrobson updated the task description. (Show Details)
Jdlrobson added a subscriber: alexhollender_WMF.

@alexhollender we may want to investigate redesigning these. This bug is just about restoring the old behaviour to avoid a regression. Feel free to open a new task to think about that.

Change 444745 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/skins/MinervaNeue@master] Talk page 'issues' are not subject to A/B test

https://gerrit.wikimedia.org/r/444745

Change 444746 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/skins/MinervaNeue@master] Don't render blank issues

https://gerrit.wikimedia.org/r/444746

Change 444746 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] Don't render blank issues

https://gerrit.wikimedia.org/r/444746

Jdlrobson renamed this task from Regression: Talk overlay "issues" do not show to Talk overlay "issues" are not visible in A/B test.Jul 9 2018, 10:03 PM
Jdlrobson lowered the priority of this task from High to Medium.
Jdlrobson removed a project: Regression.
Jdlrobson updated the task description. (Show Details)

Dropping priority now regression part of this task has been addressed.

ovasileva raised the priority of this task from Medium to High.Jul 10 2018, 4:23 PM
ovasileva set the point value for this task to 2.

Change 444745 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] Talk page 'issues' are not subject to A/B test

https://gerrit.wikimedia.org/r/444745

Jdlrobson updated the task description. (Show Details)

This code is on reading web staging.

This is working as expected.

@Jdlrobson regarding design improvements — do you think it would be appropriate to give these the same visual treatment as page issues? It seems like they are basically the same thing, just on non-main namespace pages. Also, do we need to think about how people can access them from the non-Wiki version of the Talk page?

Screen Shot 2018-07-12 at 11.33.31 AM.png (621×399 px, 43 KB)

@alexhollender ideally yes although Olga has already said this is all out of scope for the issues project. We would want to do a similar Jan style audit of all the templates before starting that adventure.

@Jdlrobson: So I see now http://reading-web-staging.wmflabs.org/wiki/Talk:Sandwich has "About this Page" link and it takes me to issues page and there is no empty rows on that issues page.

I also checked that when I go to http://reading-web-staging.wmflabs.org/wiki/Pharmacovigilance, there is this banner about issues regarding that page and it takes me to the issues page.

But, I have no idea about how to test it for different user groups for A/B testing, I tried by clearing the session with localStorage.clear(); sessionStorage.clear() in an attempt to get bucketed as a different user group, but every time I got the banner not the link. So not sure, if I am doing it right.

Hey @Ryasmeen http://reading-web-staging.wmflabs.org has an A-B test running on the main namespace with 50% bucketing.
To get bucketed you can open a link in a new tab/incognito window
Bucket 1:

Screen Shot 2018-07-24 at 5.03.36 PM.png (272×368 px, 32 KB)

Bucket 2:
Screen Shot 2018-07-24 at 5.03.56 PM.png (261×459 px, 40 KB)

Does that help?

Okay, this time I finally got bucketed to get "page issues" link rendered under the heading. So marking it as verified.