From 2de126421388e4b798685181015b9e0673cf7ef0 Mon Sep 17 00:00:00 2001 From: Kris Date: Fri, 19 Jan 2018 16:16:22 -0500 Subject: [PATCH] UX: Adjustments for embedded reply collapse button --- app/assets/stylesheets/desktop/topic-post.scss | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/app/assets/stylesheets/desktop/topic-post.scss b/app/assets/stylesheets/desktop/topic-post.scss index 938dee54a94..9df9a80fbe6 100644 --- a/app/assets/stylesheets/desktop/topic-post.scss +++ b/app/assets/stylesheets/desktop/topic-post.scss @@ -204,14 +204,15 @@ nav.post-controls { .embedded-posts { h1, h2, h3 { margin: 10px 0; } - + border: 1px solid $primary-low; + .topic-body { box-sizing: border-box; width: calc(100% - 70px); // [100% - .topic-avatar width] // WARNING: overflow hide is required for quoted / embedded images // which expect "normal" post width, but expansions are narrower overflow: hidden; - padding: 15px 15px 0 15px; + padding: 15px 25px 0 15px; } // this is covered by .topic-body .regular on a normal post @@ -225,10 +226,6 @@ nav.post-controls { padding-top: 15px; } - .row { - border: 1px solid $primary-low; - } - .collapse-down, .collapse-up { position: absolute; color: $primary-medium; @@ -244,6 +241,8 @@ nav.post-controls { // bottom means "reply expansion" below a post &.bottom { + border-top: none; + margin-bottom: 20px; &.hidden { display: block; opacity: 0; @@ -255,8 +254,6 @@ nav.post-controls { } .row { padding-bottom: .5em; - border-top: none; - border-bottom: none; .topic-avatar, .topic-body { border-top: 1px solid $primary-low; } @@ -265,11 +262,12 @@ nav.post-controls { // top means "in reply to expansion" above a post &.top { + border-bottom: none; .collapse-down { transform: translate(-50%, 55%); } margin-left: 56px; - width: 701px; + width: 699px; .row { border-bottom: none; .topic-avatar, .topic-body {