kcuilla / reactablefmtr

Streamlined Table Styling and Formatting for Reactable
https://kcuilla.github.io/reactablefmtr/
Other
208 stars 27 forks source link

icon_assign creates problem with RMarkdown report #51

Open erdnaxel opened 1 year ago

erdnaxel commented 1 year ago

I have an RMarkdown report that I've been running for a few months without any issue. Today, it started giving me this error message when I knitted it: image

After investigating, I found that the problem was created by the icon_assign() function used in one of the tables.

I solved my problem, but I thought it might be useful to report it.

I can still run the code in R and the problem only arises when knitting. That said, the circle isn't filled any more: image

image

Thanks for a great pacakge!

riskcede commented 1 year ago

Hi Everyone, as mentioned above, thanks for this very useful package!!

I believe I have the same problem. I get an error message stating that htmltools dont use shiny tag list anymore.

bindFillRole()` only works on htmltools::tag() objects (e.g., div(), p(), etc.), not objects of type 'shiny.tag.list'

It works fine in rstudio, but knitting with Rmarkdown fails.

Any advice?

Here is my code as well as environment variables:

library(palmerpenguins) library(tidyverse) library(reactablefmtr) library(reactable)

data <- sample_n(penguins, 50) %>% filter(!is.na(bill_length_mm)) %>% select(species, bill_length_mm, bill_depth_mm, flipper_length_mm, body_mass_g)

reactable( data, columns = list( bill_length_mm = colDef(cell = icon_sets(data, icons = c("ruler"))), bill_depth_mm = colDef(cell = icon_sets(data, icons = c("ruler-vertical"))), flipper_length_mm = colDef(cell = icon_sets(data, icons = c("ruler-horizontal"))), body_mass_g = colDef(cell = icon_sets(data, icons = c("balance-scale-left", "balance-scale", "balance-scale-right"))) ) )

R version 4.1.0 (2021-05-18) Platform: x86_64-pc-linux-gnu (64-bit) Running under: Ubuntu 18.04.5 LTS

Matrix products: default BLAS: /usr/lib/x86_64-linux-gnu/blas/libblas.so.3.7.1 LAPACK: /usr/lib/x86_64-linux-gnu/lapack/liblapack.so.3.7.1

locale: [1] LC_CTYPE=C.UTF-8 LC_NUMERIC=C LC_TIME=C.UTF-8 LC_COLLATE=C.UTF-8 LC_MONETARY=C.UTF-8 LC_MESSAGES=C.UTF-8 LC_PAPER=C.UTF-8
[8] LC_NAME=C LC_ADDRESS=C LC_TELEPHONE=C LC_MEASUREMENT=C.UTF-8 LC_IDENTIFICATION=C

attached base packages: [1] stats graphics grDevices utils datasets methods base

other attached packages: [1] reactablefmtr_2.0.0 reactable_0.2.3 forcats_0.5.2 stringr_1.4.1 dplyr_1.0.10 purrr_0.3.5 readr_2.1.3
[8] tidyr_1.2.1 tibble_3.1.8 ggplot2_3.4.0 tidyverse_1.3.1 palmerpenguins_0.1.1

loaded via a namespace (and not attached): [1] Rcpp_1.0.9 lubridate_1.9.0 assertthat_0.2.1 digest_0.6.30 utf8_1.2.2 mime_0.12 R6_2.5.1 cellranger_1.1.0 reactR_0.4.4
[10] backports_1.2.1 reprex_2.0.1 httr_1.4.4 pillar_1.8.1 rlang_1.0.6 readxl_1.3.1 rstudioapi_0.14 fontawesome_0.4.0 htmlwidgets_1.6.1 [19] munsell_0.5.0 shiny_1.7.4 broom_1.0.1 compiler_4.1.0 httpuv_1.6.6 modelr_0.1.8 pkgconfig_2.0.3 htmltools_0.5.4 tidyselect_1.2.0 [28] fansi_1.0.3 crayon_1.5.2 tzdb_0.3.0 dbplyr_2.1.1 withr_2.5.0 later_1.3.0 grid_4.1.0 jsonlite_1.8.3 xtable_1.8-4
[37] gtable_0.3.1 lifecycle_1.0.3 DBI_1.1.2 magrittr_2.0.3 scales_1.2.1 cli_3.4.1 stringi_1.7.8 fs_1.5.2 promises_1.2.0.1 [46] xml2_1.3.3 ellipsis_0.3.2 generics_0.1.3 vctrs_0.5.1 tools_4.1.0 glue_1.6.2 hms_1.1.2 crosstalk_1.2.0 fastmap_1.1.0
[55] yaml_2.3.6 timechange_0.1.1 colorspace_2.0-3 rvest_1.0.1 haven_2.4.3 sass_0.4.4

tashapiro commented 1 year ago

@kcuilla any updates on this issue? I noticed this issue recently with react_sparkline as well. Is it an issue with a package dependency update? Did some googling, not a lot on this topic aside from this thread. Appreciate your help!

dleopold commented 1 year ago

I am getting the same warning in a shiny application using with react_sparkbar().