Skip to content
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

Fix default size settings in plot.Layout #180

Closed
ArtemkaKun opened this issue Oct 21, 2023 · 0 comments · Fixed by #181
Closed

Fix default size settings in plot.Layout #180

ArtemkaKun opened this issue Oct 21, 2023 · 0 comments · Fixed by #181
Assignees
Labels
bug Something isn't working

Comments

@ArtemkaKun
Copy link
Contributor

Describe the bug

Currently default values for plot.Layout are

pub struct Layout {
pub mut:
	title       string
	title_x     f64  = 0.5
	autosize    bool = true
	width       int  = 550
	height      int  = 550
	xaxis       Axis
	yaxis       Axis
	annotations []Annotation
}

both autosize and width/height are defined, and autosize is set to true which makes no sense since it wouldn't be taken into account because of width/height are not 0.

Expected Behavior

autosize is set to false by default

Current Behavior

autosize is set to true while width/height are not 0 by default

Reproduction Steps

Not related

Possible Solution

No response

Additional Information/Context

No response

V version

V 0.4.2 f54f156.20a1af3

Version used

6904d72

Environment details (OS name and version, etc.)

Not related

@ArtemkaKun ArtemkaKun added the bug Something isn't working label Oct 21, 2023
@ulises-jeremias ulises-jeremias self-assigned this Oct 21, 2023
@ulises-jeremias ulises-jeremias linked a pull request Oct 21, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants