Skip to content

Bundle Zed editor with omazed for themes syncing#4873

Open
pomartel wants to merge 6 commits intobasecamp:devfrom
pomartel:omazed
Open

Bundle Zed editor with omazed for themes syncing#4873
pomartel wants to merge 6 commits intobasecamp:devfrom
pomartel:omazed

Conversation

@pomartel
Copy link
Copy Markdown
Contributor

@pomartel pomartel commented Mar 2, 2026

Omazed provides live theme switching for the Zed editor. It is maintained as an AUR package by @APS6. It uses Omarchy's themes hooks and color.toml. I have been using it for a while and it works very well.

This PR bundles omazed with Zed when installing it from the Editors menu.

Copilot AI review requested due to automatic review settings March 2, 2026 18:56
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.

@pomartel
Copy link
Copy Markdown
Contributor Author

pomartel commented Mar 5, 2026

I'm just not sure how we handle AUR packages in Omarchy. Maybe this should come with a prompt.

@dhh
Copy link
Copy Markdown
Member

dhh commented Mar 7, 2026

Given the new theme template system, I'd be willing to take a single zed tpl file instead of adding this dependency.

@pomartel
Copy link
Copy Markdown
Contributor Author

pomartel commented Mar 7, 2026

@APS6 Do you think that's something that could be done? I would very like to have this merged in Omarchy but it's your project and I don't want to intercede.

@pomartel pomartel changed the title Bundle omazed with Zed Editor installation for themes syncing Move Zed theme switching into Omarchy Mar 9, 2026
@pomartel
Copy link
Copy Markdown
Contributor Author

pomartel commented Mar 9, 2026

@dhh I extracted the Omazed templating rules and used the same logic as we have in place for VS Code. I tested it thoroughly and I think it's as minimal as it can be.

@dhh
Copy link
Copy Markdown
Member

dhh commented Mar 9, 2026

Oh wow. I'm surprised that it's such a huge chunk. I would have thought we could have gotten away with something on the scale of Alacritty or Ghostty. Hmm. Don't see us taking on ~700 lines of code for an optional editor, then.

@APS6
Copy link
Copy Markdown

APS6 commented Mar 9, 2026

@pomartel Sorry for not replying earlier. I forgot to respond in the midst of my examinations.

@dhh Yeah, editor themes are quite complex. I explained my reasoning earlier in detail here: APS6/omazed#4 (comment)

@dhh
Copy link
Copy Markdown
Member

dhh commented Mar 9, 2026

Okay, given all this, I'm happy to just installed omazed when we install zed via the menu 👍

Copilot AI review requested due to automatic review settings March 9, 2026 17:29
@pomartel
Copy link
Copy Markdown
Contributor Author

pomartel commented Mar 9, 2026

@dhh That totally makes sense. Let me revert to my initial commit then.

@APS6 Thanks for your input and thanks again for your wonderful plugin. Trying to integrate it into Omarchy made me realize the complexity of it all! Can you validate that I installed the package the right way?

@pomartel pomartel changed the title Move Zed theme switching into Omarchy Bundle Zed editor with omazed for themes syncing Mar 9, 2026
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.

@APS6
Copy link
Copy Markdown

APS6 commented Mar 9, 2026

LGTM 👍

@APS6
Copy link
Copy Markdown

APS6 commented Mar 26, 2026

@dhh Hey, it's been a while so I figured I should ask if we are waiting for something specific?

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.

4 participants