Remove prepend-top-5 CSS class
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
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