Commit d9f7ae79 authored by Justin Ho's avatar Justin Ho

Fix visibility icon on project overview page

- Change `vertical-align: text-bottom` to `-3px` which
works better with text that has larger font size
- Fix `visibility_level_icon` to properly accept the `class`
property
- Fix visibility icon on project overview page
parent b87fd999
...@@ -54,6 +54,6 @@ svg { ...@@ -54,6 +54,6 @@ svg {
} }
&.s16 { &.s16 {
vertical-align: text-bottom; vertical-align: -3px;
} }
} }
...@@ -159,7 +159,7 @@ ...@@ -159,7 +159,7 @@
font-weight: bold; font-weight: bold;
.icon { .icon {
font-size: $gl-font-size-large; vertical-align: -1px;
} }
.home-panel-topic-list { .home-panel-topic-list {
......
...@@ -117,7 +117,9 @@ module IconsHelper ...@@ -117,7 +117,9 @@ module IconsHelper
'earth' 'earth'
end end
sprite_icon(name, size: DEFAULT_ICON_SIZE, css_class: 'gl-vertical-align-text-bottom') css_class = options.delete(:class)
sprite_icon(name, size: DEFAULT_ICON_SIZE, css_class: css_class)
end end
def file_type_icon_class(type, mode, name) def file_type_icon_class(type, mode, name)
......
...@@ -13,7 +13,7 @@ ...@@ -13,7 +13,7 @@
%h1.home-panel-title.gl-mt-3.gl-mb-2{ data: { qa_selector: 'project_name_content' } } %h1.home-panel-title.gl-mt-3.gl-mb-2{ data: { qa_selector: 'project_name_content' } }
= @project.name = @project.name
%span.visibility-icon.text-secondary.gl-ml-2.has-tooltip{ data: { container: 'body' }, title: visibility_icon_description(@project) } %span.visibility-icon.text-secondary.gl-ml-2.has-tooltip{ data: { container: 'body' }, title: visibility_icon_description(@project) }
= visibility_level_icon(@project.visibility_level, options: {class: 'icon'}) = visibility_level_icon(@project.visibility_level, options: { class: 'icon' })
= render_if_exists 'compliance_management/compliance_framework/compliance_framework_badge', project: @project = render_if_exists 'compliance_management/compliance_framework/compliance_framework_badge', project: @project
.home-panel-metadata.d-flex.flex-wrap.text-secondary .home-panel-metadata.d-flex.flex-wrap.text-secondary
- if can?(current_user, :read_project, @project) - if can?(current_user, :read_project, @project)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment