Refactor(reCAPTCHA): remove icons (#2564)

cal-itp · Dec 5, 2024 · ea9f790 · ea9f790
2 parents b58eb16 + c3f63cc
commit ea9f790
Show file tree

Hide file tree

Showing 23 changed files with 15 additions and 71 deletions.
diff --git a/benefits/core/templates/core/includes/media-item.html b/benefits/core/templates/core/includes/media-item.html
@@ -1,8 +1,4 @@
 <li class="media d-flex align-items-stretch w-auto">
-    <div class="media-line">
-        {% block icon %}
-        {% endblock icon %}
-    </div>
     <div class="media-body">
         {% block heading_wrapper %}
             <{{ heading_tag|default:"h3" }} class="media-body--heading h3">

diff --git a/...des/media-item--bankcardcheck--start.html → .../media-item--contactless-card--start.html b/...des/media-item--bankcardcheck--start.html → .../media-item--contactless-card--start.html
@@ -1,10 +1,6 @@
 {% extends "core/includes/media-item.html" %}
 {% load i18n %}
 
-{% block icon %}
-    {% include "core/includes/icon.html" with name="bankcardcheck" %}
-{% endblock icon %}
-
 {% block heading %}
     {% translate "Your contactless card details" %}
 {% endblock heading %}

diff --git a/benefits/eligibility/templates/eligibility/includes/media-item--idcardcheck.html b/benefits/eligibility/templates/eligibility/includes/media-item--idcardcheck.html
diff --git a/...-idcardcheck--start--cst-agency-card.html → ...entification--start--cst-agency-card.html b/...-idcardcheck--start--cst-agency-card.html → ...entification--start--cst-agency-card.html
@@ -1,4 +1,4 @@
-{% extends "eligibility/includes/media-item--idcardcheck.html" %}
+{% extends "core/includes/media-item.html" %}
 
 {% load i18n %}
 

diff --git a/...-item--idcardcheck--start--login-gov.html → ...em--identification--start--login-gov.html b/...-item--idcardcheck--start--login-gov.html → ...em--identification--start--login-gov.html
@@ -1,4 +1,4 @@
-{% extends "eligibility/includes/media-item--idcardcheck.html" %}
+{% extends "core/includes/media-item.html" %}
 
 {% load i18n %}
 

diff --git a/...a-item--idcardcheck--start--medicare.html → ...tem--identification--start--medicare.html b/...a-item--idcardcheck--start--medicare.html → ...tem--identification--start--medicare.html
@@ -1,4 +1,4 @@
-{% extends "eligibility/includes/media-item--idcardcheck.html" %}
+{% extends "core/includes/media-item.html" %}
 
 {% load i18n %}
 

diff --git a/...-idcardcheck--start--mst-agency-card.html → ...entification--start--mst-agency-card.html b/...-idcardcheck--start--mst-agency-card.html → ...entification--start--mst-agency-card.html
@@ -1,4 +1,4 @@
-{% extends "eligibility/includes/media-item--idcardcheck.html" %}
+{% extends "core/includes/media-item.html" %}
 
 {% load i18n %}
 

diff --git a/...dcardcheck--start--sbmtd-agency-card.html → ...tification--start--sbmtd-agency-card.html b/...dcardcheck--start--sbmtd-agency-card.html → ...tification--start--sbmtd-agency-card.html
@@ -1,4 +1,4 @@
-{% extends "eligibility/includes/media-item--idcardcheck.html" %}
+{% extends "core/includes/media-item.html" %}
 
 {% load i18n %}
 

diff --git a/benefits/eligibility/templates/eligibility/start--calfresh.html b/benefits/eligibility/templates/eligibility/start--calfresh.html
@@ -12,7 +12,7 @@ <h1>{% translate "You selected a CalFresh Cardholder transit benefit." %}</h1>
 {% endblock headline %}
 
 {% block media-item %}
-  {% include "eligibility/includes/media-item--idcardcheck--start--login-gov.html" %}
+  {% include "eligibility/includes/media-item--identification--start--login-gov.html" %}
 {% endblock media-item %}
 
 {% block call-to-action %}

diff --git a/benefits/eligibility/templates/eligibility/start--cst-agency-card.html b/benefits/eligibility/templates/eligibility/start--cst-agency-card.html
@@ -12,7 +12,7 @@ <h1>{% translate "You selected an Agency Card transit benefit." %}</h1>
 {% endblock headline %}
 
 {% block media-item %}
-  {% include "eligibility/includes/media-item--idcardcheck--start--cst-agency-card.html" %}
+  {% include "eligibility/includes/media-item--identification--start--cst-agency-card.html" %}
 {% endblock media-item %}
 
 {% block call-to-action %}

diff --git a/benefits/eligibility/templates/eligibility/start--medicare.html b/benefits/eligibility/templates/eligibility/start--medicare.html
@@ -12,7 +12,7 @@ <h1>{% translate "You selected a Medicare Cardholder transit benefit." %}</h1>
 {% endblock headline %}
 
 {% block media-item %}
-    {% include "eligibility/includes/media-item--idcardcheck--start--medicare.html" %}
+    {% include "eligibility/includes/media-item--identification--start--medicare.html" %}
 {% endblock media-item %}
 
 {% block call-to-action %}

diff --git a/benefits/eligibility/templates/eligibility/start--mst-agency-card.html b/benefits/eligibility/templates/eligibility/start--mst-agency-card.html
@@ -12,7 +12,7 @@ <h1>{% translate "You selected a Courtesy Card transit benefit." %}</h1>
 {% endblock headline %}
 
 {% block media-item %}
-  {% include "eligibility/includes/media-item--idcardcheck--start--mst-agency-card.html" %}
+  {% include "eligibility/includes/media-item--identification--start--mst-agency-card.html" %}
 {% endblock media-item %}
 
 {% block call-to-action %}

diff --git a/benefits/eligibility/templates/eligibility/start--sbmtd-agency-card.html b/benefits/eligibility/templates/eligibility/start--sbmtd-agency-card.html
@@ -12,7 +12,7 @@ <h1>{% translate "You selected a Reduced Fare Mobility ID transit benefit." %}</
 {% endblock headline %}
 
 {% block media-item %}
-    {% include "eligibility/includes/media-item--idcardcheck--start--sbmtd-agency-card.html" %}
+    {% include "eligibility/includes/media-item--identification--start--sbmtd-agency-card.html" %}
 {% endblock media-item %}
 
 {% block call-to-action %}

diff --git a/benefits/eligibility/templates/eligibility/start--senior.html b/benefits/eligibility/templates/eligibility/start--senior.html
@@ -12,7 +12,7 @@ <h1>{% translate "You selected an Older Adult transit benefit." %}</h1>
 {% endblock headline %}
 
 {% block media-item %}
-  {% include "eligibility/includes/media-item--idcardcheck--start--login-gov.html" %}
+  {% include "eligibility/includes/media-item--identification--start--login-gov.html" %}
 {% endblock media-item %}
 
 {% block call-to-action %}

diff --git a/benefits/eligibility/templates/eligibility/start--veteran.html b/benefits/eligibility/templates/eligibility/start--veteran.html
@@ -12,7 +12,7 @@ <h1>{% translate "You selected a Veteran transit benefit." %}</h1>
 {% endblock headline %}
 
 {% block media-item %}
-  {% include "eligibility/includes/media-item--idcardcheck--start--login-gov.html" %}
+  {% include "eligibility/includes/media-item--identification--start--login-gov.html" %}
 {% endblock media-item %}
 
 {% block call-to-action %}

diff --git a/benefits/eligibility/templates/eligibility/start.html b/benefits/eligibility/templates/eligibility/start.html
@@ -16,7 +16,7 @@ <h2 class="media-title h2-sm-p">{% translate "You will need a few items to conti
     <ul class="media-list mx-0 px-0 d-flex justify-content-center flex-column">
       {% block media-item %}
       {% endblock media-item %}
-      {% include "eligibility/includes/media-item--bankcardcheck--start.html" %}
+      {% include "eligibility/includes/media-item--contactless-card--start.html" %}
     </ul>
   </div>
 {% endblock inner-content %}

diff --git a/...item--bankcardcheck--index--calfresh.html → ...m--contactless-card--index--calfresh.html b/...item--bankcardcheck--index--calfresh.html → ...m--contactless-card--index--calfresh.html
@@ -1,10 +1,6 @@
 {% extends "core/includes/media-item.html" %}
 {% load i18n %}
 
-{% block icon %}
-    {% include "core/includes/icon.html" with name="bankcardcheck" %}
-{% endblock icon %}
-
 {% block heading %}
     {% translate "The next step is to connect your contactless card to your transit benefit" %}
 {% endblock heading %}

diff --git a/...des/media-item--bankcardcheck--index.html → .../media-item--contactless-card--index.html b/...des/media-item--bankcardcheck--index.html → .../media-item--contactless-card--index.html
@@ -1,10 +1,6 @@
 {% extends "core/includes/media-item.html" %}
 {% load i18n %}
 
-{% block icon %}
-    {% include "core/includes/icon.html" with name="bankcardcheck" %}
-{% endblock icon %}
-
 {% block heading_wrapper %}
     {% with heading_tag="h2" %}{{ block.super }}{% endwith %}
 {% endblock heading_wrapper %}

diff --git a/benefits/enrollment/templates/enrollment/index--calfresh.html b/benefits/enrollment/templates/enrollment/index--calfresh.html
@@ -4,7 +4,7 @@
 {% block media-items %}
   <div class="col-12 col-sm-12 col-lg-8">
     <ul class="media-list mb-5 mx-0 px-0 d-flex justify-content-center flex-column">
-      {% include "enrollment/includes/media-item--bankcardcheck--index--calfresh.html" %}
+      {% include "enrollment/includes/media-item--contactless-card--index--calfresh.html" %}
     </ul>
   </div>
 {% endblock media-items %}

diff --git a/benefits/enrollment/templates/enrollment/index.html b/benefits/enrollment/templates/enrollment/index.html
@@ -23,7 +23,7 @@ <h1 class="pb-lg-8 pb-4">
   {% block media-items %}
     <div class="col-12 col-sm-12 col-lg-8">
       <ul class="media-list mx-0 px-0 d-flex justify-content-center flex-column">
-        {% include "enrollment/includes/media-item--bankcardcheck--index.html" %}
+        {% include "enrollment/includes/media-item--contactless-card--index.html" %}
       </ul>
     </div>
   {% endblock media-items %}

diff --git a/benefits/static/css/styles.css b/benefits/static/css/styles.css
@@ -634,12 +634,6 @@ footer .footer-links li a.footer-link:visited {
   margin-bottom: calc(64rem / 16);
 }
 
-.media-list .media .media-line .icon {
-  width: var(--media-item-icon-size);
-  height: var(--media-item-icon-size);
-  margin-right: var(--media-item-icon-margin);
-}
-
 .media-list-icon-left-margin {
   margin-left: calc(
     var(--media-item-icon-size) + var(--media-item-icon-margin)

diff --git a/benefits/static/img/icon/bankcardcheck.svg b/benefits/static/img/icon/bankcardcheck.svg
diff --git a/benefits/static/img/icon/idcardcheck.svg b/benefits/static/img/icon/idcardcheck.svg