Skip to content

Switch default monospace font from JetBrains Mono to AdwaitaMono Nerd Font#271

Closed
Copilot wants to merge 2 commits into
mainfrom
copilot/drop-font-diff-and-update-to-adwaita
Closed

Switch default monospace font from JetBrains Mono to AdwaitaMono Nerd Font#271
Copilot wants to merge 2 commits into
mainfrom
copilot/drop-font-diff-and-update-to-adwaita

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 13, 2026

Drops our custom monospace font diff from upstream GNOME by replacing JetBrains Mono with the nerd-fonts-patched build of GNOME's own Adwaita Mono, preserving Nerd Font icon support in terminals.

Changes

  • Containerfile: Add xz to the Alpine build deps; download and extract AdwaitaMono.tar.xz from nerd-fonts v3.4.0 into usr/share/fonts/AdwaitaMono/ in the shared system layer
  • zz0-bluefin-modifications.gschema.override: Swap monospace-font-name:
    -monospace-font-name="JetBrains Mono 16"
    +monospace-font-name="AdwaitaMono Nerd Font Mono 13"
    Uses the Mono (single-width) variant; size 13 is closer to GNOME defaults than the previous 16.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/ryanoasis/nerd-fonts/releases/tags/v3.4.0
    • Triggering command: /usr/bin/curl curl -fsSL REDACTED (http block)
  • just.systems
    • Triggering command: /usr/bin/curl curl -fsSL https://just.systems/install.sh (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI linked an issue Apr 13, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Remove font differences and set Adwaita Mono Nerd Font as default Switch default monospace font from JetBrains Mono to AdwaitaMono Nerd Font Apr 13, 2026
Copilot AI requested a review from hanthor April 13, 2026 03:30
@ehsanullahjan
Copy link
Copy Markdown
Contributor

bluefin currently includes adwaita-fonts-all and jetbrains-mono-fonts-all. these normal fonts are then patched by nerd-fonts from che/nerd-fonts.

unfortunately, che/nerdfonts have not yet updated their package for adwaita mono. i just left them a comment requesting an update for the font (a config change and then a rebuild).

ideally, you'd want to follow the same approach (install normal font, patch with nerd-fonts). however, the fact that adwaita mono is not yet patched, a direct download from the nerd fonts website is a fine stop gap.

once this is merged in common, are you planning to remove the corresponding code from bluefin?

Comment thread Containerfile
Comment on lines +21 to +24
RUN install -d /out/shared/usr/share/fonts/AdwaitaMono && \
curl -fsSLo /tmp/AdwaitaMono.tar.xz https://github.com/ryanoasis/nerd-fonts/releases/download/v3.4.0/AdwaitaMono.tar.xz && \
tar -xJf /tmp/AdwaitaMono.tar.xz -C /out/shared/usr/share/fonts/AdwaitaMono && \
rm /tmp/AdwaitaMono.tar.xz
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There's got to be an official way to get this font from GNOME itself?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

gnome already provides adwaita fonts by default. the mono font even comes with nerd font icons built-in. except, the git branch icon doesn't render correctly.

we could just remove adwaita and jetbrains fonts from packages.sh to align with upstream. plus, maybe the git branch icon getting chopped should be a bug report to gnome?

see the stock silverblue 43 image below for reference:

Screenshot From 2026-04-30 14-51-13

@castrojo
Copy link
Copy Markdown
Contributor

SGTM, is there a fallback? I know in the past this would break people's terminals font until they fixed it.

@ehsanullahjan
Copy link
Copy Markdown
Contributor

ehsanullahjan commented Apr 30, 2026

wdym by fallback font? the bundled adwaita mono works fine except the git symbol as far as i know.

rn we're explicitly installing adwaita-fonts-all which is unnecessary as upstream already bundles it. if we move to adwaita, the git symbol will be broken for starship. this will break ux for users since we ship starship by default. so, this doesn't look like a good option until either upstream fixes the font or we replace it with its patched version from https://www.nerdfonts.com/.

in order to not degrade the ux, here's what i suggest:

  • open a bug with upstream to fix adwaita mono
  • don't explicitly install adwaita-fonts-all (provided by upstream already)
  • continue shipping jetbrains font
  • continue shipping che/nerd-fonts (necessary for nerd symbols)
  • reduce mono font size to a more reasonable value (e.g., 11, 12, or 13 instead of 16) [upstream's default mono size is 11]

once upstream fixes their font, we can back out jetbrains and set adwaita mono as the default font for new installations. also not that upstream ships a number of monospace fonts like noto sans mono, liberation mono, dejavu sans mono, etc. we could also pick one of them as the default instead of shipping jetbrains font, if desired.

@ehsanullahjan
Copy link
Copy Markdown
Contributor

Opened issue upstream: https://gitlab.gnome.org/GNOME/adwaita-fonts/-/work_items/27.

@ehsanullahjan
Copy link
Copy Markdown
Contributor

upstream has closed the ticket - they won't provide a fix. so, @castrojo, your call as to how you'd like this tackled. we could just keep jetbrains mono as the default monospace font, but maybe reduce size to 11, 12 or 13. alternatively, we could just go with upstream adwaita mono as the default, and potentially remove jetbrains mono, if desired.

i've used both the upstream adwaita mono and nf adwaitamono nerd font extensively. the only difference i've ever noticed is in rendering of the starship prompt (the > symbol) and the git branch symbol. the images below show both versions.

upstream adwaita mono font:

Screenshot From 2026-05-05 20-46-51

nf adwaitamono nerd font:

Screenshot From 2026-05-05 20-47-00

the two symbols noted render slightly differently but i'm not quite sure if anything is broken. i think we can 100% go with the upstream font, no issue.

let me know what you think.

@pfanzola
Copy link
Copy Markdown
Contributor

pfanzola commented May 7, 2026

  • reduce mono font size to a more reasonable value (e.g., 11, 12, or 13 instead of 16) [upstream's default mono size is 11]

could be also useful to reduce the size of "Interface" font since in upstream it's 11.
Screenshot From 2026-05-07 10-04-12

default parameters

@castrojo
Copy link
Copy Markdown
Contributor

castrojo commented May 8, 2026

Thanks for all the work! Let's fix the font size to be normal. This is an artifact of when it was a personal image of mine and I roll with large fonts. The new font slider thing in GNOME is awesome so we can just ship stock.

Let's keep Jetbrains Mono for now it's got the full nerd coverage and it's still an awesome font.

@ehsanullahjan
Copy link
Copy Markdown
Contributor

ok, i'm going to create a pull request with the following config shortly:

font-name="Adwaita Sans 11"
document-font-name="Adwaita Sans 12"
monospace-font-name="JetBrains Mono 11"

ehsanullahjan added a commit to ehsanullahjan/bluefin-common that referenced this pull request May 8, 2026
Adjust size of system fonts to match default upstream settings [1].
We're sticking with JetBrains Mono as the default monospace font since
the upstream Adwaita Mono font renders nerd font symbols differently to
all other nerd fonts [2].

This addresses projectbluefin#266 and
supersedes projectbluefin#271.

[1]
projectbluefin#271 (comment)
[2]
projectbluefin#271 (comment)
Copy link
Copy Markdown
Member

@hanthor hanthor left a comment

Choose a reason for hiding this comment

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

Manually verified: download URL returns 200, and AdwaitaMono Nerd Font Mono is the correct font family name inside the tarball. CI build passes. LGTM — ready to merge once promoted from draft.

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label May 28, 2026
Copy link
Copy Markdown
Member

@hanthor hanthor left a comment

Choose a reason for hiding this comment

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

Switching to AdwaitaMono Nerd Font stays close to upstream GNOME while preserving Nerd Font terminal icons. CI passes, already approved. Approved.

Copy link
Copy Markdown
Member

@hanthor hanthor left a comment

Choose a reason for hiding this comment

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

Switching to AdwaitaMono Nerd Font stays close to upstream GNOME while preserving Nerd Font terminal icons. CI passes, already approved. Approved.

@castrojo
Copy link
Copy Markdown
Contributor

🤖 Copilot Test Report

Branch: copilot/drop-font-diff-and-update-to-adwaita | ⚠️ 121 commits behind main | Tested: 2026-05-30T04:50Z

Test Results

Test Result
just check (syntax validation) ✅ PASS
Branch freshness ⚠️ 121 commits behind main

Note

This is the oldest open PR in the queue. The font override change has since been merged separately (see #348 which is at HEAD and also carries lgtm). Reviewers should confirm whether this PR is still needed or if #348 supersedes it entirely.

⚠️ Action Required

Rebase onto main before merge, and confirm there's no duplicate work with #348.

@castrojo
Copy link
Copy Markdown
Contributor

Closing — superseded by #348 which drops the same monospace font override and is already at HEAD. The actual change here is identical; this branch is 121 commits behind main and has a merge conflict.

@castrojo castrojo closed this May 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Drop our font diff entirely

5 participants