Skip to content

Enable audio settings to use default terminal emulator#325

Closed
apdingman wants to merge 1 commit into
basecamp:devfrom
apdingman:wiremix-dynamic-terminal
Closed

Enable audio settings to use default terminal emulator#325
apdingman wants to merge 1 commit into
basecamp:devfrom
apdingman:wiremix-dynamic-terminal

Conversation

@apdingman
Copy link
Copy Markdown

Currently, we're defaulting the audio settings to open using Alacritty. However, if the someone has chosen to go ahead and pivot to a different terminal emulator (i.e. ghostty), then this is the experience upon opening the audio settings from walker.

image image

This change does remove the use of the --class and --title options, but removing those seems inconsequential. My experience was the same with both of these configurations:

Exec=ghostty --class=Wiremix --title=Wiremix -e wiremix
Terminal=false
Exec=wiremix
Terminal=true

Obviously those options serve a purpose for setting the window class and title, but I don't see the immediate need here.

Certainly open for discussion. Just looking to help out those that have pivoted from Alacritty already so that they can access their audio settings without issue.

@abenz1267
Copy link
Copy Markdown
Contributor

the problem is that you can't target the wiremix window via windowrules like that, in omarchy's case: to make it float by default.

@apdingman
Copy link
Copy Markdown
Author

the problem is that you can't target the wiremix window via windowrules like that, in omarchy's case: to make it float by default.

@abenz1267 You're absolutely right. I missed the windowrules portion of the implementation. I don't see a great alternative for this beyond hardcoding the terminal emulator and passing the title and class options.

@dhh
Copy link
Copy Markdown
Member

dhh commented Jul 24, 2025

I think we need something like a omarchy-setup-terminal script that'll do all this hardcoded replacement work. If you want to look into that, please do!

@dhh dhh closed this Jul 24, 2025
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.

3 participants