[CHORE]: Change default to `true` for `responsive` config attribute
Open
Beginner friendly
Nobody has claimed this yet.
chore
P1
plotly-internal
size: 1
task
- Dominant language
- JavaScript
- Stars
- 18.3k
- Forks
- 2k
- Avg merge
- 2d 12h
- Merged PRs (30d)
- 28
Description
Description
Change default to true for the responsive config attribute.
Why should this change be made?
We said we would do it in v3, but we forgot 🤷. It's arguably a better experience for users.
Scope
- Change the default in this section:
https://github.com/plotly/plotly.js/blob/174319ac1b8c58bc27c23d40f7d108ac4090e48d/src/plot_api/plot_config.js#L127-L133
Notes
- Before flipping this switch, run through the backlog and check for any issues that should be addressed first
Contributor guide
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
Open src/plot_api/plot_config.js at the linked section and inspect the current responsive configuration default. Before changing it, review the backlog for issues that should be addressed first. Done means the default is true and the relevant checks pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- data-visualization, frontend
- Issue type
- Refactor
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 82/100