Page MenuHomePhabricator

DiscussionTools not working on mobile
Closed, ResolvedPublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

What happens?:
Nothing

What should have happened instead?:
Overflow menu should open, with (at least) "Edit" menu item.

Event Timeline

Esanders triaged this task as Unbreak Now! priority.Jan 22 2026, 9:07 PM
Esanders subscribed.

This isn't just the overflow menu - reply buttons are also broken.

Esanders renamed this task from Overflow menu button stopped working to DiscussionTools not working on mobile.Jan 22 2026, 9:08 PM
Esanders updated the task description. (Show Details)

Some of the reply buttons work for me, others display the message "Could not find the comment you're replying to on the page. It might have been deleted or moved to another page.".

There's a console error:

jQuery.Deferred exception: can't access property "type", threadItem is null
overflowMenu.js/init/...

The threadItems store has the wrong IDs:

image.png (396×389 px, 79 KB)

They should be h-editsummaries-2017-08-04T11:07:00.000Z etc.

It looks like this was caused by some recent changes to heading syntax. They should be reverted immediately, we need to be very careful when modifying heading syntax, as we learnt from the last time we restructured them for accessibility.

Change #1230451 had a related patch set uploaded (by DLynch; author: DLynch):

[mediawiki/extensions/MobileFrontend@master] Revert "Toggler: Update heading toggler to match WAI ARIA pattern"

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

Change #1230454 had a related patch set uploaded (by DLynch; author: DLynch):

[mediawiki/extensions/MobileFrontend@wmf/1.46.0-wmf.12] Revert "Toggler: Update heading toggler to match WAI ARIA pattern"

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

Change #1230454 merged by jenkins-bot:

[mediawiki/extensions/MobileFrontend@wmf/1.46.0-wmf.12] Revert "Toggler: Update heading toggler to match WAI ARIA pattern"

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

Mentioned in SAL (#wikimedia-operations) [2026-01-22T22:08:24Z] <kemayo@deploy2002> Started scap sync-world: Backport for [[gerrit:1230454|Revert "Toggler: Update heading toggler to match WAI ARIA pattern" (T415303 T407908)]]

Mentioned in SAL (#wikimedia-operations) [2026-01-22T22:10:23Z] <kemayo@deploy2002> kemayo: Backport for [[gerrit:1230454|Revert "Toggler: Update heading toggler to match WAI ARIA pattern" (T415303 T407908)]] synced to the testservers (see https://wikitech.wikimedia.org/wiki/Mwdebug). Changes can now be verified there.

Mentioned in SAL (#wikimedia-operations) [2026-01-22T22:15:07Z] <kemayo@deploy2002> Finished scap sync-world: Backport for [[gerrit:1230454|Revert "Toggler: Update heading toggler to match WAI ARIA pattern" (T415303 T407908)]] (duration: 06m 43s)

DLynch lowered the priority of this task from Unbreak Now! to High.
DLynch added projects: Essential-Work, Editing QA.

Okay, that change has been reverted, and the revert has been backported. Reader Experience Team has been notified via their channel about what was done.

Change #1230451 merged by jenkins-bot:

[mediawiki/extensions/MobileFrontend@master] Revert "Toggler: Update heading toggler to match WAI ARIA pattern"

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

for QS-Test-Automation and anyone else who might be able to answer:

Is there an opportunity to review existing automation (UI/Integration) for DiscussionTools and see if this could have been prevented through automation (potentially with @EAkinloose or @Ryasmeen to help understand the scope)?

Assuming the answer is yes: what tests would need to be created to do this?

EAkinloose edited projects, added: Verified; removed: Editing QA.