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

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

Issue 2086353004: milo: put line breaks between step text strings (Closed) Base URL: https://chromium.googlesource.com/external/github.com/luci/luci-go@master
Patch Set: milo: put line breaks between step text strings Created 4 years, 6 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
Index: appengine/cmd/milo/frontend/templates/buildbot/pages/build.html
diff --git a/appengine/cmd/milo/frontend/templates/buildbot/pages/build.html b/appengine/cmd/milo/frontend/templates/buildbot/pages/build.html
index 2491385f4f50e4a33aef92ff141f5d856227743b..d4a7fd4b0c97b1bdf687a0cc8c2db7afc640e6b1 100644
--- a/appengine/cmd/milo/frontend/templates/buildbot/pages/build.html
+++ b/appengine/cmd/milo/frontend/templates/buildbot/pages/build.html
@@ -22,7 +22,7 @@
<!--- TODO(hinoka): step text -->
{{ end }}
<span class="subtitle">
- {{ range .Build.Summary.Text }}{{ . }}{{ end }}
+ {{ range .Text }}<div class="step-text">{{ . }}</div>{{ end }}
</span>
</p>
@@ -76,7 +76,9 @@
<li>
<div class="status-{{.Status}} result">
<b>{{.Label}}</b>
- {{ range .Text }} {{ . }} {{ end }}
+ <span>
+ {{ range .Text }}<div class="step-text">{{ . }}</div>{{ end }}
+ </span>
&nbsp;<span style="float:right">( {{ .Duration | humanDuration }} )</span>
</div>
<ul>

Powered by Google App Engine
This is Rietveld 408576698