[PR #1172] [MERGED] fix(borders): properly handle borders ZOrder #1236

Closed
opened 2026-01-05 14:54:33 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/LGUG2Z/komorebi/pull/1172
Author: @LGUG2Z
Created: 12/9/2024
Status: Merged
Merged: 12/9/2024
Merged by: @LGUG2Z

Base: masterHead: fix/borders_z_order


📝 Commits (1)

  • 2523922 fix(borders): properly handle borders ZOrder

📊 Changes

2 files changed (+59 additions, -49 deletions)

View changed files

📝 komorebi/src/border_manager/border.rs (+12 -0)
📝 komorebi/src/border_manager/mod.rs (+47 -49)

📄 Description

This commit makes it so a floating window only has the floating border when it is focused, if not it has the Unfocused border. It also makes the 'focused_container' have the Unfocused border when it is not the foreground window, for example when we have a floating window focused instead.

This commit also changes the border's window_kind so that the stored borders actually have that value so we can check it later (This value wasn't being updated).

This commit also makes it so we properly invalidate the borders in the situations discussed above (for example when changing focus to/from a floating window we need the floating window border to update its ZOrder as well as the previously focused window).

Lastly this commit, changes the WM_PAINT code part of the border so that it now sets the position of border so that the border's ZOrder updates to it's tracking window ZOrder.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/LGUG2Z/komorebi/pull/1172 **Author:** [@LGUG2Z](https://github.com/LGUG2Z) **Created:** 12/9/2024 **Status:** ✅ Merged **Merged:** 12/9/2024 **Merged by:** [@LGUG2Z](https://github.com/LGUG2Z) **Base:** `master` ← **Head:** `fix/borders_z_order` --- ### 📝 Commits (1) - [`2523922`](https://github.com/LGUG2Z/komorebi/commit/2523922eb4b0021e93ce2e41429cefdc88f5eedc) fix(borders): properly handle borders ZOrder ### 📊 Changes **2 files changed** (+59 additions, -49 deletions) <details> <summary>View changed files</summary> 📝 `komorebi/src/border_manager/border.rs` (+12 -0) 📝 `komorebi/src/border_manager/mod.rs` (+47 -49) </details> ### 📄 Description This commit makes it so a floating window only has the floating border when it is focused, if not it has the `Unfocused` border. It also makes the 'focused_container' have the `Unfocused` border when it is not the foreground window, for example when we have a floating window focused instead. This commit also changes the border's `window_kind` so that the stored borders actually have that value so we can check it later (This value wasn't being updated). This commit also makes it so we properly invalidate the borders in the situations discussed above (for example when changing focus to/from a floating window we need the floating window border to update its ZOrder as well as the previously focused window). Lastly this commit, changes the `WM_PAINT` code part of the border so that it now sets the position of border so that the border's ZOrder updates to it's tracking window ZOrder. <!-- Please follow the Conventional Commits specification. If you need to update your PR with changes from `master`, please run `git rebase master`. By opening this PR, you confirm that you have read and understood this project's `CONTRIBUTING.md`. --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2026-01-05 14:54:33 +01:00
adam closed this issue 2026-01-05 14:54:33 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/komorebi#1236