Skip to content

fix: code padding font height#232

Merged
flying-sheep merged 2 commits into
mainfrom
pa/code-style
Jul 13, 2026
Merged

fix: code padding font height#232
flying-sheep merged 2 commits into
mainfrom
pa/code-style

Conversation

@flying-sheep

@flying-sheep flying-sheep commented Jul 13, 2026

Copy link
Copy Markdown
Member

Before and after. Check out

  • the font height of inline code compared to surrounding text – this is now clearly a capital X and not a lowercase one.
  • the padding of the code block no longer looks ugly
image

TODO:

  • revert adding the background to inline code? It looks good except that it interferes with that parenthesis in “(obsm, varm)”

@netlify

netlify Bot commented Jul 13, 2026

Copy link
Copy Markdown

Deploy Preview for jade-cajeta-1bcca0 ready!

Name Link
🔨 Latest commit 39b53e2
🔍 Latest deploy log https://app.netlify.com/projects/jade-cajeta-1bcca0/deploys/6a54cd47feb8fd0008f42102
😎 Deploy Preview https://deploy-preview-232--jade-cajeta-1bcca0.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@flying-sheep flying-sheep requested a review from Zethson July 13, 2026 11:39

@Zethson Zethson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

Comment thread assets/main.scss
src: url("fonts/Inter/Inter-VariableFont_slnt,wght.ttf");
}
@font-face {
font-family: "JetBrains Mono";

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yay, I love Jetbrains Mono.

@muskanhashim I wonder though whether we should standardize on a scverse font though. Just a FYI for now.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We use Inter, so I had Claude check the metrics for a few fonts and this one matches Inter very well in Cap- and x-height.

I’m not that much of a font nerd so I can’t speak to if its “soul” or whatever matches Inter. If someone cares: IBM Plex Mono would also work metrically.

@Zethson

Zethson commented Jul 13, 2026

Copy link
Copy Markdown
Member

revert adding the background to inline code? It looks good except that it interferes with that parenthesis in “(obsm, varm)”

Not sure - no strong opinion

@flying-sheep flying-sheep merged commit e3cb3c5 into main Jul 13, 2026
6 checks passed
@flying-sheep flying-sheep deleted the pa/code-style branch July 13, 2026 11:50
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.

2 participants