Follow up to #2262: add a clarifying comment; update news

Author: cpsievert

NEWS.md

@@ -4,6 +4,10 @@
 
 * Closed #2216: Additional selectize.js options can now be passed along to `highlight()`'s `selectize` argument. (#2217)
 
+## Improvements
+
+* Closed #2259: `ggplotly()` now provides better support for ggplot2 >v3.4.2. (#2262)
+
 ## Bug fixes
 
 * Closed #2212: `ggplotly()` no longer silently drops legends that are customized through `ggplot2::guide_legend()`.

R/layers2traces.R

@@ -103,6 +103,8 @@ layers2traces <- function(data, prestats_data, layout, p) {
   # now to the actual layer -> trace conversion
   trace.list <- list()
 
+  # ggplot2 >v3.4.2 (specifically #4879) moved the guides system to ggproto, 
+  # which moved the location of the name->value fields
   guides <- if (inherits(p$guides, "ggproto")) p$guides$guides else p$guides
   aes_no_guide <- names(guides)[vapply(guides, identical, logical(1), "none")]