Skip to content

Resolve GlAvatar size in MR approvals summary

Jiaan Louw requested to merge 356453-fix-gl-avatar-size-in-approvals-summary into master

What does this MR do and why?

Updates the avatar size from 20 to 24 to resolve #356453 (closed) as part of the UserAvatarList migration to GlAvatar &7767 (closed).

Screenshots or screen recordings

before + feature flag disabled before + feature flag enabled after + feature flag enabled
gdk.test_3000_h5bp_html5-boilerplate<span data-escaped-char>_</span>-<span data-escaped-char>_</span>merge_requests_1 gdk.test_3000_h5bp_html5-boilerplate<span data-escaped-char>_</span>-merge_requests_1__1 gdk.test_3000_h5bp_html5-boilerplate_-merge_requests_1__6

The footer is being updated in !83753 (merged).

How to set up and validate locally

  1. Enable the feature flag by running: echo "Feature.enable(:gl_avatar_for_all_user_avatars)" | rails c
  2. View a merge request at: [GDK_HOST]/h5bp/html5-boilerplate/-/merge_requests/1
  3. Approve the merge request if no approvers are listed.

MR acceptance checklist

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

Related to #356453 (closed).

Edited by Jiaan Louw

Merge request reports