From e14f96dabb9d7d7c6cfb459043b1e7c9ee120140 Mon Sep 17 00:00:00 2001 From: Soheb Date: Sun, 31 Mar 2024 12:01:40 +0100 Subject: [PATCH] Variable reuse for min-touch-target --- app/static/structure.css | 17 ++++++----------- app/static/structure.scss | 12 ++++++------ 2 files changed, 12 insertions(+), 17 deletions(-) diff --git a/app/static/structure.css b/app/static/structure.css index b5ca6b51..4609b480 100644 --- a/app/static/structure.css +++ b/app/static/structure.css @@ -337,8 +337,7 @@ h1 .fe-bell, h1 .fe-no-bell { left: 1px; } .fe-poll:before { - content: "\e91b"; - /* possibly e985 */ + content: "\e91b"; /* possibly e985 */ } .fe-sticky-left::before { @@ -475,18 +474,14 @@ fieldset legend { .skip-link { position: absolute; - top: -40px; - /* Adjust as needed to hide the link off-screen */ + top: -40px; /* Adjust as needed to hide the link off-screen */ left: 0; - background-color: #fff; - /* Background color to cover the link for screen readers */ - z-index: 1060; - /* Ensure it's above other content */ + background-color: #fff; /* Background color to cover the link for screen readers */ + z-index: 1060; /* Ensure it's above other content */ } .skip-link:focus { - top: 0; - /* Bring the link back into view when it receives focus */ + top: 0; /* Bring the link back into view when it receives focus */ } .form-group { @@ -1164,7 +1159,7 @@ fieldset legend { padding: 5px 15px; } .comment .replies { - margin-top: 0px; + margin-top: 0; border-left: solid 1px #ddd; border-top: solid 1px #ddd; } diff --git a/app/static/structure.scss b/app/static/structure.scss index d820c798..6ae37986 100644 --- a/app/static/structure.scss +++ b/app/static/structure.scss @@ -268,8 +268,8 @@ html { .utilities_row { a { display: inline-block; - min-width: 44px; - min-height:44px; + min-width: $min-touch-target; + min-height: $min-touch-target; justify-content: center; align-content: center; } @@ -344,12 +344,12 @@ html { .post_utilities_bar { display: flex; - min-height:44px; + min-height: $min-touch-target; .post_options_link { display: flex; - width: 44px; - height: 44px; + width: $min-touch-target; + height: $min-touch-target; padding: 3px; justify-content: center; align-items: center; @@ -831,7 +831,7 @@ html { } .replies { - margin-top: 0px; + margin-top: 0; border-left: solid 1px $light-grey; border-top: solid 1px $light-grey; }