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

Issues Search
    My Issues | Starred     Open | Closed | All

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

Issue 2768913005: Milo: Fix empty lines in step text (Closed)
Patch Set: Review and Tests 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 | « milo/appengine/frontend/expectations/buildbot.build-Debug_page-_newline_1234.html ('k') | milo/appengine/logdog/internal/stream.pb.go » ('j') | 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..c3512224a3f99c10f99040b25306e14bb6747e86 100644
--- a/milo/appengine/frontend/templates/pages/build.html
+++ b/milo/appengine/frontend/templates/pages/build.html
@@ -173,7 +173,11 @@
<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 }}
</span>
</div>
<ul>
« no previous file with comments | « milo/appengine/frontend/expectations/buildbot.build-Debug_page-_newline_1234.html ('k') | milo/appengine/logdog/internal/stream.pb.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698