Follow-up from "Restyle changes header & file tree"
The following discussion from !23665 (merged) should be addressed:
-
@pslaughter started a discussion: suggestion (non-blocking): I'm 90% certain we can actually just remove our usage of
content-blockhere. How would you feel about creating a follow up issue for this?diff --git a/app/assets/javascripts/diffs/components/compare_versions.vue b/app/assets/javascripts/diffs/components/compare_versions.vue index 63ce43a193d..6c0ec62b2bc 100644 --- a/app/assets/javascripts/diffs/components/compare_versions.vue +++ b/app/assets/javascripts/diffs/components/compare_versions.vue @@ -89,7 +89,7 @@ export default { <template> <div class="mr-version-controls border-top"> <div - class="mr-version-menus-container content-block" + class="mr-version-menus-container" :class="{ [CENTERED_LIMITED_CONTAINER_CLASSES]: isLimitedContainer, }" diff --git a/app/assets/stylesheets/pages/merge_requests.scss b/app/assets/stylesheets/pages/merge_requests.scss index 84daec4fb43..c0a6d385baf 100644 --- a/app/assets/stylesheets/pages/merge_requests.scss +++ b/app/assets/stylesheets/pages/merge_requests.scss @@ -731,11 +731,6 @@ } } - .content-block { - padding: $gl-padding; - border-bottom: 0; - } - .comments-disabled-notif { line-height: 28px;