Page MenuHomePhabricator

Design review the visual enhancement and layout changes
Closed, ResolvedPublic

Description

The release beginning the week of 25th sees various visual changes to Vector. With this in mind I think it makes sense to review all the changes together. All changes are live on the beta cluster.

To aid with this Pixel gives a visual regression summary of all the changes made this week:
https://pixel.wmcloud.org/reports/T313562/index.html

Please review all the changes and move this to sign off if everything looks good.

Expected changes

  • Search: Introduction of Codex (Bug: T300573, Bug: T302137, Bug: T303558, Bug: T309722, Bug: T310525)
  • Tabs: Changes to the language variant dropdown position ( Bug: T309223, Bug: T308344, Bug: T312212) and chevron (T308344, T310838)
  • Link: External link icon change (phab:T261391)
    • Footer, TOC: The new grid was rolled out resulting in changes to the footer and table of contents alignment (phab:T312241)
  • TOC: Changes to table of contents text alignment and padding (T312156)

Event Timeline

Jdlrobson updated the task description. (Show Details)
Jdlrobson added a subscriber: alexhollender_WMF.

The next release is quite risky from a design perspective as there have been a lot of design changes! Pixel is flagging 71/90 failures for test cases. I’ve done my best to describe the changes going out here @alexhollender_WMF I realize this is not our usual design process, but I think it makes sense given the unique situation we're in this week. Let me know if you have any questions about the next release.

@Jdlrobson - unsure if this is related to the grid work, but seeing an overlap along the footer of the page:

Screen Shot 2022-07-22 at 9.43.20 AM.png (630×2 px, 303 KB)

@Jdlrobson - unsure if this is related to the grid work, but seeing an overlap along the footer of the page:

Screen Shot 2022-07-22 at 9.43.20 AM.png (630×2 px, 303 KB)

This is T313060

@Jdlrobson I'm having a difficult time understanding the pixel results.

Per standup there's a bug where two different image sizes are compared. Please review these manually.

i've reviewed the pixel screenshots and don't have any concerns