From 54cb0ecefee3bac6df3a4c3ee871dcffa8b6eedb Mon Sep 17 00:00:00 2001
From: Vaidehi Joshi <vaidehi.sj@gmail.com>
Date: Wed, 14 Oct 2020 15:04:36 -0700
Subject: [PATCH] Fix required button styling in SiteConfig (#10857) [deploy]

* Use crayons-indicator--critical class when rendering Required note

* Add a bit of padding to the site-config label to give button room
---
 app/assets/stylesheets/admin.scss | 1 +
 app/helpers/application_helper.rb | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/app/assets/stylesheets/admin.scss b/app/assets/stylesheets/admin.scss
index 27d64db9b..a8d793084 100644
--- a/app/assets/stylesheets/admin.scss
+++ b/app/assets/stylesheets/admin.scss
@@ -56,6 +56,7 @@ label {
 .site-config__label {
   display: flex;
   justify-content: space-between;
+  padding-bottom: var(--su-2);
 }
 
 .profile__group-toggle {
diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
index 60d62675c..0582d9c72 100644
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -285,7 +285,7 @@ module ApplicationHelper
   def admin_config_label(method, content = nil)
     content ||= raw("<span>#{method.to_s.humanize}</span>")
     if method.to_sym.in?(VerifySetupCompleted::MANDATORY_CONFIGS)
-      content = safe_join([content, raw("<span class='crayons-indicator'>Required</span>")])
+      content = safe_join([content, raw("<span class='crayons-indicator crayons-indicator--critical'>Required</span>")])
     end
 
     tag.label(content, class: "site-config__label crayons-field__label", for: "site_config_#{method}")