Use litedown instead of pkgdown #7534
Conversation
ben-schwen
requested review from
MichaelChirico and
jangorecki
as code owners
|
We need to finish either this PR or #7402 since otherwise our |
|
We don't need docker image for litedown, it was needed for pkgdown because of maaany OS deps. For litedown we can use one of our other docker images. |
| '<body>', | ||
| navbar_html, | ||
| '<div class="body">', | ||
| html_content, |
There was a problem hiding this comment.
It would be great to find a way to avoid starting the HTML document from scratch here:
<div class="body"> <!-- DOCTYPE and <html>, <head> below are invalid --> <!DOCTYPE html><html><head><title>R: Integer based date class</title> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> <meta name="viewport" content="width=device-width, initial-scale=1.0, user-scalable=yes"> <link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/[email protected]/dist/katex.min.css"> <script> const macros = { "\\R": "\\textsf{R}", "\\mbox": "\\text", "\\code": "\\texttt"}; function processMathHTML() { var l = document.getElementsByClassName('reqn'); for (let e of l) { katex.render(e.textContent, e, { throwOnError: false, macros }); } return; }</script> <script defer src="https://cdn.jsdelivr.net/npm/[email protected]/dist/katex.min.js" onload="processMathHTML();"></script> <link rel="stylesheet" type="text/css" href="R.css"> <!-- starting a new <body> inside a <body> is invalid --> </head><body><div class="container"><main>
litedown::pkg_manual() does it using scary regexp hacks.
|
Somehow, in our case, the indentation added by Edit: that's easily fixed by making the first heading of |
In order to use fancy left-side TOC, we'll need
meta:
css: ['@default', '@Article']
js: ['@sidenotes']
in the YAML header and
.body {
max-width: 800px;
}
in the global site styles.
| @@ -1,4 +1,4 @@ | |||
| ## data.table news and updates | |||
| # data.table news and updates | |||
There was a problem hiding this comment.
I think there was a reason for that, @MichaelChirico any idea?
|
On Sat, 17 Jan 2026 23:46:49 -0800 Jan Gorecki wrote:
I think there was a reason for that, @MichaelChirico any idea?
The use of <h2> instead of <h1> in the first heading of the NEWS was
required to work around r-lib/pkgdown#2897.
|
3 participants
Closes #6584
Much appreciated if someone with more web dev skills than me can fix .css styling :D
Whats still missing to be fully equivalent to pkgdown