Skip to content

option to control margin.minfinal(width|height) #5246

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
archmoj opened this issue Nov 4, 2020 · 2 comments
Closed

option to control margin.minfinal(width|height) #5246

archmoj opened this issue Nov 4, 2020 · 2 comments
Labels
feature something new

Comments

@archmoj
Copy link
Contributor

archmoj commented Nov 4, 2020

I like that idea - layout.margin.minfinal(width|height) perhaps, with a default of something like 100 (px).

Originally posted by @alexcjohnson in #5237 (comment)

@archmoj archmoj added the feature something new label Nov 4, 2020
@archmoj
Copy link
Contributor Author

archmoj commented Nov 4, 2020

By exposing these two constants:

plotly.js/src/plots/plots.js

Lines 1867 to 1868 in 96d7511

var minFinalWidth = 64; // could possibly be exposed as layout.margin.minfinalwidth
var minFinalHeight = 64; // could possibly be exposed as layout.margin.minfinalheight

@gvwilson
Copy link
Contributor

Hi - we are trying to tidy up the stale issues and PRs in Plotly's public repositories so that we can focus on things that are still important to our community. Since this one has been sitting for several years, I'm going to close it; if it is still a concern, please add a comment letting us know what recent version of our software you've checked it with so that I can reopen it and add it to our backlog. Thanks for your help - @gvwilson

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature something new
Projects
None yet
Development

No branches or pull requests

2 participants