From 42f525e7498de72d009b909ced6bdafba1862ca1 Mon Sep 17 00:00:00 2001
From: John Carroll <jcarroll@macadmins-MBP-4.fios-router.home>
Date: Thu, 18 Mar 2021 15:07:43 -0400
Subject: [PATCH 1/3] restore tabindex of 0 to desktop menu links

---
 .../partials/navigation/navigation.html       | 28 +++----------------
 1 file changed, 4 insertions(+), 24 deletions(-)

diff --git a/fec/fec/templates/partials/navigation/navigation.html b/fec/fec/templates/partials/navigation/navigation.html
index 206cd02277..c6393ac931 100644
--- a/fec/fec/templates/partials/navigation/navigation.html
+++ b/fec/fec/templates/partials/navigation/navigation.html
@@ -2,26 +2,6 @@
  <button class="js-nav-toggle site-nav__button" aria-controls="site-menu">Menu</button>
   <div id="site-menu" class="site-nav__container">
     <ul class="site-nav__panel site-nav__panel--main">
-      <!-- <li class="mobile-search utility-nav__search">
-        <form accept-charset="UTF-8" action="/search" id="search_form" class="combo" method="get" role="search">
-          <input type="hidden" name="type" value="candidates">
-          <input type="hidden" name="type" value="committees">
-          <input type="hidden" name="type" value="site">
-          <label class="u-visually-hidden" for="query">Search</label>
-          <div class="combo combo--search combo--search--mini">
-            <input
-              class="js-site-search combo__input"
-              autocomplete="off"
-              id="query"
-              name="query"
-              type="text"
-              aria-label="Search FEC.gov">
-            <button type="submit" class="button--standard combo__button button--search">
-              <span class="u-visually-hidden">Search</span>
-            </button>
-         </div>
-        </form>
-      </li> -->
       <li><h2 class="site-nav__title u-under-lg-only">Menu</h2></li>
       <li class="site-nav__item u-under-lg-only">
         <a class="site-nav__link" href="/" rel="home">
@@ -29,26 +9,26 @@
         </a>
       </li>
       <li class="site-nav__item" data-submenu="data">
-        <a class="site-nav__link {% if self.content_section == 'data' or parent == 'data' %}is-parent{% endif %}" href="/data/" >
+        <a class="site-nav__link {% if self.content_section == 'data' or parent == 'data' %}is-parent{% endif %}" href="/data/" tabindex="0">
           <span class="site-nav__link__title">
           Campaign finance data</span>
         </a>
         {% include 'partials/navigation/nav-data.html' %}
       </li>
       <li class="site-nav__item site-nav__item--secondary" data-submenu="help">
-        <a href="/help-candidates-and-committees/" class="site-nav__link {% if self.content_section == 'help' %}is-parent{% endif %}">
+        <a href="/help-candidates-and-committees/" class="site-nav__link {% if self.content_section == 'help' %}is-parent{% endif %}" tabindex="0">
           <span class="site-nav__link__title">Help for candidates and committees</span>
         </a>
         {% include 'partials/navigation/nav-help.html' %}
       </li>
       <li class="site-nav__item" data-submenu="legal">
-        <a href="/legal-resources/" class="site-nav__link {% if self.content_section == 'legal' or parent == 'legal' %}is-parent{% endif %}">
+        <a href="/legal-resources/" class="site-nav__link {% if self.content_section == 'legal' or parent == 'legal' %}is-parent{% endif %}" tabindex="0">>
           <span class="site-nav__link__title">Legal resources</span>
         </a>
         {% include 'partials/navigation/nav-legal.html' %}
       </li>
       <li class="site-nav__item site-nav__item--secondary" data-submenu="about">
-        <a href="/about/" class="site-nav__link {% if self.content_section == 'about' %}is-parent{% endif %}">
+        <a href="/about/" class="site-nav__link {% if self.content_section == 'about' %}is-parent{% endif %}" tabindex="0">
           <span class="site-nav__link__title">About</span>
         </a>
         {% include 'partials/navigation/nav-about.html' %}

From 2b37fb25c0760f6a437a6609b25f6c13aeb0e31c Mon Sep 17 00:00:00 2001
From: John Carroll <jcarroll@macadmins-MBP-4.fios-router.home>
Date: Thu, 18 Mar 2021 16:01:43 -0400
Subject: [PATCH 2/3] remove extra >

---
 fec/fec/templates/partials/navigation/navigation.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fec/fec/templates/partials/navigation/navigation.html b/fec/fec/templates/partials/navigation/navigation.html
index c6393ac931..8d4ce0d999 100644
--- a/fec/fec/templates/partials/navigation/navigation.html
+++ b/fec/fec/templates/partials/navigation/navigation.html
@@ -22,7 +22,7 @@
         {% include 'partials/navigation/nav-help.html' %}
       </li>
       <li class="site-nav__item" data-submenu="legal">
-        <a href="/legal-resources/" class="site-nav__link {% if self.content_section == 'legal' or parent == 'legal' %}is-parent{% endif %}" tabindex="0">>
+        <a href="/legal-resources/" class="site-nav__link {% if self.content_section == 'legal' or parent == 'legal' %}is-parent{% endif %}" tabindex="0">
           <span class="site-nav__link__title">Legal resources</span>
         </a>
         {% include 'partials/navigation/nav-legal.html' %}

From 7653f1523c76350e53811851997829c13326e7c9 Mon Sep 17 00:00:00 2001
From: Laura Beaufort <lbeaufort@fec.gov>
Date: Thu, 18 Mar 2021 19:11:59 -0400
Subject: [PATCH 3/3] Fix filter when results have { or }

---
 fec/home/templatetags/filters.py | 13 ++++++++++---
 fec/home/tests/test_filters.py   | 23 +++++++++++++++++++++++
 2 files changed, 33 insertions(+), 3 deletions(-)
 create mode 100644 fec/home/tests/test_filters.py

diff --git a/fec/home/templatetags/filters.py b/fec/home/templatetags/filters.py
index 6c0b6fe626..1cfced0cd1 100644
--- a/fec/home/templatetags/filters.py
+++ b/fec/home/templatetags/filters.py
@@ -68,15 +68,22 @@ def web_app_url(path):
 
 @register.filter()
 def highlight_matches(text):
-    """Replaces the highlight markers with span tags for digitalgov search results"""
-    highlighted_text = text.replace('\ue000', '<span class="t-highlight">').replace('\ue001', '</span>')
+    """
+    Replaces the highlight markers with span tags for digitalgov search results.
+    Because format_html uses str.format, remove { and } because they are special characters.
+    """
+    cleaned_text = text.replace("{", "").replace("}", "")
+    highlighted_text = cleaned_text.replace(
+        "\ue000", '<span class="t-highlight">'
+    ).replace("\ue001", "</span>")
+
     return format_html(highlighted_text)
 
 
 @register.filter(name='splitlines')
 def splitlines(value):
     """
-        Returns the value turned into a list.
+    Returns the value turned into a list.
     """
     return value.splitlines()
 
diff --git a/fec/home/tests/test_filters.py b/fec/home/tests/test_filters.py
new file mode 100644
index 0000000000..1bc238c6ed
--- /dev/null
+++ b/fec/home/tests/test_filters.py
@@ -0,0 +1,23 @@
+from django.test import TestCase
+
+from home.templatetags.filters import highlight_matches
+
+
+class TestFilters(TestCase):
+
+    def test_highlight_matches(self):
+        text = '\ue000Highlighted\ue001 results'
+        highlighted_text = highlight_matches(text)
+        self.assertEqual(
+            highlighted_text,
+            '<span class="t-highlight">Highlighted</span> results'
+        )
+
+    def test_highlight_matches_with_brackets(self):
+        """highlight_matches should remove { and } from results"""
+        text = '\ue000Highlighted\ue001 {results}'
+        highlighted_text = highlight_matches(text)
+        self.assertEqual(
+            highlighted_text,
+            '<span class="t-highlight">Highlighted</span> results'
+        )