Skip to content

refactor: simplify the quartoorg site code - #2298

Draft
mcanouil wants to merge 4 commits into
quarto-dev:mainfrom
mcanouil:refactor/simplify-quartoorg
Draft

mcanouil wants to merge 4 commits into
quarto-dev:mainfrom
mcanouil:refactor/simplify-quartoorg

Conversation

@mcanouil

@mcanouil mcanouil commented Oct 17, 2026 •

Copy link
Copy Markdown
Collaborator

Merge order: #2140 first, then #2298, then the restyle with design tokens, which isn't open yet.

This tidies leftovers from the move of the site theme into the quartoorg extension in #2222.
The rendered site doesn't change.

  • The light and dark themes now share one #quarto-announcement rule in _announcement.scss, and the prerelease banner only overrides its background colour.
  • The navbar drops the unused .bi-twitter selector.
  • page-tweaks.html detects macOS from navigator.userAgent instead of the deprecated navigator.platform. Macs and iPads in desktop mode still get ⌘, and other devices ⌃.
  • The same script now marks .pymd comments with a class, and the font-weight lives in the stylesheet instead of an inline style.
  • _cards.scss drops the .carousel* and .gallery-category rules, which only the old gallery page uses.

Four files in images/ go. Nothing references them, and git history shows why:

  • hello-jupyter.bak.png and hello-julia.bak2.png are backups of older renders, saved in d1c85bf and 74b66ba.
  • demo-jupyter.png (0596468) and hello-jupyter-notebook.png (404c16d) were home page drafts from March 2023, replaced within two days by demo-jupyter-plain.png (47b63d7), which is still in use.

Screenshots of 15 pages and the prerelease banner, in light and dark at three widths, match main.
Two of them, pixel-identical:

Prerelease banner and navbar in light and dark, main on the left and this branch on the right, identical Dashboard code block with bold comment lines in light and dark, main on the left and this branch on the right, identical

The compiled CSS differs only by the removed selector, the new class rule, the dropped duplicate declarations and the removed card rules.

It merges after #2140, because #2140 replaces the old gallery page that used the removed card rules.
Both add an import to _site.scss, so the rebase onto #2140 needs a one-line conflict fix.
The restyle builds on this one.

The light and dark themes now share one announcement banner rule, and the
prerelease banner only overrides its colour. The navbar drops a selector
for an icon the site no longer uses. The page tweaks script detects macOS
from the user agent instead of the deprecated navigator.platform, and
marks pymd comments with a class instead of setting the font weight
inline.
Two are backups of older renders that nothing ever referenced, and two
are home page drafts from March 2023 that newer images replaced.
@mcanouil mcanouil self-assigned this Oct 17, 2026
The old gallery page was the only user of these rules, and the new gallery doesn't use them.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant