From d5aa270225b90835e17d93822d4f98c9b3db6d67 Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Oct 11 2017 08:52:26 +0000
Subject: [PATCH 1/3] Fix the title of the percentage when hovering over the red bar in issues


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

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

---

diff --git a/pagure/templates/issues.html b/pagure/templates/issues.html
index f917367..9561b9f 100644
--- a/pagure/templates/issues.html
+++ b/pagure/templates/issues.html
@@ -52,7 +52,7 @@
     <div class="addrem_bar issues_pbar m-b-1 {%
       if status|lower not in ['open', 'true'] %}closed{%
       endif %}" title="{{
-      (100.0 * (oth_issues_cnt / total_issues_cnt))|round|int }}% of {%
+      (100 - (100.0 * (issues_cnt / total_issues_cnt)))|round|int }}% of {%
       if status|lower in ['open', 'true'] %}closed{%
       else %}open{%
       endif %} issues of total {{ total_issues_cnt }} issues">

From 71f685bfa2ae9ee902444208a70af00826ec0180 Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Oct 11 2017 08:53:07 +0000
Subject: [PATCH 2/3] Flake8 fix


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

---

diff --git a/pagure/ui/filters.py b/pagure/ui/filters.py
index 7e3ae75..f692295 100644
--- a/pagure/ui/filters.py
+++ b/pagure/ui/filters.py
@@ -221,7 +221,8 @@ def format_loc(loc, commit=None, filename=None, tree_id=None, prequest=None,
                     '<div class="card-block">'
                     '<small><div id="comment-%(commentid)s">'
                     '<img class="avatar circle" src="%(avatar_url)s"/>'
-                    '<a href="%(url)s" title="%(user_html)s"> %(user)s</a> commented '
+                    '<a href="%(url)s" title="%(user_html)s">'
+                    '%(user)s</a> commented '
                     '<a class="headerlink" title="Permalink '
                     'to this headline" href="#comment-%(commentid)s">'
                     '<span title="%(date)s">%(human_date)s</span>'

From 75d3e38d623642e5e8edc628fa5fc8ac4f45c776 Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Oct 11 2017 08:58:18 +0000
Subject: [PATCH 3/3] Add tests for the progress bar on the issue list


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

---

diff --git a/tests/test_pagure_flask_ui_issues.py b/tests/test_pagure_flask_ui_issues.py
index 393f6e1..eec2912 100644
--- a/tests/test_pagure_flask_ui_issues.py
+++ b/tests/test_pagure_flask_ui_issues.py
@@ -520,6 +520,12 @@ class PagureFlaskIssuestests(tests.Modeltests):
         self.assertIn('<title>Issues - test - Pagure</title>', output.data)
         self.assertTrue(
             '<h2>\n      2 Open Issues' in output.data)
+        self.assertIn(
+            '<div class="addrem_bar issues_pbar m-b-1 " title="33% of '
+            'closed issues of total 3 issues">', output.data)
+        self.assertIn(
+            '<span style="width: 67%" title="67% of open issues of total '
+            '3 issues">', output.data)
 
         # Status = closed (all but open)
         output = self.app.get('/test/issues?status=cloSED')
@@ -527,6 +533,12 @@ class PagureFlaskIssuestests(tests.Modeltests):
         self.assertIn('<title>Issues - test - Pagure</title>', output.data)
         self.assertTrue(
             '<h2>\n      1 Closed Issues' in output.data)
+        self.assertIn(
+            '<div class="addrem_bar issues_pbar m-b-1 closed" '
+            'title="67% of open issues of total 3 issues">', output.data)
+        self.assertIn(
+            '<span style="width: 33%" title="33% of closed issues '
+            'of total 3 issues">', output.data)
 
         # Status = fixed
         output = self.app.get('/test/issues?status=fixed')