diff --git a/apps/browse/templates/browse/impala/category_landing.html b/apps/browse/templates/browse/impala/category_landing.html
index 066ed6a4c156..410482bbe749 100644
--- a/apps/browse/templates/browse/impala/category_landing.html
+++ b/apps/browse/templates/browse/impala/category_landing.html
@@ -29,7 +29,7 @@ <h1>{{ title }}</h1>
   {% set link = request.get_full_path()|urlparams(sort=key) %}
   {% set hc_src = 'cb-hc-' + src_dict[key] %}
   {% set dl_src = 'cb-dl-' + src_dict[key] %}
-  {% set vital = ('adu', 'rating') if key == popular else ('rating', 'adu') %}
+  {% set vital = ('adu', 'rating') if key == 'popular' else ('rating', 'adu') %}
   <div class="island c">
     <h2>
       {{ filter.opts_dict[key] }}
@@ -39,7 +39,7 @@ <h2>
                          vital_summary=vital[0], vital_more=vital[1]) }}
   </div>
   {% endfor %}
-  <h2 class="seeall"><a href="{{ request.get_full_path()|urlparams(sort=popular) }}">
+  <h2 class="seeall"><a href="{{ request.get_full_path()|urlparams(sort='popular') }}">
     {% if section == 'extensions' %}
       {% trans cnt = category.count, name = category.name %}
         See the {{ cnt }} extension in {{ name }} &raquo;
diff --git a/apps/browse/tests.py b/apps/browse/tests.py
index e4e8eb9ac970..7c76684706e9 100644
--- a/apps/browse/tests.py
+++ b/apps/browse/tests.py
@@ -269,6 +269,14 @@ def test_popular_adu_unit(self):
                 '%s weekly download%s' % (numberfmt(adu),
                                           's' if adu != 1 else ''))
 
+    def test_seeall_link_should_have_a_sort(self):
+        category = Category.objects.get(pk=1)
+        url = reverse('browse.extensions', kwargs={'category': category.slug})
+        response = self.client.get(url)
+        self.assertTemplateUsed(response, "browse/impala/category_landing.html")
+        doc = pq(response.content)
+        assert "sort=popular" in doc('.seeall a').attr('href')
+
 
 class TestLanguageTools(amo.tests.TestCase):
     fixtures = ['browse/test_views']