Maniphest T206406

Main Page on a wiki doesn't have a Discussion page / Talk page on Mobile View
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	xSavitar
	Oct 6 2018, 8:16 PM

Description

While reading this task: T174995 and it's discussion beneath, I realized something also in mobile view. The main page doesn't have a discussion button or link to talk page in mobile view but other page have. See below;

Link to Main page on mw.org (Desktop view): https://www.mediawiki.org/w/index.php?title=MediaWiki&mobileaction=toggle_view_desktop (discussion page present)
Link to Main page on mw.org (Mobile view): https://m.mediawiki.org/w/index.php?title=MediaWiki&mobileaction=toggle_view_mobile (no link to discussion page)

Link to a random page on mw.org (Desktop view): https://www.mediawiki.org/wiki/Sites_using_MediaWiki (link to discussion page available)
Link to a (same random page) on mw.org (Mobile view): https://m.mediawiki.org/wiki/Sites_using_MediaWiki (link to discussion/talk page at the bottom using "Discussion" button)

So this only happens for a wiki's main page? Is this intentional?

NOTE: This has been tested cross wiki (Meta-Wiki, MediaWiki, etc), same behavior.

Developer notes

Looking through the code history I think this is an oversight.
It was added in https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/MobileFrontend/+/213263/1/includes/skins/SkinMinerva.php to disable watch link on main page and then swept into the generic logic in https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/MobileFrontend/+/301104/1/includes/skins/SkinMinerva.php

The problem is "page actions" covers all the links at the top of the page and the button
isAllowedPageAction should be extended to allow "talk" buttons on the main page, but isAllowedPageAction needs to still return false on main page for edit, watch.

Feel free to have a go - make sure to add new tests and not to break the existing ones!

Testing instructions

Visit https://en.m.wikipedia.beta.wmflabs.org/wiki/Main_Page while logged in
Scroll to the bottom of the page
Ensure that there is a "Discussion" button

Details

	Subject	Repo	Branch	Lines +/-
	Add "Discussion" button to Main page on Mobile View	mediawiki/skins/MinervaNeue	master	+15 -4

Customize query in gerrit

Related Objects

Mentioned In: T174995: Link to talk / discussion page should indicate if page exists or not
Mentioned Here: T54165: Links to talk pages in mobile view for all anonymous users
T174995: Link to talk / discussion page should indicate if page exists or not

Event Timeline

xSavitar created this task.Oct 6 2018, 8:16 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptOct 6 2018, 8:16 PM

xSavitar renamed this task from Main Page doesn't have a Discussion page / Talk page on Mobile View to Main Page on a wiki doesn't have a Discussion page / Talk page on Mobile View.Oct 6 2018, 8:16 PM

xSavitar updated the task description. (Show Details)

xSavitar mentioned this in T174995: Link to talk / discussion page should indicate if page exists or not.

This appears to be intentional:
https://github.com/wikimedia/mediawiki-skins-MinervaNeue/blob/master/includes/skins/SkinMinerva.php#L231

You'll need to do some digging (back in the MobileFrontend repo to work out the motivation here) but I suspect it was to avoid spam on the main talk page. Regardless, the code should document the reasoning or the line should be removed.

In case this is intentional @Jdlrobson, I think the reasoning is valid, the code should be documented to avoid future confusions. So could I submit a patch for this with the reason you mentioned?

Feel free to have a go - make sure to add new tests and not to break the existing ones!

Jdlrobson updated the task description. (Show Details)Oct 8 2018, 6:25 PM

Jdlrobson moved this task from Incoming to Triaged but Future on the Web-Team-Backlog board.

ovasileva triaged this task as Medium priority.Oct 16 2018, 9:15 PM

Change 468836 had a related patch set uploaded (by D3r1ck01; owner: Alangi Derick):
[mediawiki/skins/MinervaNeue@master] Add "Discussion" button to Main page on Mobile View

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

gerritbot added a project: Patch-For-Review.Oct 21 2018, 3:15 PM

@Jdlrobson, a patch is up for review :)

Will look later today!

Jdlrobson moved this task from To Do to Needs Code Review on the Web-Team-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.Oct 22 2018, 6:06 PM

@D3r1ck01 patch looks good but I'm seeing an alignment issue

relates to

.page-Main_Page #page-secondary-actions .language-selector {
    margin-top: 1em;
}

so would be good to understand why that rule exists and if it's needed
(at minimum we'll want to apply the same for talk)

Jdlrobson moved this task from Needs Code Review to Needs More Work on the Web-Team-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.Oct 22 2018, 9:14 PM

Hmm..... Didn't spot this. Please Jon, don't merge the patch. Let me include this UI fix too at once. Then the cleanup you suggested too before you do final review and possibly merge. But I'm glad the patch is in the right direction. Thanks :)

Thank YOU 🙏

@Jdlrobson, so that rule exist and is highly used :) So I've just applied same style that goes to .language-selector to the .talk which is a class for talk / discussion button. New CSS now look like;

.page-Main_Page #page-secondary-actions .talk .language-selector {
    margin-top: 1em;
}

I made sure .talk comes before .language-selector as it appears on the interface, but reversing their positions doesn't change anything on the UI, I just want code and UI to be in sync.

Also, I've reworked the logic a little bit and dried it up like you suggested but the logic gate is a little different now ;). You can review the PS on Gerrit :)

Jdlrobson moved this task from Needs More Work to Needs Code Review on the Web-Team-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.Oct 23 2018, 5:03 PM

Jdlrobson moved this task from Needs Code Review to Needs More Work on the Web-Team-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.Oct 23 2018, 6:39 PM

Jdlrobson moved this task from Needs More Work to Needs Design Review on the Web-Team-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.Oct 23 2018, 8:07 PM

Jdlrobson updated the task description. (Show Details)

xSavitar moved this task from Doing [WIP] to Completed (To be Resolved) on the User-xSavitar board.Oct 23 2018, 8:10 PM

Change 468836 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] Add "Discussion" button to Main page on Mobile View

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

@Jdlrobson, when is the right time to test on beta cluster? When it's finally deployed if I'm not making a mistake right? If so, does this need to go in SWAT?

ReleaseTaggerBot added a project: MW-1.33-notes (1.33.0-wmf.2; 2018-10-30).Oct 23 2018, 9:01 PM

No need for SWAT. This will ride the train. Your change is now live on https://en.m.wikipedia.beta.wmflabs.org/ !

@Jdlrobson, thanks the clarification about SWAT. Also, I've just checked https://en.m.wikipedia.beta.wmflabs.org/ and don't seem to find the "Discussion" button, do you see it on your end?

In T206406#4691165, @D3r1ck01 wrote:

@Jdlrobson, thanks the clarification about SWAT. Also, I've just checked https://en.m.wikipedia.beta.wmflabs.org/ and don't seem to find the "Discussion" button, do you see it on your end?

Sorry about this! I was not logged in :), just logged in now and I see it. This is another call for concern, should the discussion button show only for logged in users? Usually, annon users should have access to this functionality right?

xSavitar moved this task from Completed (To be Resolved) to Doing [WIP] on the User-xSavitar board.Oct 24 2018, 9:46 AM

T54165: Links to talk pages in mobile view for all anonymous users

xSavitar moved this task from Doing [WIP] to Completed (To be Resolved) on the User-xSavitar board.Oct 24 2018, 7:31 PM

Looks good 👍

• alexhollender_WMF reassigned this task from xSavitar to Ryasmeen.Oct 25 2018, 1:09 AM

• alexhollender_WMF added a project: Product-QA.

• alexhollender_WMF updated the task description. (Show Details)

• alexhollender_WMF moved this task from Needs Design Review to Needs Code Review on the Web-Team-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.

@Jdlrobson two clarifications:

I should be testing this on https://en.m.wikipedia.beta.wmflabs.org/wiki/Main_Page (originally the task said https://en.wikipedia.beta.wmflabs.org/wiki/Main_Page) correct?
since we moved the Needs Design Review column, shall I move them to Needs Code Review after I'm done with them?

• alexhollender_WMF removed Ryasmeen as the assignee of this task.Oct 25 2018, 1:12 AM

• alexhollender_WMF added a subscriber: Ryasmeen.

In T206406#4693597, @alexhollender wrote:

@Jdlrobson two clarifications:

I should be testing this on https://en.m.wikipedia.beta.wmflabs.org/wiki/Main_Page (originally the task said https://en.wikipedia.beta.wmflabs.org/wiki/Main_Page) correct?

Yup.

since we moved the Needs Design Review column, shall I move them to Needs Code Review after I'm done with them?

Yes

This one is not working for me on https://en.m.wikipedia.beta.wmflabs.org/wiki/Main_Page on the device simulators. I also tried on a couple of real devices and it is not working there either. It does work on regular desktop browsers.

Screen Shot 2018-11-06 at 5.13.26 PM.png (1×680 px, 506 KB)

Screen Shot 2018-11-06 at 5.10.47 PM.png (1×622 px, 665 KB)

Screen Shot 2018-11-06 at 5.11.56 PM.png (1×650 px, 629 KB)

@ABorbaWMF you need to be logged in to see the button. See T206406#4691583

Jdlrobson updated the task description. (Show Details)Nov 7 2018, 1:22 AM

Jdlrobson moved this task from Needs More Work to Needs QA on the Web-Team-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.Nov 7 2018, 7:15 PM

@Jdlrobson - Sorry about that!

Looks good to me

phuedx removed Ryasmeen as the assignee of this task.Nov 8 2018, 6:05 PM

looks good to me! resolving.

Restricted Application added a project: User-Ryasmeen. · View Herald TranscriptNov 8 2018, 9:14 PM

xSavitar awarded a token.Nov 8 2018, 9:17 PM

	F27086890: Screen Shot 2018-11-06 at 5.10.47 PM.png
	Nov 7 2018, 1:15 AM

	F27086891: Screen Shot 2018-11-06 at 5.11.56 PM.png
	Nov 7 2018, 1:15 AM

	F27086889: Screen Shot 2018-11-06 at 5.13.26 PM.png
	Nov 7 2018, 1:15 AM

	F26716386: Screen Shot 2018-10-22 at 2.13.01 PM.png
	Oct 22 2018, 9:14 PM

	F27101838: image.png
	Nov 7 2018, 11:39 PM

	F27101841: image.png
	Nov 7 2018, 11:39 PM

	F27101845: image.png
	Nov 7 2018, 11:39 PM

Main Page on a wiki doesn't have a Discussion page / Talk page on Mobile ViewClosed, ResolvedPublicActions