Skip to content

Adapt user avatar in diff_discussion_header to mobile view

What does this MR do and why?

Fixes #360679 (closed), a follow up after !84937 (comment 923098678)

Adds adaptive sizes for avatar in app/assets/javascripts/notes/components/diff_discussion_header.vue

And removes the reliance on UserAvatarLink as per &7731 (closed) 👣 # 5

Screenshots or screen recordings

desktop mobile
localhost_3000_h5bp_html5-boilerplate_-_merge_requests_4 Screenshot_2022-07-20_at_20.47.58

How to set up and validate locally

  1. On your local instance open an (existing) MR, switch to "changes" tab and add a comment to some code line.
  2. Switch to "Overview" tab and note how your comment looks like
  3. Try switching mobile view on and off

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 Kos Palchyk

Merge request reports