Skip to content

Theme: Corporate - #46

Open
brianblakely wants to merge 1 commit into
omacom-io:masterfrom
brianblakely:master
Open

Theme: Corporate#46
brianblakely wants to merge 1 commit into
omacom-io:masterfrom
brianblakely:master

Conversation

@brianblakely

Copy link
Copy Markdown
Contributor

No description provided.

@omarchybot

Copy link
Copy Markdown
Collaborator

Reviewed. Nothing blocking.

The added block matches the neighbouring entries byte for byte (indentation, loading="lazy" decoding="async", alt="Corporate theme", same URL in both links), src matches the file the commit adds, and the entry sits correctly between Cobalt2 and CpUnk.

The preview checks out exactly: running the conversion documented in the comment above the list — magick preview.png -strip -resize '1200>' -quality 80 — on your repo's 3840x2160 preview.png reproduces the committed corporate.webp byte for byte, 1200x675 at 100,200 bytes. That is the twelfth largest of the previews on the page, inside an existing range of 19,816 to 179,490 bytes, so the size is just what a 4K screenshot of a busy desktop compresses to under the documented recipe — nothing to change.

brianblakely/omarchy-corporate-theme is public and lays out a real Omarchy theme at its root — colors.toml with mode = "light", alacritty.toml, backgrounds/, icons.theme, keyboard.rgb — so pasting the URL into Install > Style > Theme resolves to slug corporate and works.

The pull request has no description. Not a blocker, since the diff carries the link, but a line saying what the theme is helps whoever reads the history later.

Merges cleanly onto master alongside #41, #42, #43, #44, #45 and #47 — I merged all seven locally to check, including across the older merge base this branch sits on. The only conflicting pair in the batch is #41 with #42.

Tests: none run, and there are none to run — omarchy-site has no test suite or link checker, and its only workflows are GitHub's own Copilot reviewer, CodeQL and Pages deploy. Instead I parsed the merged page: tags balance, all 129 figure blocks match the canonical shape, and every img src resolves to a file in assets/themes/.

Second opinion from codex at xhigh: it found no defect either, agreeing with what I had already concluded; its independence is not currently guaranteed. It had no network access, so the theme-repo and image-provenance checks above are mine.

Waiting on the maintainer. Not merged and not approved; that call is theirs.

@omarchybot omarchybot mentioned this pull request Aug 22, 2026
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