miniUI required but not installed

[KasperSkytte]

Hi!

The miniUI package is required but not installed with the listviewer package. Maybe it should be?

if (!require(listviewer)) install.packages("listviewer")
#> Loading required package: listviewer
library(listviewer)
list <- list(a = 1, b = 2)
jsonedit_gadget(list)
#> Loading required namespace: miniUI
#> Failed with error:  ‘there is no package called ‘miniUI’’
#> Error in jsonedit_gadget(list): requireNamespace("miniUI") is not TRUE

[timelyportfolio]

Thanks so much for the report, but this is intentional to reduce the set of required dependencies. In the case of jsonedit_gadget, I consider this as non-core and ancillary. Hope this does not cause too much trouble.

[KasperSkytte]

Alright. Mind if I make a PR where the user will be asked interactively to install the packages if they are not when they use jsonedit_gadget?

[timelyportfolio]

@KasperSkytte thanks so much for this. I expect to merge, but for some reason I feel like CRAN does not like for a package to be able to installed from another package in this way. I'll test locally to insure that this still passes CRAN checks, and if so will happily merge your pull request. Thanks again!!

[KasperSkytte]

Alright, let's see. You're welcome and thank you too!