Skip to content

Restore omarchy-launch-tui for btop and lazydocker keybindings#3

Closed
Copilot wants to merge 2 commits intomainfrom
copilot/revert-bindings-conf-reversion
Closed

Restore omarchy-launch-tui for btop and lazydocker keybindings#3
Copilot wants to merge 2 commits intomainfrom
copilot/revert-bindings-conf-reversion

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 5, 2026

Reverts commit aa25737 which removed omarchy-launch-tui wrapper from btop and lazydocker launch bindings.

Changes

  • SUPER+SHIFT+T (btop): $terminal -e btopomarchy-launch-tui btop
  • SUPER+SHIFT+D (lazydocker): $terminal -e lazydockeromarchy-launch-tui lazydocker

This aligns these bindings with other TUI application launchers in the file (e.g., Neovim uses omarchy-launch-editor).

Original prompt

Objective

Revert commit aa25737 which itself was a revert of the bindings.conf changes from omarchy.

Details

Commit aa25737a7170eaa54caa511fc7d2379c4c6dc1f7 reverted the original commit 6202669c15aeda24c4aa2f945206954b06ce8938 that brought bindings.conf changes from omarchy.

By reverting this revert commit, we will restore the omarchy-launch-tui functionality.

Required Changes

In the file hypr/.config/hypr/bindings.conf, make the following changes:

Line 12: Change from:

bindd = SUPER SHIFT, T, Activity, exec, $terminal -e btop

To:

bindd = SUPER SHIFT, T, Activity, exec, omarchy-launch-tui btop

Line 13: Change from:

bindd = SUPER SHIFT, D, Docker, exec, $terminal -e lazydocker

To:

bindd = SUPER SHIFT, D, Docker, exec, omarchy-launch-tui lazydocker

This will restore the use of omarchy-launch-tui for launching btop and lazydocker instead of using $terminal -e.

This pull request was created from Copilot chat.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: eapolinario <653394+eapolinario@users.noreply.github.com>
Copilot AI changed the title [WIP] Revert commit aa25737 to restore omarchy-launch-tui functionality Restore omarchy-launch-tui for btop and lazydocker keybindings Jan 5, 2026
Copilot AI requested a review from eapolinario January 5, 2026 22:00
@eapolinario eapolinario marked this pull request as ready for review January 5, 2026 22:03
Copilot AI review requested due to automatic review settings January 5, 2026 22:03
@eapolinario
Copy link
Copy Markdown
Owner

@copilot , rebase?

Copy link
Copy Markdown

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

This PR restores the use of the omarchy-launch-tui wrapper for launching btop and lazydocker TUI applications by reverting a previous revert commit. The changes align these keybindings with the established pattern of using specialized omarchy-launch-* wrappers for different application types in the configuration.

Key changes:

  • Replace direct terminal invocations ($terminal -e) with the omarchy-launch-tui wrapper for btop and lazydocker
  • Establish consistency with other application launchers that use similar wrapper patterns (e.g., omarchy-launch-editor for Neovim)

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

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Jan 5, 2026

@copilot , rebase?

The changes from this PR were already merged to main via PR #4 (commit 82f274c). The same modifications to lines 12-13 in bindings.conf are already present on the main branch. This PR can be closed as the work is complete.

@eapolinario
Copy link
Copy Markdown
Owner

@copilot , close this PR and leave a comment explaining why.

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Jan 5, 2026

@copilot , close this PR and leave a comment explaining why.

I don't have the ability to close PRs directly. However, this PR should be closed because the changes were already merged to main via PR #4 (commit 82f274c "Reapply 'bring bindings.conf changes from omarchy'"). Both PRs made identical modifications to hypr/.config/hypr/bindings.conf lines 12-13, replacing $terminal -e btop and $terminal -e lazydocker with omarchy-launch-tui btop and omarchy-launch-tui lazydocker respectively. The work is complete and this PR is redundant.

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