This reverts commit0a909c8.

* New version of scales no longer 'spans the gamut' of a qualitative color palette* reduce Reduce() for merging listsuse do.call() instead to avoid quadratic complexity of growing lists* ensure_one(): reduce noise if attrs are NULLignore NULL values* ggplotly: fix Xaxis anchor if the last row incompleteattach X axis to the last non-empty row in a column* stylistic changes* validate new visual baselines* Doesn't seem like we need to handle yaxis being a factorand if we do, it likely needs a different fix* update shinytest baselines* ugh maintaining shinytest tests for htmlwidgets is a real pain* update sf baselines* go back to using ggplotly() to prepare the widget for shiny rendering* the extra call to plotly_build() forces more computation then necessary andfixesplotly#1569* Generation of non-intercept data in hline/vline should respect coord_flip(),closesplotly#1519* update news; fix typo* Account for new changes in ggplot2's internal API,fixesplotly#1561* Break values of positional scales have moved from  to* Text labels of positional scales have moved from  to* sf graticule degree labels are now quoted?* include braces* gsub* moar braces* new sf baselines* update news* upgrade to plotly.js v1.49.0* upgrade to plotly.js v1.49.2* upgrade to plotly.js v1.49.4* update baseline* update shinytest baseline* make a deep copy of x.layout.width/x.layout.height for use in the resize method,closesplotly#1553* Wrap user-supplied expression in evalq(),closesplotly#1528* Use shiny::installExprFunction to register debug hooks* Asynchronously register plot events* Revert "Merge pull requestplotly#1539 from ropensci/renderWidgetPrep"This reverts commitd416cea, reversingchanges made tod11bb5a.* no need to be making deep copies* Improvements to algorithm for attaching key attribute for shiny's event data,fixesplotly#1610* Account for pointNumbers* Do nothing if we don't recognize the case* When pointNumber is a constant, relay it as such* handle the 3 cases separately* use idx instead of i* update news* Call setInputValue only if it's defined,closesplotly#1624* Revert "Revert "Merge pull requestplotly#1539 from ropensci/renderWidgetPrep""This reverts commit2856731.* pass along a quoted expr that calls func()* Revert "go back to using ggplotly() to prepare the widget for shiny rendering"This reverts commita0fa68f.* update news* verify_webgl() shouldn't warn about types that are already gl,closesplotly#1569* fix test* Do not create structure with NULL to remove warnings* Fix management of environments in renderPlotly(),fixesplotly#1639Both assignment and evaluation of func() should happen in the local environment* Add test to proveplotly#1299 fix* proposed change to avoid peeking at 'grid' unit internals* improve previous commit* refactor Paul's unit type commits fromplotly#1646* Some CRAN builds don't have pandoc* plot_mapbox() shouldn't force a scattermapbox trace type,closesplotly#1608* fix R CMD check notes* r-devel apparently doesn't yet have grid::unitType()* remove outdated travis config* Validate changes visual baselines due to changes in R3.6's new RNG method* Add support for new plotly_sunburstclick event,closesplotly#1648* validate new shinytest baselines* update news* fix silly mistake from3cbccfc* document* test expectation should account for changes in the default color palette on r-devel* improve the aforementioned test example and add a visual test* Ignore calculated aesthetics that match specified aesthesticsAftertidyverse/ggplot2@10fa0014, it's possible for calculated aes to exist for all default_aes* fix broken link raster2uri and improve the description* cran hyperlinks must be https* fix roxygen warning* In contrast to attr(x, unit), the new grid::unitType() function always return a vector the same length as it's input* bump to release version; submit to CRAN* Respect sf's plot12 graticule attribute when building axis ticks,fixesplotly#1673* Remove missing values in ticktext/tickvalsAs discussed intidyverse/ggplot2#3566 (comment), it's now possible for the ggplot2 labels/positions to contains missing values, and we should be able to simply ignore them* link to comment* remove outdated/bad test expectation...the visual baseline covers it* Don't assume sf geometry is always accessible via [['geometry']],fixesplotly#1659* validate baselines* update news* update to plotly.js 1.52.2* new baselines* Add image and treemap visual tests. Also, ensure treemap's stroke reflects the paper's bgcolor* Add Object.setPrototypeOf polyfill for shinytest and update shinytest json baseline* Add add_image() for easier rendering of image traces via raster objects* Bump version, update news, document, add unit test for add_image()* roxygen escapes % now* Only call locale_dependency() for locales not included with standard bundle,closesplotly#1686* CRAN submission* plot_mapbox() should default to scattermapbox unless z is present,fixesplotly#1707* add visual test forplotly#1674* update poor test expectations that assume stringsAsFactors defaults to TRUE* bump version; update news* update visual tests* remove rnaturalearth dependency in testsCo-authored-by: Carson Sievert <cpsievert1@gmail.com>Co-authored-by: Alexey Stukalov <astukalov@gmail.com>Co-authored-by: Wim <wfjvdham@gmail.com>Co-authored-by: Paul Murrell <paul@stat.auckland.ac.nz>

…nsci-master

* Fix bug in verify_attr. Closes issueplotly#1720.* Add test to check that ggplotly does not break discrete x-axis in facet panels with only one category.* Update tests/testthat/test-facet-axis.RCo-authored-by: Carson Sievert <cpsievert1@gmail.com>* Add news item.* Translate `\n` to `<br />` in fct labels, too (plotly#1700)* Translate `\n` to `<br />` in fct labels, tooBefore, this only worked for colums of type character.* add@salim-b as ctb* Add test for `translate_linebreaks()`* remove superfluous char conversion* change order of authors* replace forcats code with base R* add news itemCo-authored-by: Stefan Moog <moogs@gmx.de>Co-authored-by: Carson Sievert <cpsievert1@gmail.com>Co-authored-by: Salim B <salim@posteo.de>

* Get ggplotly() playing nicely with thematic (plotly#1801)* Pull plotly's custom ggplot_build into a function and hook into thematic if relevant* add thematic to suggests and use it only when loaded* update shinytest tests* Go back to the assign() based approach* bust cache* CRAN version of crosstalk* add visual tests* update newsCo-authored-by: Carson Sievert <cpsievert1@gmail.com>

…nsci-master7

Use Plotly version 1.58

Merge branch 'master' ofhttps://github.com/ropensci/plotly into ropensci-master# Conflicts:#DESCRIPTION#R/sysdata.rda#inst/docker/Dockerfile.vtest#inst/htmlwidgets/lib/plotlyjs/plotly-latest.min.js

Merge branch 'ropensci-master'# Conflicts:#R/plotly.R#inst/htmlwidgets/lib/plotlyjs/plotly-latest.min.js

RS-7135: re-setting plotly-latest to 1.58

Plotly 1.58 plus autoMargin minHeight patch

* Use latest bundle 2.35.2* Turn off warning for barcornerradius* Add documentation* Only run standard tests* Remove github workflow* Remove travis CI config* Use same circle CI config as other flip packages* Replace with default config* Try to turn off all tests* Stop returning error* Turn off build

DPI-2832: Pack plotly in nix

* Initial commit* Remove prepush* Final line for plotly.yml* Re-add lib* Refactor* Create test* Specify rhtmlBuildUtils version* Remove preinstall script* Delete package-lock.json* Update documentation* Update documentation and remove library* Restore files* Restore file* Update build files

* Test callback* Update compiled files* Add rhtmlwidget-status attribute* wording change

* Candidate fix* Bump version

* Use latest bundle 2.35.2* Turn off warning for barcornerradius* Add documentation* Only run standard tests* Remove github workflow* Remove travis CI config* Use same circle CI config as other flip packages* Replace with default config* Try to turn off all tests* Stop returning error* Turn off build