Page MenuHomePhabricator

Sliding Drawer Not Rendering When Clicking Bell/Tray Icon - Minerva - Echo Extension
Open, MediumPublic0 Estimated Story Points

Description

NOTE: In Minerva desktop skin there is an existing bug relating to the tray icon (T343839). This is out of scope for the bug.

I am experiencing an issue with the Minerva skin and the Echo extension. When I click on the bell or tray icon, instead of the sliding drawer rendering as expected, it is redirecting me to the notification page. This behavior is not consistent with the intended functionality of the Echo extension.

Steps to Reproduce:

  1. Switch to mobile site (Minerva skin) e.g. https://en.m.wikipedia.beta.wmflabs.org/
  2. Ensure the Echo extension is enabled.
  3. Click on the bell icon in the user interface.

Expected Behavior:
Upon clicking the bell icon, a sliding drawer should appear on the right, displaying notifications.

Actual Behavior:
Clicking the bell icon redirects me to the notification page, and the sliding drawer does not render.

Sep-18-2023 14-26-18.gif (220×320 px, 61 KB)

Pixel Error

Screenshot 2023-09-18 at 1.12.46 PM.png (1×3 px, 280 KB)

QA Results - Beta

ACStatusDetails
1T346685#9189604

Event Timeline

KSarabia-WMF triaged this task as Unbreak Now! priority.Mon, Sep 18, 7:18 PM
KSarabia-WMF added a project: Web-Team-Backlog.
KSarabia-WMF updated the task description. (Show Details)
KSarabia-WMF updated the task description. (Show Details)

Change 958570 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/MinervaNeue@master] Use checkRoute when using navigate

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

Change 958570 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] Fixes notification overlay: Change URL using location

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

Jdlrobson lowered the priority of this task from Unbreak Now! to Medium.
Jdlrobson added a subscriber: Jdlrobson.

Edward, if this fails QA please change priority back to unbreak now :)

Jdlrobson set the point value for this task to 0.Tue, Sep 19, 5:06 PM
Edtadros raised the priority of this task from Medium to Unbreak Now!.Wed, Sep 20, 1:38 AM

Test Result - Beta

Status: ❌ FAIL
Environment: beta
OS: macOS Ventura
Browser: Chrome
Device: MBA
Emulated Device:NA

Test Artifact(s):

QA Steps

Steps to Reproduce:

Set to Minerva
Ensure the Echo extension is enabled.
Click on the bell or tray icon in the user interface.
Upon clicking the bell or tray icon, a sliding drawer should appear on the right, displaying notifications.

As best as I can tell this is on the desktop Minerva skin. That's the only way I get the tray icon. When in desktop a drawer doesn't open. When in mobile, I can only confirm that notifications opens up a drawer. See the desktop behavior below.

screenshot 22.mov.gif (822×1 px, 734 KB)

Jdlrobson lowered the priority of this task from Unbreak Now! to Medium.EditedWed, Sep 20, 2:01 AM

That's an unrelated documented bug. Will share URL tomorrow with better QA steps (have confirmed mobile notification button works for me)

Edtadros added a subscriber: Edtadros.

Test Result - Beta

Status: ✅ PASS
Environment: beta
OS: macOS Ventura
Browser: Chrome
Device: MBA
Emulated Device:NA

Test Artifact(s):

QA Steps

Switch to mobile site (Minerva skin) e.g. https://en.m.wikipedia.beta.wmflabs.org/
Ensure the Echo extension is enabled.

screenshot.png (852×1 px, 225 KB)

Click on the bell icon in the user interface.
✅ AC1: Upon clicking the bell icon, a sliding drawer should appear on the right, displaying notifications.
screenshot 30.mov.gif (850×1 px, 800 KB)