From c3047a903d5a8d34cb92828e7a77daf86c65fe10 Mon Sep 17 00:00:00 2001 From: Kris Date: Tue, 2 Apr 2019 17:22:22 -0400 Subject: [PATCH] UX: Reviewable spacing adjustments --- .../stylesheets/common/base/reviewables.scss | 18 ++++++++---------- app/assets/stylesheets/mobile/reviewables.scss | 5 ----- 2 files changed, 8 insertions(+), 15 deletions(-) diff --git a/app/assets/stylesheets/common/base/reviewables.scss b/app/assets/stylesheets/common/base/reviewables.scss index 9e8b0d1e191..07238854dfe 100644 --- a/app/assets/stylesheets/common/base/reviewables.scss +++ b/app/assets/stylesheets/common/base/reviewables.scss @@ -116,18 +116,18 @@ .reviewable-item { margin-bottom: 1em; - border-bottom: 1px solid dark-light-choose($primary-low, $secondary-low); + padding-top: 2em; + border-top: 1px solid dark-light-choose($primary-low, $secondary-low); .topic-statuses { font-size: $font-up-2; - line-height: $line-height-medium; } .reviewable-meta-data { color: dark-light-choose($primary-medium, $secondary-medium); display: flex; width: 100%; - margin-bottom: 0.15em; + margin-bottom: 0.5em; font-size: $font-down-1; align-items: baseline; .reviewable-type { @@ -146,6 +146,9 @@ .status { color: dark-light-choose($primary-high, $secondary-high); } + .reply-count { + margin-right: 1em; + } } .reviewable-contents { @@ -215,7 +218,7 @@ } > tr > th, > tr > td { - padding: 0.25em; + padding: 0.5em; } } } @@ -261,10 +264,10 @@ } .post-topic { width: 100%; - font-weight: bold; color: $primary-medium; margin-bottom: 0.75em; .title-text { + font-weight: bold; color: $primary-high; display: block; font-size: $font-up-2; @@ -275,11 +278,6 @@ display: none; } } - .reply-count { - font-size: $font-down-1; - font-weight: normal; - margin-right: 1em; - } } } diff --git a/app/assets/stylesheets/mobile/reviewables.scss b/app/assets/stylesheets/mobile/reviewables.scss index 50dce6a4d49..2e08bf39a9b 100644 --- a/app/assets/stylesheets/mobile/reviewables.scss +++ b/app/assets/stylesheets/mobile/reviewables.scss @@ -77,9 +77,4 @@ .reviewable-scores-and-history table { width: 100%; - table-layout: fixed; - td { - display: block; - white-space: nowrap; - } }