Skip to content

Remove prepend-top-5 CSS class

Jose Ivan Vargas requested to merge jivanvl-remove-prepend-top-5 into master

What does this MR do?

This removes the .prepend-top-5 css class and replaces its usage for GitLab UI, gl-mt-2 utility class

This is due to the rem values of gl-mt-2, 0.5 * 8px = 4px

Since this rule does not comply with our design system guidelines, I "floored" the measurement to the closest one that did comply

#217418

File list

File converted
app/views/snippets/_actions.html.haml
app/views/projects/snippets/_actions.html.haml
app/views/projects/wikis/_form.html.haml
app/views/projects/diffs/_stats.html.haml
app/views/admin/application_settings/_account_and_limit.html.haml
app/views/shared/web_hooks/_hook.html.haml
app/views/shared/issuable/form/_contribution.html.haml
app/views/shared/deploy_tokens/_new_deploy_token.html.haml
app/views/shared/milestones/_form_dates.html.haml
app/views/groups/labels/index.html.haml
app/views/profiles/show.html.haml
app/assets/javascripts/ide/components/ide_review.vue
app/assets/javascripts/vue_shared/components/file_finder/item.vue
app/assets/javascripts/reports/accessibility_report/components/accessibility_issue_body.vue
app/assets/javascripts/reports/components/test_issue_body.vue
app/assets/stylesheets/framework/common.scss
ee/app/assets/javascripts/related_issues/components/related_issues_list.vue
ee/app/assets/javascripts/subscriptions/new/components/order_summary.vue
ee/app/assets/javascripts/vue_merge_request_widget/components/codequality_issue_body.vue
ee/app/assets/javascripts/vue_merge_request_widget/components/performance_issue_body.vue
ee/app/assets/javascripts/billings/components/subscription_table_row.vue
ee/app/assets/javascripts/vue_shared/security_reports/components/vulnerability_details.vue
ee/app/assets/javascripts/vue_shared/security_reports/components/security_issue_body.vue

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Edited by Jose Ivan Vargas

Merge request reports