From 9bd91ed60a85a743ea4738a6330249a81de9750a Mon Sep 17 00:00:00 2001 From: Chris Hill-Scott Date: Mon, 19 Nov 2018 15:18:15 +0000 Subject: [PATCH] Style spacing for sticky footers MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sticky footers need slightly different spacing to sticky headers so they look balanced. They can’t just inherit the same spacing. --- .../components/stick-at-top-when-scrolling.scss | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/app/assets/stylesheets/components/stick-at-top-when-scrolling.scss b/app/assets/stylesheets/components/stick-at-top-when-scrolling.scss index 71662c80c..ec75473d3 100644 --- a/app/assets/stylesheets/components/stick-at-top-when-scrolling.scss +++ b/app/assets/stylesheets/components/stick-at-top-when-scrolling.scss @@ -40,7 +40,15 @@ .js-stick-at-bottom-when-scrolling { + $vertical-padding: $gutter * 2 / 3; + transition: bottom 0.1s ease-out, box-shadow 1s ease-in-out; + padding: $vertical-padding 0 $vertical-padding $gutter-half; + margin-top: -$vertical-padding; + + .page-footer { + margin-bottom: 0; + } }