Skip to content

Added Tokyo Night Storm theme - #153

Open
ivanskodje wants to merge 1 commit into
omacom:masterfrom
ivanskodje:tokyo-night-storm
Open

Added Tokyo Night Storm theme#153
ivanskodje wants to merge 1 commit into
omacom:masterfrom
ivanskodje:tokyo-night-storm

Conversation

@ivanskodje

@ivanskodje ivanskodje commented Sep 1, 2026

Copy link
Copy Markdown

Added Tokyo Night Storm theme.

tokyo-night-storm

There is also an optional bash script in the repo that can fix the lack of themeing on Files (Nautilus).
It changes with your Omarchy theme. See the README.

@omarchybot

Copy link
Copy Markdown
Collaborator

Reviewed. The theme repo link resolves (HTTP 200, both anchors), assets/themes/tokyo-night-storm.webp is committed at exactly the path the entry references (1200x675, 80KB) rather than hotlinked, the figure block matches its neighbours attribute for attribute, and the entry is correctly placed between Tokyo Night OLED and Tycho. The existing Tokyo Night OLED card is a different theme by a different author and is untouched, so this is not a duplicate.

One thing to flag, and it is not a problem with your PR: #152 (Tron) inserts at exactly the same point in the file, so whichever of the two merges second will need a rebase. I confirmed that with a real test merge. Alphabetically Tokyo Night Storm sorts before Tron, so the settled order would be Tokyo Night OLED, Tokyo Night Storm, Tron, Tycho.

Nothing else found. Checked by Claude Opus 5 with an independent second opinion from Codex at xhigh reasoning, run once across tonight's nine theme PRs; it ran the full 36-pair merge matrix itself and found this to be the only collision in the batch. This repo has no build or test suite — the only CI is the Pages deploy — so nothing was executed, and the optional Nautilus script in your theme repo is outside this diff and was not run. Waiting on the maintainer.

@omarchybot omarchybot mentioned this pull request Sep 2, 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