Skip to content

Fix invalid clock glyph in time notification#5957

Open
stefanomainardi wants to merge 1 commit into
basecamp:devfrom
stefanomainardi:fix/notification-time-glyph
Open

Fix invalid clock glyph in time notification#5957
stefanomainardi wants to merge 1 commit into
basecamp:devfrom
stefanomainardi:fix/notification-time-glyph

Conversation

@stefanomainardi
Copy link
Copy Markdown

Summary

Fixes the "invalid icon" rendered for the Show time notification (SUPER + CTRL + ALT + T) reported in #5776.

Root cause

bin/omarchy-notification-time passed -g <U+F43A> to omarchy-notification-send. That codepoint is in the Nerd Font legacy MDI range (3-byte UTF-8, EF 90 BA). The mako default font=sans-serif 14px does not reliably fall through to a Nerd Font for PUA characters in that range, so the glyph renders as a blank box / tofu.

Every other glyph used by omarchy notifications lives in the modern MDI range (4-byte UTF-8, U+F0000U+F1AB9). Examples already in tree:

Glyph Codepoint Name Used in
󰍹 U+F0379 nf-md-monitor omarchy-hyprland-monitor-internal
󱂬 U+F30AC nf-md-monitor_dashboard omarchy-hyprland-workspace-layout-toggle
󰴑 U+F0D11 nf-md-content_paste omarchy-capture-text-extraction
󱫖 U+F1AD6 nf-md-sleep_off omarchy-toggle-idle
󰂛 U+F009B nf-md-bell_off_outline omarchy-toggle-notification-silencing
󱐋 U+F340B nf-md-flash omarchy-battery-monitor

The time notification was the only outlier on the legacy range — making it the only one that fails to render on a stock mako setup.

Change

-omarchy-notification-send -g  -u low "$(date +"%A %H:%M  ·  %d %B %Y  ·  Week %V")"
+omarchy-notification-send -g 󰅐 -u low "$(date +"%A %H:%M  ·  %d %B %Y  ·  Week %V")"

󰅐 is nf-md-clock-outline (U+F0150) — same MDI family used elsewhere.

Note for master backport

The same buggy glyph exists on master in default/hypr/bindings/utilities.conf:55 (inline notify-send call). If you'd like the fix backported there too, happy to open a follow-up PR — just say the word.

Test plan

  • bash -n bin/omarchy-notification-time — syntax clean
  • Verified omarchy-notification-send -g <glyph> accepts the new value (script signature unchanged)
  • Confirmed JetBrainsMono Nerd Font (omarchy default) ships U+F0150 (fc-match -s :charset=F0150)

The "Show time" notification used U+F43A — in the Nerd Font legacy MDI
codepoint range (3-byte UTF-8). With mako's default `font=sans-serif`
this PUA character does not always fall through to a Nerd Font, so the
icon renders as a "tofu" / blank box (reported in basecamp#5776).

Every other glyph in omarchy (monitor, clipboard, idle, silenced
notifications, recharge, etc.) is in the modern MDI range (4-byte UTF-8,
U+F0000–U+F1AB9). Replace the legacy clock with `nf-md-clock-outline`
(U+F0150, 󰅐) so it follows the same convention as the rest.

Closes basecamp#5776.
Copilot AI review requested due to automatic review settings May 23, 2026 17:46
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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