From bbecda2df731880addcdb879f2c0f12608568f98 Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Nov 02 2017 09:02:59 +0000
Subject: Restructure the my star page and include the project's description


Fixes https://pagure.io/pagure/issue/2664

Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>

---

diff --git a/pagure/templates/user_stars.html b/pagure/templates/user_stars.html
index bfeee2e..8635a78 100644
--- a/pagure/templates/user_stars.html
+++ b/pagure/templates/user_stars.html
@@ -8,44 +8,34 @@
     <h3 class="p-b-2">
       {{ repos | length }} Projects Starred by {{ username }}
     </h3>
-    <hr>
-      <table class="table table-hover table-striped m-b-0 m-t-1">
-        <thead class="thead-default">
-          <tr>
-            <th class="nowrap">Starred Projects</th>
-          </tr>
-        </thead>
-        <tbody>
+
+    <div class="card">
+      <div class="card-header">
+        <p>Starred Projects</p>
+      </div>
+      <div class="card-block p-a-0">
+        <div class="list-group">
           {% for repo in repos %}
-            <tr class="nowrap">
-              <td>
-                {% if repo.avatar_email %}
-                <img src="{{ repo.avatar_email | avatar_url | safe }}" width=20 height=20 />
-                <a href="{{ url_for(
+            <a class="list-group-item" href="{{ url_for(
                 'view_repo',
                 repo=repo.name,
                 username=repo.user.user if repo.is_fork else None,
                 namespace=repo.namespace,
-                )}}">{{repo.fullname}}</a>
-                </br>
+                )}}">
+                {% if repo.avatar_email %}
+                <img src="{{ repo.avatar_email | avatar_url | safe }}" width=20 height=20 />
                 {% else %}
-               <div class="col-sm-12">
-                 <span class="oi" data-glyph="document"></span>
-                <a href="{{ url_for(
-                'view_repo',
-                repo=repo.name,
-                username=repo.user.user if repo.is_fork else None,
-                namespace=repo.namespace,
-                )}}">{{repo.fullname}}</a>
-                </div>
-                </br>
+                <span class="oi" data-glyph="document"></span>
                 {% endif %}
-              </td>
-            </tr>
+                {{repo.fullname}}
+                <div class="pull-xs-right">
+                  {{ repo.description or '' }}
+                </div>
+            </a>
           {% endfor %}
-        </tbody>
-      </table>
-      <br>
+        </div>
+      </div>
+    </div>
   </div>
 </div>
 {% endblock %}
diff --git a/tests/test_pagure_flask_ui_star_project.py b/tests/test_pagure_flask_ui_star_project.py
index 059422b..06f128b 100644
--- a/tests/test_pagure_flask_ui_star_project.py
+++ b/tests/test_pagure_flask_ui_star_project.py
@@ -228,9 +228,11 @@ class TestStarProjectUI(tests.SimplePagureTest):
             output.data
         )
         self.assertIn(
-            '<a href="/test">test</a>\n',
-            output.data
-        )
+            '<a class="list-group-item" href="/test">', output.data)
+        self.assertEqual(output.data.count('class="list-group-item"'), 1)
+        self.assertEqual(
+            output.data.count('<span class="oi" data-glyph="document"></span>'),
+            1)
 
         # make pingou unstar the project
         user = tests.FakeUser()
@@ -260,9 +262,13 @@ class TestStarProjectUI(tests.SimplePagureTest):
             output.data
         )
         self.assertNotIn(
-            '<a href="/test">test</a>\n',
+            '<a class="list-group-item" href="/test">test</a>\n',
             output.data
         )
+        self.assertEqual(
+            output.data.count('<span class="oi" data-glyph="document"></span>'),
+            0)
+        self.assertEqual(output.data.count('class="list-group-item"'), 0)
 
 
 if __name__ == '__main__':