plotly / plotly/react-plotly.js

Is layout.height really disallowed when layout.autosize=true ?

Open
#332 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
JavaScript
Stars
1.1k
Forks
138
Avg merge
3d 2h
Merged PRs (30d)
4

Description

I've got a responsive table created per this repo's docs:
To make a plot responsive, i.e. to fill its containing element and resize when the window is resized, use style or className to set the dimensions of the element (i.e. using width: 100%; height: 100% or some similar values) and set useResizeHandler to true while setting layout.autosize to true and leaving layout.height and layout.width undefined. This can be seen in action in this CodePen and will implement the behaviour documented here: https://plot.ly/javascript/responsive-fluid-layout/

I would like to reduce the whitespace below the table by using the layout.height property which seems to work in plotly.js (CodePen) and doesn't appear to be prohibited per the responsive-fluid-layout section cited above. Is there another approach we can/should take in react-plotly if the layout.height property must remain undefined here?

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the responsive-fluid-layout documentation and the two linked CodePens, then compare how layout.height behaves with layout.autosize in plotly.js and react-plotly.js. Done means the project has a decided, documented approach for reducing the table's whitespace while preserving responsive resizing.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, react
Domain
data-visualization, frontend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.