Fix toggle load order so user overrides take precedence#6116
Fix toggle load order so user overrides take precedence#6116axelfrache wants to merge 2 commits into
Conversation
There was a problem hiding this comment.
Pull request overview
This PR fixes Hyprland toggle load ordering so user configuration reliably overrides toggle-provided settings, aligning behavior with the intended precedence: Omarchy defaults → toggles → user config.
Tip
If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.
Changes:
- Reorders
require("default.hypr.toggles")in the Hyprland Lua template to run before user modules. - Adds a migration to automatically reorder existing
~/.config/hypr/hyprland.luaand~/.config/hypr/hyprland.confinstalls to preserve user override precedence.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| migrations/1780929365.sh | Migrates existing Hyprland configs to the corrected toggle/user override load order. |
| config/hypr/hyprland.lua | Updates the default Hyprland Lua template to load toggles before user modules. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
It seems correct to me that toggles would override user settings. That's the entire point. Otherwise we can't guarantee that, say, no borders actually does what we promise. |
Summary
require("default.hypr.toggles")before userrequire()calls in thehyprland.luatemplatehyprland.luaandhyprland.confinstalls automaticallyToggles were loaded after user customizations, so any toggle would silently override settings in
~/.config/hypr/looknfeel.conf. The correct order is: Omarchy defaults → toggles → user config.Fixes #6068