merging rmarkdown branch with master

[xiangnandang]

merge rmarkdown branch with master to allow further development of the package to support rmarkdown documents with shiny runtime

[daattali]

Thanks. That code looks like it was written in a hurry (understandably - Andrew was just writing it as a quick proof of concept), I don't feel comfortable merging it at its state. The code style is poor, I see there's a paramter header_file with a default value but does that default value work? There is no documentation saying how to use this function, and the rmd_in_header function is using the old style of manually adding all the CSS styles in the code (see this refactor).

I don't want to add any bad code into the repository because then it becomes a maintenance liability. If you have some time you can start going through and cleaning this up. When I find time to devote to this package again I can do it too.

[daattali]

Related to https://github.com/daattali/shinycssloaders/issues/8

[xiangnandang]

thanks, that makes sense. I'll go through and clean up at a slight later time.

[daattali]

Hi @xiangnandang would you like to come back to this PR and fix it up?

[daattali]

@xiangnandang I want to do some cleanup of old PRs - do you think you'll have time in 2023 to resolve this PR, or should we abandon it?

[daattali]

Closing due to inactivity. I'd be happy to accept a PR if someone wants to take this up in the future.