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

Unified Diff: tracing/tracing/value/ui/histogram_span.html

Issue 2994183002: Results.html: fix height of statistics table. (Closed)
Patch Set: Created 3 years, 4 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 | tracing/tracing/value/ui/histogram_span_test.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tracing/tracing/value/ui/histogram_span.html
diff --git a/tracing/tracing/value/ui/histogram_span.html b/tracing/tracing/value/ui/histogram_span.html
index 8d58630b578d59b52aa565a0596aae471b5ca636..ddc0b9aceb537cdb1f4567196e8ab0e47f65fb26 100644
--- a/tracing/tracing/value/ui/histogram_span.html
+++ b/tracing/tracing/value/ui/histogram_span.html
@@ -34,6 +34,9 @@ found in the LICENSE file.
#chart svg {
display: block;
}
+ #stats_container {
+ overflow-y: auto;
+ }
</style>
<div id="container">
@@ -194,6 +197,8 @@ tr.exportTo('tr.v.ui', function() {
}
this.chart_.graphHeight = heightPx - (this.chart_.margin.top +
this.chart_.margin.bottom);
+ this.$.stats_container.style.maxHeight =
+ this.chart_.getBoundingClientRect().height + 'px';
},
/**
@@ -522,6 +527,8 @@ tr.exportTo('tr.v.ui', function() {
}
];
}
+ this.$.stats_container.style.maxHeight =
+ this.chart_.getBoundingClientRect().height + 'px';
await this.viewState.update({
brushedBinRange: tr.b.math.Range.fromExplicitRange(
0, this.histogram.allBins.length)});
@@ -574,6 +581,8 @@ tr.exportTo('tr.v.ui', function() {
this.chart_.overrideDataRange = dataRange;
this.chart_.data = chartData;
+ this.$.stats_container.style.maxHeight =
+ this.chart_.getBoundingClientRect().height + 'px';
}
});
});
« no previous file with comments | « no previous file | tracing/tracing/value/ui/histogram_span_test.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698