refactor: no need for icon tag, use modified include

benefits/core/templates/core/includes/icon__direct_args.html

@@ -0,0 +1,13 @@
+
+{% if not name %}
+  <p class="bg-grey-lightest color-standout">
+    Missing <strong><code>name</code></strong> argument
+  </p>
+{% elif not alt %}
+  <p class="bg-grey-lightest color-standout">
+    Missing <strong><code>alt</code></strong> argument
+  </p>
+{% else %}
+  {% load static %}
+  <img class="icon" width="150" height="150" src="{% get_static_prefix %}img/icon/{{ name }}.svg" alt="{{ alt }}" />
+{% endif %}

benefits/core/templates/core/tags/media_item.html

@@ -1,7 +1,7 @@
 {% load core %}
 
 <li class="media d-flex align-items-stretch w-auto">
-    <div class="media-line">{% icon name=icon alt=icon_alt %}</div>
+    <div class="media-line">{% include "core/includes/icon__direct_args.html" with name=icon alt=icon_alt %}</div>
     <div class="media-body">
         {% if heading %}<h3 class="media-body--heading">{{ heading }}</h3>{% endif %}
         {{ body }}

benefits/core/templatetags/core.py

@@ -3,14 +3,6 @@
 register = template.Library()
 
 
-@register.inclusion_tag("core/tags/icon.html")
-def icon(name, alt):
-    """
-    Defines a tag `{% icon name="" alt="" %}`
-    """
-    return {"name": name, "alt": alt}
-
-
 @register.tag
 def media_item(parser, token):
     """