diff mbox

[46/51] patch-list: Use table-condensed to display the patch list

Message ID 1440440620-25937-47-git-send-email-damien.lespiau@intel.com
State Changes Requested
Headers show

Commit Message

Damien Lespiau Aug. 24, 2015, 6:23 p.m. UTC
Wolfram didn't like the fact we're now seeing far fewer patches on a
single page. That sounds fair enough and the table-condensed bootstrap
class removes some padding, hopefully making it better.

Suggested-by: Wolfram Sang <wsa@the-dreams.de>
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
---
 patchwork/templates/patchwork/patch-list.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Stephen Finucane Sept. 10, 2015, 8:46 a.m. UTC | #1
> Wolfram didn't like the fact we're now seeing far fewer patches on a

> single page. That sounds fair enough and the table-condensed bootstrap

> class removes some padding, hopefully making it better.

> 

> Suggested-by: Wolfram Sang <wsa@the-dreams.de>

> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>


Acking, but please combine this with '25/51 patch-list: Re-design the main list of patches with Bootsrap'. I knew it would be contentious :)

Acked-by: Stephen Finucane <stephen.finucane@intel.com>
diff mbox

Patch

diff --git a/patchwork/templates/patchwork/patch-list.html b/patchwork/templates/patchwork/patch-list.html
index 8f48dbc..c81fe88 100644
--- a/patchwork/templates/patchwork/patch-list.html
+++ b/patchwork/templates/patchwork/patch-list.html
@@ -44,7 +44,7 @@  $(document).ready(function() {
 {% csrf_token %}
 <input type="hidden" name="form" value="patchlistform"/>
 <input type="hidden" name="project" value="{{project.id}}"/>
-<table class="table table-hover" id="patchlist">
+<table class="table table-hover table-condensed" id="patchlist">
  <thead>
   <tr>
    {% if user.is_authenticated %}