Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(357)

Unified Diff: milo/appengine/frontend/templates/pages/build.html

Issue 2768913005: Milo: Fix empty lines in step text (Closed)
Patch Set: Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: milo/appengine/frontend/templates/pages/build.html
diff --git a/milo/appengine/frontend/templates/pages/build.html b/milo/appengine/frontend/templates/pages/build.html
index 277e8adc460d345fdf1c00052040fb7241d41bc8..2bb44a7e3aaad6603b5ec0ce4bfaa2fc9d55d6b4 100644
--- a/milo/appengine/frontend/templates/pages/build.html
+++ b/milo/appengine/frontend/templates/pages/build.html
@@ -173,7 +173,10 @@
<span style="float:right">( {{ .Duration | humanDuration }} )</span>
{{- end }}
<span>
- {{ range .Text }}<div class="step-text">{{ . }}</div>{{ end }}
+ {{ range .Text -}}
+ <div class="step-text">
+ {{- if . -}}{{- . -}}{{- else -}}&nbsp;{{- end -}}
+ </div>{{- end }}
nodir 2017/03/24 18:39:26 please put </div> to the same indentation level as
hinoka 2017/03/27 16:21:01 Done.
</span>
</div>
<ul>
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698