-
Notifications
You must be signed in to change notification settings - Fork 50
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
move layout properties to layout options group in widget #260
Conversation
@SGroe I'm +1 on that. The layout configuration box can be extended with a lot of new widgets (axis color, fonts, sizes, etc) but this will be also something to think in a future (many actually ;) ) PR. But we should discuss this somewhere else. This is ploty chart studio: https://chart-studio.plotly.com/create/#/ where basically all plotly customization are available (p.s. I never used chart studio ;) ). |
To avoid empty space above layout option group
@ghtmtt, what do you think about moving the remaining non-layout properties to the first settings page (within "Properties" group). These include:
|
are you suggesting to move basically all the widgets of the Advanced Configuration within the Layout Options block? |
I suggest to move the mentioned properties (legend title, hover tooltip, ...) to the "Properties" group on the first page. Currently, this group contains properties like "Marker Color" or "Opacity". |
I'd move the |
I can move it from 'Plot properties' to 'Properties'. I don't want to move it to the 'Layout options' because this group should only contain (layout) properties. Layout properties are set once and apply for all sub plots. The 'Legend Title' will be different for each sub plot. |
got it, + 1 for me! Thanks |
I would like to move all layout properties (e.g. plot title) to the new layout option group (introduced by #259). Only the ui file is changed here.
These layout properties only apply to the first plot, if multiple plots in a layout are defined. One idea I have in my mind is to (optionally) synchronize these layout properties across all plots of a layout item (to be done in a future pr).
@ghtmtt: What do you think about that? I you like that, I would add some minor improvements.