Page MenuHomePhabricator

The watchlist menu extends beyond its area on mobile
Closed, ResolvedPublic1 Estimated Story PointsBUG REPORT

Description

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

What happens?:

image.png (418×388 px, 30 KB)

What should have happened instead?:
Buttons should be wrapped on a new line

Other information (browser name/version, screenshots, etc.):
iPhone 12 mini, latest OS, latest Chrome

QA Results - Beta

ACStatusDetails
1T358424#9645826

QA Results - Prod

ACStatusDetails
1T358424#9645832

Event Timeline

I assume this should be added to the list of pages where limited width does not apply @ovasileva @JScherer-WMF?

I assume this should be added to the list of pages where limited width does not apply @ovasileva @JScherer-WMF?

+1, I think for this limited width not applying makes sense. Also wanted to note that I'm having trouble reproducing. I'm seeing wrapping correctly on my side.

Screenshot 2024-02-27 at 10.51.36 AM.png (1×972 px, 274 KB)

@ovasileva that doesn't look like what you should be seeing on a mobile device - is that a screenshot of a resized screen?
I can replicate this on desktop at 1010px with both sidebar and main menu pinned to the right and left.

Screenshot 2024-02-27 at 9.46.19 AM.png (1×2 px, 410 KB)

Jdlrobson renamed this task from The watchlist menu extends beyond its area on mobile to [Config] Disable limited width on large pages. The watchlist menu extends beyond its area on mobile.Feb 29 2024, 4:28 PM
Jdlrobson set the point value for this task to 1.
Jdlrobson renamed this task from [Config] Disable limited width on large pages. The watchlist menu extends beyond its area on mobile to The watchlist menu extends beyond its area on mobile.Mar 11 2024, 7:17 PM

I misunderstood the work this task involves - the limited width for content is already disabled on watchlist. There is a limited width that applies to all pages that is causing issues here.

The issue here is the filters UI is not responsive. The label for the mark all as read button is exceptionally longer than other projects.

Did not get a chance to work on this today. Plan to look at it tomorrow.

Jdlrobson removed Edtadros as the assignee of this task.
Jdlrobson moved this task from Doing to QA on the Web-Team-Backlog (FY2023-24 Q3 Sprint 5) board.
Jdlrobson added a subscriber: Edtadros.

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

[mediawiki/core@master] Make watchlist more responsive

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

Change 1011004 merged by jenkins-bot:

[mediawiki/core@master] Make watchlist more responsive

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

Jdlrobson lowered the priority of this task from High to Medium.Mar 18 2024, 5:15 PM

Test Result - Beta

Status: ✅ PASS
Environment: rubeta
OS: macOS Sonoma
Browser: Chrome
Device: MBA
Emulated Device:NA

Test Artifact(s):

QA Steps

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

Open on mobile or with devtools https://ru.wikipedia.org/wiki/Служебная:Список_наблюдения?uselang=ru on beta https://ru.wikipedia.beta.wmflabs.org/wiki/%D0%91%D0%B5%D0%BB%D1%8B%D0%B9_%D0%BC%D0%B5%D0%B4%D0%B2%D0%B5%D0%B4%D1%8C
Look at filters
✅ AC1: Buttons should be wrapped on a new line

screenshot 6.mov.gif (792×1 px, 3 MB)

Test Result - Prod

Status: ✅ PASS
Environment: testwiki
OS: macOS Sonoma
Browser: Chrome
Device: MBA
Emulated Device:NA

Test Artifact(s):

QA Steps

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

Open on mobile or with devtools https://ru.wikipedia.org/wiki/Служебная:Список_наблюдения?uselang=ru on beta https://ru.wikipedia.beta.wmflabs.org/wiki/%D0%91%D0%B5%D0%BB%D1%8B%D0%B9_%D0%BC%D0%B5%D0%B4%D0%B2%D0%B5%D0%B4%D1%8C
Look at filters
✅ AC1: Buttons should be wrapped on a new line
tested the watchlist filters on testwiki with uselang=ru

screenshot 7.mov.gif (792×1 px, 3 MB)

Test Result - Beta

Status: ✅ PASS
Environment: rubeta
OS: macOS Sonoma
Browser: Chrome
Device: MBA
Emulated Device:NA

Test Artifact(s):

QA Steps

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

Open on mobile or with devtools https://ru.wikipedia.org/wiki/Служебная:Список_наблюдения?uselang=ru on beta https://ru.wikipedia.beta.wmflabs.org/wiki/%D0%91%D0%B5%D0%BB%D1%8B%D0%B9_%D0%BC%D0%B5%D0%B4%D0%B2%D0%B5%D0%B4%D1%8C
Look at filters
✅ AC1: Buttons should be wrapped on a new line

screenshot 6.mov.gif (792×1 px, 3 MB)

Just FYI: Win10, Chrome

image.png (483×1 px, 68 KB)

On iPhone - OK.

Just FYI: Win10, Chrome

I can replicate that on Chrome only between 1200-1237px. I think to address this a more significant refactor of the Watchlist design would be needed to move it away from table-based layout (and that would be out of scope for the web team - as this is not a product we maintain). If we wanted a quick fix we could make the header scrollable.

My feeling is this is acceptable for now, particularly given the sidebar can be collapsed.

Just FYI: Win10, Chrome

I can replicate that on Chrome only between 1200-1237px. I think to address this a more significant refactor of the Watchlist design would be needed to move it away from table-based layout (and that would be out of scope for the web team - as this is not a product we maintain). If we wanted a quick fix we could make the header scrollable.

My feeling is this is acceptable for now, particularly given the sidebar can be collapsed.

It also seems to me that this is not critical, this resolution is extremely rare. But this is a bug and it seems to me that it should be “remembered” for the future.

Just FYI: Win10, Chrome

I can replicate that on Chrome only between 1200-1237px. I think to address this a more significant refactor of the Watchlist design would be needed to move it away from table-based layout (and that would be out of scope for the web team - as this is not a product we maintain). If we wanted a quick fix we could make the header scrollable.

My feeling is this is acceptable for now, particularly given the sidebar can be collapsed.

It also seems to me that this is not critical, this resolution is extremely rare. But this is a bug and it seems to me that it should be “remembered” for the future.

Totally. @ovasileva let me know if you need help creating a new ticket capturing the "correct" way to address this. This would be a task for the Growth-Team

I’ve been wanting to ask for a long time, do we have any analytics for line length in different languages? :)

Do you mean recommendations for optimum line length?

Just FYI: Win10, Chrome

I can replicate that on Chrome only between 1200-1237px. I think to address this a more significant refactor of the Watchlist design would be needed to move it away from table-based layout (and that would be out of scope for the web team - as this is not a product we maintain). If we wanted a quick fix we could make the header scrollable.

My feeling is this is acceptable for now, particularly given the sidebar can be collapsed.

It also seems to me that this is not critical, this resolution is extremely rare. But this is a bug and it seems to me that it should be “remembered” for the future.

Totally. @ovasileva let me know if you need help creating a new ticket capturing the "correct" way to address this. This would be a task for the Growth-Team

I can't seem to replicate at that resolution. @Iniquity - do you have a screenshot?

Just FYI: Win10, Chrome

I can replicate that on Chrome only between 1200-1237px. I think to address this a more significant refactor of the Watchlist design would be needed to move it away from table-based layout (and that would be out of scope for the web team - as this is not a product we maintain). If we wanted a quick fix we could make the header scrollable.

My feeling is this is acceptable for now, particularly given the sidebar can be collapsed.

It also seems to me that this is not critical, this resolution is extremely rare. But this is a bug and it seems to me that it should be “remembered” for the future.

Totally. @ovasileva let me know if you need help creating a new ticket capturing the "correct" way to address this. This would be a task for the Growth-Team

I can't seem to replicate at that resolution. @Iniquity - do you have a screenshot?

image.png (896×1 px, 145 KB)

image.png (889×1 px, 113 KB)

I’ve been wanting to ask for a long time, do we have any analytics for line length in different languages? :)

Do you mean recommendations for optimum line length?

Yes, because this is not the first bug where the layout goes wrong due to long words :)