knit_print is not the default currently

yihui / printr

Some (magical) printing methods for knitr

https://yihui.org/printr/

118 stars 24 forks source link

knit_print is not the default currently #27

Closed rubenarslan closed 7 years ago

rubenarslan commented 7 years ago

This is just a minor note, but render currently defaults to NULL, not to knit_print as you say here and in the vignette. You do a bit of black magic for htmlwidgets, before passing things along to knit_print. Or maybe it's not black magic, but it tripped me up when I tried to handle things with pander where possible.

yihui commented 7 years ago

Sorry, it is black magic, unfortunately...

rubenarslan commented 7 years ago

Does this implementation replicate the black magic sufficiently? Sorry, I'm sure this is a very niche application, but it's much easier for my students if things pretty-print automagically.

yihui commented 7 years ago

For now, yes. The problem is that I don't quite like my own black magic, and I cannot say for sure that I won't change it in the future...