Adapt user avatar in diff_discussion_header to mobile view
requested to merge 360679-adapt-migrated-user_avatar_link-in-app-assets-javascripts-notes-components into master
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)
Screenshots or screen recordings
desktop | mobile |
---|---|
How to set up and validate locally
- On your local instance open an (existing) MR, switch to "changes" tab and add a comment to some code line.
- Switch to "Overview" tab and note how your comment looks like
- 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.
-
I have evaluated the MR acceptance checklist for this MR.
Edited by Kos Palchyk