Skip to content

Fix avatar size for issue and MR placeholder notes

Peter Hegman requested to merge 355855-check-that-user_avatar_link-in-app-assets-javascripts-vue_shared-components-notes into master

What does this MR do and why?

Related to #355855 (closed)

The avatar in the placeholder note component was using an unsupported size. This MR updates that component to use supported sizes. Styling should match what was done in !86252 (merged)

Screenshots or screen recordings

Desktop

MR/issue comment

Before After
Screen_Shot_2022-06-30_at_11.52.48_AM Screen_Shot_2022-06-30_at_11.43.18_AM

Diff comment

Before After
Screen_Shot_2022-06-30_at_11.57.26_AM Screen_Shot_2022-06-30_at_11.56.38_AM

Mobile

MR/issue comment

Before After
Screen_Shot_2022-06-30_at_12.01.15_PM Screen_Shot_2022-06-30_at_12.03.23_PM

Diff comment

Before After
Screen_Shot_2022-06-30_at_12.02.17_PM Screen_Shot_2022-06-30_at_12.03.58_PM

How to set up and validate locally

  1. Navigate to an issue or MR
  2. Open Chrome dev tools and navigate to the Network tab
  3. From the No throttling dropdown select Add
  4. Click Add custom profile and enter 1 for Download and Upload.
  5. Choose your custom throttling from the dropdown
  6. Add a comment

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Peter Hegman

Merge request reports