Page MenuHomePhabricator

Notifications counter partially hidden
Closed, ResolvedPublic2 Estimated Story PointsBUG REPORT

Description

Hi. Last week notifications icon code was changed. There are a couple of problems aftermath. The worst is that you can't see any more the counter if it's more than 9.

  • Have at least 10 notifications on the same wiki.
  • Open some wikipage with safemode.
  • Make sure you can see the correct counter.
  • Click on the notifications icon.
  • Expected: The same number remains.
  • Got: You can see the msd only.

Observed on Samsung Internet. Thank you.

Screenshot_20230511_220604_Samsung Internet.jpg (293×1 px, 87 KB)

QA Results - Beta

ACStatusDetails
1T336543#8890754

Event Timeline

LGoto set the point value for this task to 2.May 15 2023, 5:24 PM

Change 923388 had a related patch set uploaded (by Bernard Wang; author: Bernard Wang):

[mediawiki/skins/Vector@master] Ensure echo notifications counter is always visible

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

Change 923388 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Ensure echo notifications counter is always visible

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

Edtadros subscribed.

Test Result - Beta

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

Test Artifact(s):

QA Steps

Have at least 10 notifications on the same wiki.
Open some wikipage with safemode.
Make sure you can see the correct counter.
Click on the notifications icon.
✅ AC1: Expected: The same number remains.

Screen Recording 2023-05-30 at 4.54.12 PM.mov.gif (754×1 px, 1 MB)