Add CSS class for UserAvatarInline rendered with `GlAvatar`
What does this MR do and why?
Pre-history: The feature flag gl_avatar_for_all_user_avatars was introduced in !81437 (merged) to migrate all avatars implementations to GlAvatar. The FF and related code will be deleted when all instances will be checked with FF enabled. This specific MR was targeted to prepare UserAvatarList to use GlAvatar in its implementation. I hoped to use GlUserAvatarsInline component but it lacks some functionality. So UserAvatarList component still uses UserAvatarLink (as before) which relies on UserAvatarImage which internally uses GlAvatar.
So what does this MR do?
Most of the work was done in 2 previous MRs !81437 (merged) && !82736 (merged). UserAvatarList can be switched to using GlAvatar by enabling the feature flag gl_avatar_for_all_user_avatars. We only need some spacing between the avatars so the CSS class is added for that behind the FF.
Screenshots or screen recordings
| With FF (avatar size adhusted to be valid 24px) | Without FF |
|---|---|
 |
 |
How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.
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.