The "help" icon and tag was broken which produced bad tags and text s…

…ince the attributes were not quoted correctly. See the Alt-text and title text for accessibility for the help button.
CDLUC3 · Nov 13, 2024 · cee5d44 · cee5d44
1 parent 0f04f59
commit cee5d44
Showing 1 changed file with 3 additions and 7 deletions.
diff --git a/ui_tags/templatetags/layout_extras.py b/ui_tags/templatetags/layout_extras.py
@@ -125,19 +125,15 @@ def help_icon(
         "Click for additional help" + " " + str(specifics)
     )
     return django.utils.html.format_html(
-        '<a href="#" title="ID type information" class="button__icon-link" id={} role="button" data-toggle="popover" data-placement={} data-trigger="click" tabindex="0">'
-        '<img src="/static/images/iconHelp.svg" alt={}  class={} title={}/>'
+        '<a href="#" title="ID type information" class="button__icon-link" id="{}" role="button" data-toggle="popover" '
+        'data-placement="{}" data-trigger="click" tabindex="0">'
+        '<img src="/static/images/iconHelp.svg" alt="{}"  class="{}" title="{}" />'
         '</a>',
         id_of_help,
         placement,
         title,
         css_class,
         title
-        # '<a href="#" class="button__icon-link" id="' + str(id_of_help) + '" ' +
-        # 'role="button" data-toggle="popover" data-placement="' + placement + '" ' +
-        # 'data-trigger="click" tabindex="0">' +
-        # '<img src="/static/images/iconHelp.svg" alt="' + str(title) + '"' +
-        # ' class="' + str(css_class) + '" title="' + str(title) + '"/></a>'
     )