plotly / plotly/react-plotly.js
Is layout.height really disallowed when layout.autosize=true ?
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
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- 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