clean up wording

plotly · cpsievert · Jan 25, 2018 · Jan 11, 2018 · Jan 11, 2018 · Jan 11, 2018
commit 35fe715b8b789dbf376b7930b2fdbce9b13372a1
diff --git a/NEWS.md b/NEWS.md
@@ -2,8 +2,8 @@
 
 ## NEW FEATURES & IMPROVEMENTS
 
-* Upgraded to plotly.js v1.33.0. A _huge_ amount of features and improvements have been made since v1.29.2 (i.e., the version included in the last CRAN release of the R package - v4.7.1). Read the complete list of changes here: <https://github.com/plotly/plotly.js/releases>.
-* Persistent selection can now be enabled via the 'shift' key. It's still possible to _force_ persistent selection by setting `persistent = TRUE` in `highlight()`, but `persistent = FALSE` (the default) is now recommended since it allows one to switch between [transient/persistent selection](https://plotly-book.cpsievert.me/linking-views-without-shiny.html#transient-versus-persistent-selection) in the browser, rather than at the command line.
+* Upgraded to plotly.js v1.33.0. A _huge_ amount of features and improvements have been made since v1.29.2 (i.e., the version included in the last CRAN release of the R package - v4.7.1). Highlights include a complete re-write of `scattergl` to make it nearly feature complete with `scatter`, localization of text rendering (i.e., international translations), and two new trace types (`violin` & `table`). Read more about the v1.32.0 release [here](https://codeburst.io/notes-from-the-latest-plotly-js-release-b035a5b43e21) and the complete list of changes [here](https://github.com/plotly/plotly.js/releases).
+* The selection mode can now switch from 'transient' to 'persistent' by holding the 'shift' key. It's still possible to _force_ persistent selection by setting `persistent = TRUE` in `highlight()`, but `persistent = FALSE` (the default) is now recommended since it allows one to switch between [persistent/transient selection](https://plotly-book.cpsievert.me/linking-views-without-shiny.html#transient-versus-persistent-selection) in the browser, rather than at the command line.
 
 ## CHANGES
 

diff --git a/R/highlight.R b/R/highlight.R
@@ -102,12 +102,12 @@ highlight <- function(p, on = "plotly_click", off,
     stop("opacityDim must be between 0 and 1", call. = FALSE)
   }
   if (dynamic && length(color) < 2) {
-    message("Adding more colors to the selection color palette")
+    message("Adding more colors to the selection color palette.")
     color <- c(color, RColorBrewer::brewer.pal(4, "Set1"))
   }
   if (!dynamic && length(color) > 1) {
     warning(
-      "Can only use a single color for selections when dynamic=FALSE",
+      "Can only use a single color for selections when `dynamic = FALSE`.",
       call. = FALSE
     )
     color <- color[1] 
@@ -139,10 +139,9 @@ highlight <- function(p, on = "plotly_click", off,
 
   if (isTRUE(persistent)) {
     message(
-      "Persistent selection can now be enabled via the 'shift' key. ",
-      "Thus, it's recommended to use the default (persistent = FALSE) since ",
-      "this allows one to switch between persistent/transient selection ",
-      "'on-demand' (in the browser)."
+      "We recommend setting `persistent` to `FALSE` (the default) because ",
+      "persistent selection mode can now be used by holding the shift key ",
+      "(while triggering the `on` event)."
     )
   }