From 31bc576b4d8ea1be857849b4a76a497e4e24c76a Mon Sep 17 00:00:00 2001
From: "Ernest W. Durbin III" <ewdurbin@gmail.com>
Date: Wed, 25 Nov 2020 08:40:22 -0500
Subject: [PATCH] fully handle css for active/package_only labels

handle the css class on the same tag to avoid conflicts
---
 static/sass/style.css                             | 6 +++---
 static/sass/style.scss                            | 5 ++++-
 templates/sponsors/sponsorship_benefits_form.html | 4 ++--
 3 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/static/sass/style.css b/static/sass/style.css
index b4775810c..3ca913fc7 100644
--- a/static/sass/style.css
+++ b/static/sass/style.css
@@ -3628,9 +3628,9 @@ span.highlighted {
   text-align: center; }
   #sponsorship_application_container form {
     margin-bottom: 0; }
-    #sponsorship_application_container form {
-      .package_only_label { color: #666666; }
-      .active { color: #3776ab; } }
+    #sponsorship_application_container form .package_only { color: #666666; }
+    #sponsorship_application_container form .package_only.active { color: #3776ab; }
+    #sponsorship_application_container form .active { color: #3776ab; }
   #sponsorship_application_container #package_selection {
     display: inline-block; }
     #sponsorship_application_container #package_selection li {
diff --git a/static/sass/style.scss b/static/sass/style.scss
index a837ff17d..0923fe3ba 100644
--- a/static/sass/style.scss
+++ b/static/sass/style.scss
@@ -2407,9 +2407,12 @@ span.highlighted {
     form {
         margin-bottom: 0;
 
-        .package_only_label {
+        .package_only {
             color: $grey;
         }
+        .package_only.active {
+            color: $blue;
+        }
         .active {
             color: $blue;
         }
diff --git a/templates/sponsors/sponsorship_benefits_form.html b/templates/sponsors/sponsorship_benefits_form.html
index cc4530ad4..f03cb585d 100644
--- a/templates/sponsors/sponsorship_benefits_form.html
+++ b/templates/sponsors/sponsorship_benefits_form.html
@@ -53,9 +53,9 @@ <h3 class="title">{{ field.label }}</h3>
             <ul id="id_{{ field.name }}">
             {% for benefit in field.field.queryset %}
                 <li class="{% cycle '' 'highlight' %}">
-                    <label for="id_{{field.name}}_{{ forloop.counter0 }}" benefit_id="{{ benefit.id }}">
+                    <label for="id_{{field.name}}_{{ forloop.counter0 }}" {% if benefit.package_only %} class='package_only'{% endif %} benefit_id="{{ benefit.id }}">
                         <input id="id_{{field.name}}_{{ forloop.counter0 }}" name="{{ field.name }}" type="checkbox" value="{{ benefit.id }}" {% if benefit.unavailability_message %}disabled{% endif %} {% if benefit.id in field.initial %}checked{% endif %} {% if benefit.package_only %}package_only='true'{% endif %}>
-                        <span {% if benefit.package_only %} class='package_only_label'{% endif %}>{{ benefit.name }}</span>
+                        <span>{{ benefit.name }}</span>
                         {% if benefit.description %}<i class="fa fa-info" title="{{ benefit.description }}"></i>{% endif %}
                         {% if benefit.package_only %}<i class="fa fa-cubes"  title="{{ benefit_model.PACKAGE_ONLY_MESSAGE }}"></i>{% endif %}
                         {% if not benefit.has_capacity %}<i class="fa fa-close"  title="{{ benefit_model.NO_CAPACITY_MESSAGE }}"></i>{% endif %}