Patrol link on diff pages disappeared on Persian Wikipedia
Closed, DuplicatePublicBUG REPORT
Actions

Assigned To

Authored By

	Jeeputer
	Nov 7 2023, 4:53 AM

Description

Steps to replicate the issue:

See this unpatrolled page on a desktop computer. If you have patroller right you'll see a link labeled "[به این صفحه برچسب گشت بزن]" to patrol the whole page.
now go to this diff page (the second revision of that page), again, using a desktop computer. You will see the patrol link labeled "[برچسب گشت بزن]" below the text difference section.
Now open both links using a mobile device. You will probably see the patrol link on the page itself, but you won't see it on the diff page.

What happens?:
Patrol link on diff pages disappeared on mobile devices. If you view the diff page using the MinervaNeue skin on a desktop computer, the patrol link is still present.

Other information:
Based on a report on Persian Wikipedia, I guess the issue reported on T350515 may have a similar origin as they appeared at the same time.

Details

	Subject	Repo	Branch	Lines +/-
	Restore content below diff	mediawiki/skins/MinervaNeue	master	+0 -1

Customize query in gerrit

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Resolved	BUG REPORT	Jdlrobson	T350454 Can't see anything past table.diff (including patrol links)
		Duplicate	BUG REPORT	Jdlrobson	T350650 Patrol link on diff pages disappeared on Persian Wikipedia

Event Timeline

Jeeputer created this task.Nov 7 2023, 4:53 AM

Restricted Application added subscribers: Huji, Masumrezarock100, Aklapper. · View Herald TranscriptNov 7 2023, 4:53 AM

Ladsgroup subscribed.Nov 7 2023, 1:56 PM

Arian_Ar subscribed.Nov 7 2023, 3:52 PM

Jdlrobson added a project: Web-Team-Backlog (Web Team FY2023-24 Q2 Sprint 4).Nov 7 2023, 10:29 PM

(I don't have patroller right so was unable to check this right now). Would be helpful if someone can determine what CSS is responsible for hidding the link and if this is also caused by FlaggedRevisions loading mediawiki.diff.styles out of context (and thus require an answer to T350596 for how to organize these styles more reliably).

In T350650#9314755, @Jdlrobson wrote:

I don't have patroller right so was unable to check this right now.

I gave autopatrolled and patroller rights to your account (Jdlrobson) on fawiki.

Would be helpful if someone can determine what CSS is responsible for hidding the link

As far as I can see, when you zoom in on the diff page, the display: none attribute of the div with class mw-diff-revision-history-links is struck through, and a new div with the same class appears with display: flex.
I can see a new @media (max-width: 1000px) above the new div after zooming, but as I am not that familiar with CSS, I don't know what that means. However, according to the description of T350515, I think it can be related to this issue.
Thanks.