-
Notifications
You must be signed in to change notification settings - Fork 113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Blog post for tibble 2.0.1 #266
Conversation
Co-Authored-By: krlmlr <krlmlr@users.noreply.github.com>
@brodieG: Enabling colored HTML output using |
I can look at it more later, but at a glance looks like your chunk options are not right. For example, you need |
I tried just building the document with I'm guessing its a CSS conflict of some sort, unless maybe you don't have the latest version of One other thing that would be helpful: can you report what |
crayon reports 256 colors, I have fansi 0.4.0 on my system. The site is rendered using I don't understand why the problem happens with some but not all tibbles in the output. I'll turn off colors as a stop gap, perhaps we can find a solution and update the blog post later. |
What's the timeline for merging and communicating this? There's no need to rush, binaries for some platforms haven't been built yet. I'm planning a 2.0.1 release fairly soon, but I'd like to wait before all binaries are built. |
OTOH, I think the binaries for R-release and R-devel will never be built due to the incompatibility with rlang 0.3.1. Let's wait for tibble 2.0.1 then. |
Rename your file *.Rmd, not *.Rmarkdown. I believe the latter triggers a blackfriday or some such markdown->html translation, which #?##! gratuitously substitutes HTML line breaks for new lines. If you use Rmd I think you get the normal pandoc conversion directly to HTML. I am not 100% certain this is what's happening because debugging the blogdown -> knitr -> rmarkdown -> pandoc chain (or any permutations of them) is about as pleasant and easy as pulling my nails out with wooden spoons and since the change was enough to make it work for me I do not want to spend one additional second confirming the exact nature of the failure. However, this seems very likely because using Rmarkdown files leave .markdown artifacts, whereas using Rmd files leave html artifacts. Please let me know if the change does not fix it for you. I really really hope it does since I was both able to reproduce the problem and fix it by making the change. |
@krlmlr there's no rush, but at the same time, I don't think including coloured output is that important if it requires a very different setup to the rest of the blog. @brodieG ah bummer — we deliberately use |
I don't have the courage to mess with blackfriday, but it does look like there is an option that might be causing the problem: What's concerning though is that it is off by default, so if it is indeed this it was presumably turned on for some reason. That aside, it is possible to have TOCs in Rmd. If the positioning of the existing TOC in the sidebar is CSS, then it should be possible to do the same with the Rmd generated one. If not looks like Garrick has examined the problem. But I agree with @hadley that this is not worth doing, especially when the colors in this case are so subtle. |
I tweaked a bit for 2.0.1, should be ready to go. Thanks again Brodie for looking into it, sorry for the pain. In the long run I'd love to see color in console output. Would it work if |
Coloring works now, examples look good to me. |
I'm planning to add links to documentation for the funcitons mentioned in the text, unless there's some pkgdown magic which I could use.