diff --git a/app/assets/javascripts/vue_merge_request_widget/components/review_app_link.vue b/app/assets/javascripts/vue_merge_request_widget/components/review_app_link.vue index 116bb2518319546af27d93de714e8907aed6e578..1af00c3509340e5c52c772d5fe27d2b74f62968d 100644 --- a/app/assets/javascripts/vue_merge_request_widget/components/review_app_link.vue +++ b/app/assets/javascripts/vue_merge_request_widget/components/review_app_link.vue @@ -43,6 +43,6 @@ export default { data-track-action="open_review_app" data-track-label="review_app" > - {{ display.text }} <gl-icon class="fgray" name="external-link" /> + {{ display.text }} <gl-icon class="gl-fill-gray-500" name="external-link" /> </gl-button> </template> diff --git a/app/assets/stylesheets/framework/common.scss b/app/assets/stylesheets/framework/common.scss index 75d15e207eaa2c91ca9d8d195ea834de72f1486c..829420128e14fd985018864acb7f416c79799374 100644 --- a/app/assets/stylesheets/framework/common.scss +++ b/app/assets/stylesheets/framework/common.scss @@ -76,9 +76,6 @@ .cgreen { color: $green-600; } .cdark { color: $gray-800; } -.fwhite { fill: $white; } -.fgray { fill: $gray-500; } - .text-plain, .text-plain:hover { color: $gl-text-color; diff --git a/app/views/users/_profile_sidebar.html.haml b/app/views/users/_profile_sidebar.html.haml index 1ae9afffd98b8df772d4e28a9e3c4b7e8a2886e5..43f2f91572004beb00ac7f7517d632c65461f8b8 100644 --- a/app/views/users/_profile_sidebar.html.haml +++ b/app/views/users/_profile_sidebar.html.haml @@ -30,21 +30,21 @@ %h2.gl-font-base.gl-mb-2.gl-mt-4= s_('UserProfile|Info') - if work_information(@user).present? .gl-display-flex.gl-gap-2.gl-mb-2 - = sprite_icon('work', css_class: 'fgray gl-mt-1 flex-shrink-0') + = sprite_icon('work', css_class: 'gl-fill-gray-500 gl-mt-1 flex-shrink-0') %span = work_information(@user, with_schema_markup: true) - if @user.location.present? .gl-display-flex.gl-gap-2.gl-mb-2 - = sprite_icon('location', css_class: 'fgray gl-mt-1 flex-shrink-0') + = sprite_icon('location', css_class: 'gl-fill-gray-500 gl-mt-1 flex-shrink-0') %span{ itemprop: 'addressLocality' } = @user.location - if user_local_time.present? .gl-display-flex.gl-gap-2.gl-mb-2{ data: { testid: 'user-local-time' } } - = sprite_icon('clock', css_class: 'fgray gl-mt-1 flex-shrink-0') + = sprite_icon('clock', css_class: 'gl-fill-gray-500 gl-mt-1 flex-shrink-0') %span = user_local_time .gl-display-flex.gl-gap-2.gl-mb-2 - = sprite_icon('calendar', css_class: 'fgray gl-mt-1 flex-shrink-0') + = sprite_icon('calendar', css_class: 'gl-fill-gray-500 gl-mt-1 flex-shrink-0') %span= s_('Member since %{date}') % { date: l(@user.created_at.to_date, format: :long) } - if @user.website_url.present? || display_public_email?(@user) || @user.skype.present? || @user.linkedin.present? || @user.twitter.present? || @user.mastodon.present? || @user.discord.present? @@ -52,29 +52,29 @@ %h2.gl-font-base.gl-mb-2.gl-mt-4= s_('UserProfile|Contact') - if @user.website_url.present? .gl-display-flex.gl-gap-2.gl-mb-2 - = sprite_icon('earth', css_class: 'fgray gl-mt-1 flex-shrink-0') + = sprite_icon('earth', css_class: 'gl-fill-gray-500 gl-mt-1 flex-shrink-0') = link_to @user.short_website_url, @user.full_website_url, class: 'gl-text-gray-900', target: '_blank', rel: 'me noopener noreferrer nofollow', itemprop: 'url' - if display_public_email?(@user) .gl-display-flex.gl-gap-2.gl-mb-2 - = sprite_icon('mail', css_class: 'fgray gl-mt-1 flex-shrink-0') + = sprite_icon('mail', css_class: 'gl-fill-gray-500 gl-mt-1 flex-shrink-0') = link_to @user.public_email, "mailto:#{@user.public_email}", class: 'gl-text-gray-900', itemprop: 'email' - if @user.skype.present? .gl-display-flex.gl-gap-2.gl-mb-2 - = sprite_icon('skype', css_class: 'fgray gl-mt-1 flex-shrink-0') + = sprite_icon('skype', css_class: 'gl-fill-gray-500 gl-mt-1 flex-shrink-0') = link_to @user.skype, "skype:#{@user.skype}", class: 'gl-text-gray-900', title: "Skype" - if @user.linkedin.present? .gl-display-flex.gl-gap-2.gl-mb-2 - = sprite_icon('linkedin', css_class: 'fgray gl-mt-1 flex-shrink-0') + = sprite_icon('linkedin', css_class: 'gl-fill-gray-500 gl-mt-1 flex-shrink-0') = link_to linkedin_name(@user), linkedin_url(@user), class: 'gl-text-gray-900', title: "LinkedIn", target: '_blank', rel: 'noopener noreferrer nofollow' - if @user.twitter.present? .gl-display-flex.gl-gap-2.gl-mb-2 - = sprite_icon('x', css_class: 'fgray gl-mt-1 flex-shrink-0') + = sprite_icon('x', css_class: 'gl-fill-gray-500 gl-mt-1 flex-shrink-0') = link_to @user.twitter, twitter_url(@user), class: 'gl-text-gray-900', title: _("X (formerly Twitter)"), target: '_blank', rel: 'noopener noreferrer nofollow' - if @user.mastodon.present? .gl-display-flex.gl-gap-2.gl-mb-2 - = sprite_icon('mastodon', css_class: 'fgray gl-mt-1 flex-shrink-0') + = sprite_icon('mastodon', css_class: 'gl-fill-gray-500 gl-mt-1 flex-shrink-0') = link_to @user.mastodon, mastodon_url(@user), class: 'gl-text-gray-900', title: "Mastodon", target: '_blank', rel: 'noopener noreferrer nofollow' - if @user.discord.present? .gl-display-flex.gl-gap-2.gl-mb-2 - = sprite_icon('discord', css_class: 'fgray gl-mt-1 flex-shrink-0') + = sprite_icon('discord', css_class: 'gl-fill-gray-500 gl-mt-1 flex-shrink-0') = link_to @user.discord, discord_url(@user), class: 'gl-text-gray-900', title: "Discord", target: '_blank', rel: 'noopener noreferrer nofollow' diff --git a/ee/spec/frontend/environments_dashboard/components/__snapshots__/environment_header_spec.js.snap b/ee/spec/frontend/environments_dashboard/components/__snapshots__/environment_header_spec.js.snap index 75b094f08de82ea91c26c4f6fd3d302d9c160294..f72e01c05795d8fd02f4efe34350547e1df00fee 100644 --- a/ee/spec/frontend/environments_dashboard/components/__snapshots__/environment_header_spec.js.snap +++ b/ee/spec/frontend/environments_dashboard/components/__snapshots__/environment_header_spec.js.snap @@ -90,7 +90,7 @@ exports[`Environment Header renders name and link to app matches the snapshot 1` View app <svg aria-hidden="true" - class="fgray gl-fill-current gl-icon s16" + class="gl-fill-current gl-fill-gray-500 gl-icon s16" data-testid="external-link-icon" role="img" >