Page MenuHomePhabricator

[beta & prod] simplified Minerva toolbelt - the Comments counter doesn't reflect the actual number of topics
Closed, ResolvedPublicBUG REPORT

Description

Steps:
On enwiki beta

What happened:

  • the counter of topics did not increase

The articles with existing topics, e.g. https://en.wikipedia.beta.wmcloud.org/wiki/Polar_bear?minimalMinerva=1, also show 0 comments.

From T434182

I’ve checked one article (Madrid) across the usual target wikis (ar, zh, fr, id, vi, and en) in production (1.47.0-wmf.14): the comments chip displays zero comments for all wikis but zh:

NOTE: with 1.47.0-wmf.15, the URL parameter to activate MinMin's toolbar is ?minMinToolbar=1.
NOTE: all target wikis have DiscussionTools in their Special:Version pages.

Event Timeline

mfossati renamed this task from [beta] simplified Minerva toolbelt - the Comments counter doesn't reflect the actual number of topics to [beta & prod] simplified Minerva toolbelt - the Comments counter doesn't reflect the actual number of topics.Fri, Aug 7, 9:28 AM
mfossati updated the task description. (Show Details)

Change #1324402 had a related patch set uploaded (by Elliottetzkorn; author: Elliottetzkorn):

[mediawiki/skins/MinervaNeue@master] [MinMin] Fix comments counter to reflect discussion topics count

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

SherryYang-WMF lowered the priority of this task from High to Low.Wed, Aug 12, 4:40 PM

Change #1324402 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] [MinMin] Fix comments counter to reflect discussion topics count

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

Verified the counter fix on a wiki page that I created locally.

Notes:

  • With this change, the number of topics is displayed when (a) DiscussionTools is loaded or (b) when there is 1 or more topics. When that condition is not met, it displays "comments" rather than "0 comments".
MinMin comments with number.png (248×116 px, 9 KB)
MinMin comments without number.png (238×110 px, 8 KB)
  • Suggestion to replace the "comments" label with "discussions" or something else to represent the talk topics better. Bringing this up in case the intention behind this label was initially for the total number of comments on the talk page and then later changed to the number of topic threads. cc: @JScherer-WMF

Already taken into account in the main minmin QA ticket

SherryYang-WMF raised the priority of this task from Low to High.Thu, Aug 13, 10:50 PM

Test wiki created on Patch demo by Etonkovidova (WMF) using patch(es) linked to this task:
https://66e682ecb6.catalyst.wmcloud.org/w/

I used ?minMinToolbar=1 for testing, and it works on enwiki wmf.15, but strangely did not work on testwiki wmf.15 and nor on patchdemo that I created: https://66e682ecb6.catalyst.wmcloud.org/wiki/Gail_Sheehy?minMinToolbar=1

I was able to check the fix on enwiki beta (before beta cluster crashed)

  • the counter correctly reflects the number of topics on the Talk page
  • the counter correctly increases when the number of topics changes

Screenshot 2026-08-17 at 3.41.59 PM.png (760×1,364 px, 374 KB)

Verified the counter fix on a wiki page that I created locally.

Notes:

  • With this change, the number of topics is displayed when (a) DiscussionTools is loaded or (b) when there is 1 or more topics. When that condition is not met, it displays "comments" rather than "0 comments".
MinMin comments with number.png (248×116 px, 9 KB)
MinMin comments without number.png (238×110 px, 8 KB)

Limited testing on beta cluster did not show "comments" for zero comments. I'll test more when beta will be back to normal.

  • Suggestion to replace the "comments" label with "discussions" or something else to represent the talk topics better. Bringing this up in case the intention behind this label was initially for the total number of comments on the talk page and then later changed to the number of topic threads. cc: @JScherer-WMF

Agree. Should it be "Topics"?
Talk pages differentiate between topics and replies. When a user publish a reply to a topic - the message says: "Your comment was published".

Test wiki on Patch demo by LWatson-WMF using patch(es) linked to this task was deleted:

https://a69169a3c6.catalyst.wmcloud.org/wiki/Main Page

Test wiki on Patch demo by Etonkovidova (WMF) using patch(es) linked to this task was deleted:

https://66e682ecb6.catalyst.wmcloud.org/w/

Testing Summary:

Re-tested on enwiki beta

  • the counter correctly reflects the number of topics on the Talk page
  • the counter correctly increases/decreases when the number of topics changes
Questions

Looks good. Follow-up ticket T435249 discussed today in standup