This is unfortunately getting to be more not less complicated. Your
approach might be 100% correct however I'm regrettably not able to put
enough time into digesting it to ensure it's an ok patch. I'd be in favor
of just adding a #!header# field after the existing #!mathjax# field and
being done with it. Having the custom header content have the last word
over markdown_css is clear and simple, and I think the right approach.


On Thu, Apr 18, 2013 at 9:02 AM, Jim Hester <notifications@github.com>wrote:

> In inst/resources/markdown.html:
>
> > @@ -6,14 +6,16 @@
> >
> >  <title>#!title#</title>
> >
> > -<style type="text/css">
> > -#!markdown_css#
> > -</style>
> > -
>
> Well we already have the skip_style option, but it is not currently being
> used. I made a quick commit using it, see what you think.
> … <#13e1d3cc244107a8_>
> On Thu, Apr 18, 2013 at 8:40 AM, JJ Allaire <notifications@github.com>wrote:
> In inst/resources/markdown.html: > @@ -6,14 +6,16 @@ > >
> <title>#!title#</title> > > -<style type="text/css"> > -#!markdown_css# >
> -</style> > - I think we should keep it simple -- if you want to override
> what's in markdown_css you can just override it in the header (if you want
> to keep most of it you can just copy and paste). If we have too many
> cascading options then people will by and large not be able to figure out
> how to use the mechanism. If you move the header below markdown_css then
> I'll merge the pull request. — Reply to this email directly or view it on
> GitHub<https://github.com/rstudio/markdown/pull/23/files#r3852329> .
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/rstudio/markdown/pull/23/files#r3852721>
> .
>