Page MenuHomePhabricator

Add a link: Toolbar border is missing on desktop
Closed, ResolvedPublicBUG REPORT

Description

List of steps to reproduce (step by step, including full links if applicable):

  • Enable link recommendations
  • Click on task card from Special:Homepage

What happens?:
The editing toolbar doesn't have a bottom border.

What should have happened instead?:
The editing toolbar should have a bottom border.

Software version (if not a Wikimedia wiki), browser information, screenshots, other information, etc:
MediaWiki 1.37.0-alpha
VisualEditor 0.1.2

Screen Shot 2021-09-10 at 9.49.23 AM.png (2×2 px, 1 MB)

Event Timeline

The toolbar has ve-ui-toolbar-empty class when it's not actually empty. VisualEditor recently updated styling for empty toolbar (no border, box-shadow). VE's determination is done in ve.ui.Toolbar.js via checking the length of toolbarGroups array. In the current implementation of Suggestions mode toolbar, we are initially this to an empty array and then setup the toolbar manually afterwards.

Change 720346 had a related patch set uploaded (by MewOphaswongse; author: MewOphaswongse):

[mediawiki/extensions/GrowthExperiments@master] Suggestions Mode: Set toolbarGroups so the toolbar isn't treated as empty

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

Change 720346 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] Suggestions Mode: Set toolbarGroups so the toolbar isn't treated as empty

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

Etonkovidova subscribed.

Checked in betalabs (ran cross-browser testing) and in wmf.23.