Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 2db7deb

Browse filesBrowse files
committed
expand on comment
1 parent 207fa05 commit 2db7deb
Copy full SHA for 2db7deb

File tree

Expand file treeCollapse file tree

1 file changed

+11
-1
lines changed
Filter options
Expand file treeCollapse file tree

1 file changed

+11
-1
lines changed

‎src/plot_api/plot_api.js

Copy file name to clipboardExpand all lines: src/plot_api/plot_api.js
+11-1Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3675,7 +3675,17 @@ function makePlotFramework(gd) {
36753675
// The plot container should always take the full with the height of its
36763676
// parent (the graph div). This ensures that for responsive plots
36773677
// without a height or width set, the paper div will take up the full
3678-
// height & width of the graph div.
3678+
// height & width of the graph div.
3679+
// So, for responsive plots without a height or width set, if the plot
3680+
// container's height is left to 'auto', its height will be dictated by
3681+
// its childrens' height. (The plot container's only child is the paper
3682+
// div.)
3683+
// In this scenario, the paper div's height will be set to 100%,
3684+
// which will be 100% of the plot container's auto height. That is
3685+
// meaninglesss, so the browser will use the paper div's children to set
3686+
// the height of the plot container instead. However, the paper div's
3687+
// children do not have any height, because they are all positioned
3688+
// absolutely, and therefore take up no space.
36793689
.style({
36803690
width: "100%",
36813691
height: "100%"

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.