Skip to content

finally: how to style px #2156

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

Merged
merged 3 commits into from
Feb 3, 2020
Merged

finally: how to style px #2156

merged 3 commits into from
Feb 3, 2020

Conversation

nicolaskruchten
Copy link
Contributor

No description provided.

display_as: file_settings
language: python
layout: base
name: Plotly Express
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Styling Plotly Express?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah yeah I forgot to actually edit the front matter here, it's a straight copy.

@emmanuelle
Copy link
Contributor

It looks great! The only thing which is missing maybe is the case where you cannot call one of the update_x methods, for example when you want to change the duration of frames in an animation etc. It might be a good idea to link here to the structure of a figure (https://plot.ly/python/creating-and-updating-figures/), say that doing print(fig) can be a good idea, and that it's also possible to modify manually the contents of the figure. Or does it make the message too complicated?

@emmanuelle
Copy link
Contributor

but it can also be merged as it is, so 💃 anyway

@nicolaskruchten
Copy link
Contributor Author

Yeah I should definitely link to that page... I toyed with printing a figure and saying "here are all the things you can change" but it did dilute the message a fair bit. The animation thing is a bit of a special case in how deep the edit goes ;)

@nicolaskruchten nicolaskruchten merged commit 2a9aafd into doc-prod Feb 3, 2020
@nicolaskruchten nicolaskruchten deleted the px_styling branch June 19, 2020 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants