change: reorder arguments of girafe() with argument ggobj in first position

see #327

Author: davidgohel

DESCRIPTION

@@ -1,7 +1,7 @@
 Type: Package
 Package: ggiraph
 Title: Make 'ggplot2' Graphics Interactive
-Version: 0.9.2
+Version: 0.9.3.001
 Authors@R: c(
     person("David", "Gohel", , "david.gohel@ardata.fr", role = c("aut", "cre")),
     person("Panagiotis", "Skintzos", , "sigmapi@posteo.net", role = "aut"),

NEWS.md

@@ -1,3 +1,11 @@
+# ggiraph 0.9.3
+
+## Change
+
+- reorder arguments of `girafe()` with argument `ggobj` in first position. 
+IT DOES NOT makes ggplot objects pipe-able with `girafe()` (unless you add
+parenthesis around your ggplot code).
+
 # ggiraph 0.9.2
 
 ## Feature

R/girafe.R

@@ -94,8 +94,8 @@
 #' @seealso [girafe_options()], [validated_fonts()], [dsvg()]
 #' @export
 girafe <- function(
-  code,
   ggobj = NULL,
+  code,
   pointsize = 12,
   width_svg = NULL,
   height_svg = NULL,

inst/tinytest/test-girafe_options.R

@@ -267,7 +267,7 @@ library(ggiraph)
 # girafe_options ----
 {
   expect_error(girafe_options("foo"), info = "check x argument")
-  g <- girafe({
+  g <- girafe(code = {
     NULL
   })
   expect_identical(girafe_options(g), g, info = "no options set")

inst/tinytest/test-trailing-comma.R

@@ -41,7 +41,7 @@ library(ggplot2)
 
 # girafe_options, trailing comma ----
 {
-  g <- girafe({
+  g <- girafe(code = {
     NULL
   })
   expect_identical(girafe_options(g, ), g, info = "no options set")