Merge pull request #6224 from readthedocs/davidfischer/improve-version-listview

Improve the version listview

Author: ericholscher

readthedocs/projects/views/public.py

@@ -284,7 +284,14 @@ def project_versions(request, project_slug):
         only_active=False,
     )
     active_versions = versions.filter(active=True)
+
+    # Limit inactive versions in case a project has a large number of branches or tags
+    # Filter inactive versions based on the query string
     inactive_versions = versions.filter(active=False)
+    version_filter = request.GET.get('version_filter', '')
+    if version_filter:
+        inactive_versions = inactive_versions.filter(verbose_name__icontains=version_filter)
+    inactive_versions = inactive_versions[:100]
 
     # If there's a wiped query string, check the string against the versions
     # list and display a success message. Deleting directories doesn't know how

readthedocs/templates/projects/project_version_list.html

@@ -78,7 +78,22 @@ <h1>{% trans "Active Versions" %}</h1>
 
         {% if inactive_versions %}
           <div class="module-header">
-            <h1>{% trans "Inactive Versions" %}</h1>
+            <h1>{% trans "Activate a version" %}</h1>
+            <p>{% trans "Active versions are built whenever new code is pushed to that branch or tag." %}</p>
+          </div>
+
+          {# Version search #}
+          <div class="wide-search-bar">
+            <div class="wide-search-bar-wrapper clearfix">
+              <form action="." method="GET">
+                <div class="text-input-wrapper">
+                  <input type="text" name="version_filter" value="{{ request.GET.version_filter }}" placeholder="1.0, 2.0, devel">
+                </div>
+                <div class="submit-input-wrapper">
+                  <input type="submit" value="{% trans 'Filter Inactive Versions' %}">
+                </div>
+              </form>
+            </div>
           </div>
 
           <div class="module-list">
@@ -104,7 +119,7 @@ <h1>{% trans "Inactive Versions" %}</h1>
 
                       {% if request.user|is_admin:project %}
                         <ul class="module-item-menu">
-                          <li><a href="{% url "project_version_detail" project.slug version.slug %}">{% trans "Edit" %}</a></li>
+                          <li><a href="{% url "project_version_detail" project.slug version.slug %}">{% trans "Activate" %}</a></li>
                         </ul>
                       {% endif %}