Skip to content

Commit

Permalink
Add default viewer dimensions for Plotly-based viewers.
Browse files Browse the repository at this point in the history
  • Loading branch information
Carifio24 committed Jul 26, 2024
1 parent c6c3150 commit 8cab99f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion glue_plotly/common/common.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ def dimensions(viewer):
elif isinstance(viewer, BqplotBaseView):
return 1200, 600 # TODO: What to do here?
else: # For now, this means a Plotly-based viewer
return viewer.figure.layout.width, viewer.figure.layout.height
return viewer.figure.layout.width or 600, viewer.figure.layout.height or 400

Check warning on line 28 in glue_plotly/common/common.py

View check run for this annotation

Codecov / codecov/patch

glue_plotly/common/common.py#L28

Added line #L28 was not covered by tests


def layers_to_export(viewer):
Expand Down

0 comments on commit 8cab99f

Please sign in to comment.