Skip to content

Add wezterm terminal config and support theme switcher#2680

Open
dansdantas wants to merge 1 commit intobasecamp:devfrom
dansdantas:add-wezterm
Open

Add wezterm terminal config and support theme switcher#2680
dansdantas wants to merge 1 commit intobasecamp:devfrom
dansdantas:add-wezterm

Conversation

@dansdantas
Copy link

No description provided.

@martin-braun
Copy link

I want to try Omarchy and love the theme switcher. I am a wezterm user, so a deep integration into wezterm is actually what I appreciate when finally starting out. I hope this PR can land sometime. 🙏

Thank you @dansdantas! You actually took the time to patch all the themes that ship with Omarchy? That's just awesome, but it will require other theme maintainers to add wezterm support as well.

I also have have 3 more questions of which one is off-topic:

  1. If I install Omarchy, how can I test a PR like that? I don't think I'm ready to review anything right now, but it would be good to know.
  2. Looking at the code I don't see where wezterm is added to the actual terminal selection of the main menu which allows you to do most of the stuff out of the box. To me it looks like you now support wezterm in the theme switcher, but still require it to install it conventionally. This is fine for us, but I think it would be nice to have wezterm in the official terminal selection window to indicate that we have first class support for it? Maybe I am missing something, Omarchy is still new to me.
  3. [Off-topic] Once I have Omarchy installed, how can I build PRs like you did, how would I test my implementation without building an ISO every time. Could you share your workflow with me, please?

@dhh
Copy link
Member

dhh commented Feb 20, 2026

We can now do this without needing per-theme configs. If you'd like to explore this again with the new theming system, please do!

@dansdantas dansdantas force-pushed the add-wezterm branch 3 times, most recently from b99d0e9 to ef6eca2 Compare February 23, 2026 13:26
@iltumio
Copy link

iltumio commented Mar 1, 2026

sorry, didn't realize this pr was already in place.

I have a similar one that also include the support for wezterm for the screensaver launcher script here #4836.

Feel free to close mine in case this get merged

Copilot AI review requested due to automatic review settings March 1, 2026 20:01
Copy link
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.

Pull request overview

Adds WezTerm as an additional supported terminal by shipping a default WezTerm config and a theme-template intended to integrate with Omarchy’s theme switcher.

Changes:

  • Add a migration to copy the default wezterm.lua into ~/.config/wezterm/ if missing.
  • Add a themed template for generating ~/.config/omarchy/current/theme/wezterm.lua.
  • Update terminal restart script to trigger WezTerm config reload by touching wezterm.lua.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 4 comments.

File Description
migrations/1771799066.sh Installs default WezTerm config for existing users if not already present.
default/themed/wezterm.lua.tpl Introduces a theme template intended to produce WezTerm color configuration from colors.toml.
config/wezterm/wezterm.lua New default WezTerm config that loads theme output from ~/.config/omarchy/current/theme/wezterm.lua.
bin/omarchy-restart-terminal Adds WezTerm to terminal restart flow by touching its config for reload.

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

Copilot AI review requested due to automatic review settings March 2, 2026 03:06
Copy link
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.

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated 5 comments.


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

Copy link
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.

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.


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

use new template system

rename migration using standard command

follow pattern to restart terminal

Guard theme loading against emtpy HOME

Change default font

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Grammar corrections

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Adjust warning message

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

receive correct colors after requesting from theme
Copy link
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.

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated no new comments.


💡 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.

5 participants