Skip to content

Upgrade to v1.46 #1506

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 37 commits into from
Apr 4, 2019
Merged

Upgrade to v1.46 #1506

merged 37 commits into from
Apr 4, 2019

Conversation

cpsievert
Copy link
Collaborator

@cpsievert cpsievert commented Apr 2, 2019

TODO

These plotly.js releases will require some R changes:

  • title attributes linked to strings are now deprecated. Please use title.text instead to fill in your title text [#3276]
  • title* attributes are now deprecated. They moved to title.*. For example, colorbar.titleside is now colorbar.title.side [#3276]

FWIW, I'm pretty sure layout.title, layout.titlefont, layout[axisid].title, layout[axisid].titlefont, marker.colorbar.title, marker.colorbar.titlefont, marker.colorbar.titleside, and carpet-specific attributes are the only ones effected.

  • It looks like these attributes still work, but be good form to identify these cases on the R side and throw an information deprecation message/warning.
  • Update any 'internal' logic using these attributes to use the new nested structure.
  • Would also be a good time to have ggplotly() leverage the new layout.title placement API.

@cpsievert cpsievert mentioned this pull request Apr 2, 2019
3 tasks
@cpsievert
Copy link
Collaborator Author

cpsievert commented Apr 3, 2019

Add tests of warnings thrown for titlefont, and config() (Update: done in 11ba0e0)

@schloerke schloerke self-requested a review April 3, 2019 18:49
Copy link
Contributor

@schloerke schloerke left a comment

Choose a reason for hiding this comment

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

See your comments

@cpsievert
Copy link
Collaborator Author

@schloerke looking good now?

@schloerke schloerke self-requested a review April 4, 2019 13:10
@cpsievert cpsievert merged commit c1d74bf into master Apr 4, 2019
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