diff mbox

[19/51] filters: Put the active filter values in bold

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

Commit Message

Damien Lespiau Aug. 24, 2015, 6:23 p.m. UTC 
From: Belén Barros Peña <belen.barros.pena@intel.com>

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
---
 patchwork/templates/patchwork/filters.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Stephen Finucane Sept. 9, 2015, 2:56 p.m. UTC | #1 
> From: Belén Barros Peña <belen.barros.pena@intel.com>

> 

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


You're also changing the string that's showed to the user, so this commit title is misleading. Surely "Filters" is clearer/less wordy than "Show patches with"?
diff mbox

Patch

diff --git a/patchwork/templates/patchwork/filters.html b/patchwork/templates/patchwork/filters.html
index 7652db5..ddd20dd 100644
--- a/patchwork/templates/patchwork/filters.html
+++ b/patchwork/templates/patchwork/filters.html
@@ -144,10 +144,10 @@  function submitter_field_change(field)
 
 <div class="filters">
  <div id="filtersummary">
-  <strong><a href="javascript:filter_click()">Filters</a>:</strong>
+  <a href="javascript:filter_click()">Show patches with</a>:
  {% if filters.applied_filters %}
   {% for filter in filters.applied_filters %}
-   {{ filter.name }} = {{ filter.condition }}
+   {{ filter.name }} = <strong>{{ filter.condition }}</strong>
     {% if not filter.forced %}
      <a href="{{ filter.url_without_me }}">&nbsp;&nbsp;<span
       class="glyphicon glyphicon-minus-sign" /></a>