[PATCH] D29991: Daily report: only display non empty result tables to avoid cluttering the UI.

[Arnaud A. de Grandmaison via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL295334: Daily report: only display non empty result tables to avoid cluttering the UI. (authored by aadg).

Changed prior to commit:
  https://reviews.llvm.org/D29991?vs=88741&id=88744#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D29991

Files:
  lnt/trunk/lnt/server/ui/templates/reporting/daily_report.html


Index: lnt/trunk/lnt/server/ui/templates/reporting/daily_report.html
===================================================================
--- lnt/trunk/lnt/server/ui/templates/reporting/daily_report.html
+++ lnt/trunk/lnt/server/ui/templates/reporting/daily_report.html
@@ -232,6 +232,7 @@
 {%- endmacro %}
 
 {% for field,field_results in report.result_table|reverse %}
+{%- if field_results -%}
 <h3>Result Table ({{ field.name }})</h3>
 {{ result_header()+"\n  <tbody>" if not report.for_mail }}
 {%- for test,visible_results in field_results -%}
@@ -267,6 +268,9 @@
 {{ "</tbody></table><p>" if report.for_mail }}
 {%- endfor %}
 {{ "</tbody></table>" if not report.for_mail }}
+{%- else -%}
+<h3>No significant {{ field.name }} changes found.</h3>
+{%- endif -%}
 {%- endfor -%}
 
 {% endif %}


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D29991.88744.patch
Type: text/x-patch
Size: 798 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170216/3f106952/attachment.bin>