Page MenuHomePhabricator

EditCheck: Fix issues in touchable devices
Closed, ResolvedPublic

Description

Background

There are the following issues in the Mid-edit in touchable devices (tablet, mobile):

  1. Text size on mobile/tablet is currently too small within the bottom sheet. We need to update it from 14px to 16px.
  2. When the Edit Check appears in Mid-wdit on touchable screens, it’s difficult to recognize that there is something to fix. We could fix it by displaying the bottom sheet by default when the Edit Check appears.
  3. At the moment, when the user collapses the bottom sheet once, it remains collapsed when selecting the next Edit Check, making it hard to find since it's collapsed at the bottom. We could expand the bottom sheet every time the user clicks on an Edit Check (either the status icon or the highlighted text).
  4. There are issues with the scroll when the bottom sheet is displayed, and it’s not possible to scroll until the end. This is not happening on iPad.
  5. The line indicator does not cover all the highlighted text when the pasted check is long on mobile.

Acceptance criteria (or Done)

  • Update text size in bottom sheet from 14px to 16px
    • Done
  • Display the bottom sheet by default when the Edit Check appears
    • We shouldn't this per comments below
  • Expand the bottom sheet every time the user clicks on an Edit Check (either the status icon or the highlighted text)
    • We shouldn't this per comments below
  • Solve the blocked scroll on mobile
  • Fix the vertical line indicator to make it cover all the highlighted text
    • Done

Related Objects

Event Timeline

Aklapper renamed this task from EditChek: Fix issues in touchable devices to EditCheck: Fix issues in touchable devices.Jun 17 2025, 11:17 PM

When the Edit Check appears in Mid-wdit on touchable screens, it’s difficult to recognize that there is something to fix. We could fix it by displaying the bottom sheet by default when the Edit Check appears.

We don't do this because mid-edit checks appear while you're typing, and so we cannot show a check without dismissing your keyboard and interrupting your typing. This is incredibly annoying when it happens unexpectedly, so we decided to show checks subtly, and then you'll be shown them all pre-save if you don't address them.

Change #1161523 had a related patch set uploaded (by Esanders; author: Esanders):

[mediawiki/extensions/VisualEditor@master] EditCheck: Fix font size resets

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

Change #1161523 merged by jenkins-bot:

[mediawiki/extensions/VisualEditor@master] EditCheck: Fix font size resets

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

Esanders updated the task description. (Show Details)

Change #1163805 had a related patch set uploaded (by Esanders; author: Esanders):

[mediawiki/extensions/VisualEditor@master] EditCheck mobile: Compute bounding rect for multi-selection checks

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

Change #1163805 merged by jenkins-bot:

[mediawiki/extensions/VisualEditor@master] EditCheck mobile: Compute bounding rect for multi-selection checks

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

The blocked scroll part remains to be addressed, and we should do that.

I'm having trouble with reproducing the blocked scroll bug.

There are issues with the scroll when the bottom sheet is displayed, and it’s not possible to scroll until the end.

I've loaded this up on iOS safari, Android Chrome and in responsive mode in Firefox and Chrome, and I've tried it on my local wiki with latest ve, and on an enwiki page.

I chose a section, opened mobile editor, clicked on a "citation needed" suggestion, and scrolled down to the bottom. The bottom panel did not obscure the text. Example here from https://en.wikipedia.org/wiki/Coffeehouse?ecenable=2#/editor/2

Screenshot 2026-01-13 at 17.00.35.png (898×922 px, 140 KB)

Am I looking at the wrong panel?

zoe removed zoe as the assignee of this task.Jan 13 2026, 6:29 PM
El T397170#11517958, @zoe escribió:

I'm having trouble with reproducing the blocked scroll bug.

There are issues with the scroll when the bottom sheet is displayed, and it’s not possible to scroll until the end.

@zoe I've reproduced this on my Android and iOS and I cannot find any issue. Since this task was created some time ago, maybe this was a past issue that was solved in some moment. So let's ignore this for now.

DLynch moved this task from Ready to Be Worked On to QA on the Editing-team (Kanban Board) board.
DLynch added a project: Editing QA.

In that case, I think we have done everything that remains on the task, so I will pass it over to QA.

For item 2:

When the Edit Check appears in Mid-edit on touchable screens, it’s difficult to recognize that there is something to fix. We could fix it by displaying the bottom sheet by default when the Edit Check appears.

For this case, it appears for pastecheck but not tone check.

Unrelated to ticket description, I see that we have extra spacing when EditCheck is enabled.

No ecenable passed

image.png (1×780 px, 205 KB)

ecenable=2

No checks yet

image.png (1×782 px, 180 KB)

Check. At this point, space goes back to normal

image.png (1×772 px, 172 KB)

That's deliberate. When checks are enabled, the content is displayed slightly narrower so that there's space for the gutter to appear without causing the content to reflow.