komorebi icon indicating copy to clipboard operation
komorebi copied to clipboard

[BUG]: shadow borders

Open Kibouo opened this issue 3 years ago • 2 comments

Some applications have shadow borders that cause komorebi colored borders to be misplaced. These shadow borders are also included in the window size when placing the windows. The clearest way of illustrating this is with a screenshot: image

The windows shown here are:

  • left: vscode
  • top-right: firefox
  • bottom-right: vscode

The red border snugly fits on the vscode windows, while the red border on firefox is offset (offset can be seen in screenshot). Also firefox's tile is smaller due to this.

Other applications that behave similar to firefox:

  • windows terminal
  • [placeholder, haven't tested much more yet]

Other applications that behave similar to vscode (correctly):

  • discord
  • [placeholder, haven't tested much more yet]

This is not related to Windows' universal window shadow settings (already tried turning that off). identify-border-overflow-application also didn't seem to help (maybe I'm using it wrong? I did komorebic identify-border-overflow-application exe "firefox.exe").

My border settings are as follows:

komorebic active-window-border enable
komorebic active-window-border-colour 255 0 0 --window-kind single
komorebic active-window-border-width 2
komorebic active-window-border-offset 0
komorebic invisible-borders 0 0 0 0

Host info:

Edition	Windows 11 Pro N
Version	22H2
Installed on	‎7/‎05/‎2022
OS build	22621.1413
Experience	Windows Feature Experience Pack 1000.22639.1000.0

Kibouo avatar Mar 19 '23 14:03 Kibouo

identify-border-overflow-application is generally (but not always!) used with Electron apps, because Electron apps generally try to draw over the invisible borders reserved by the operating system.

The most common applications have generally been handled in this provided file.

In the case of Firefox and Windows Terminal, I don't think you should be using identify-border-overflow-application, which is probably what is resulting in the behaviour you have shown.

LGUG2Z avatar Apr 16 '23 04:04 LGUG2Z

Thank you for explaining what the option does.

However, the behavior is identical when not using the option.


From: جاد @.> Sent: Sunday, April 16, 2023, 06:56 To: LGUG2Z/komorebi @.> Cc: Kibouo @.>; Author @.> Subject: Re: [LGUG2Z/komorebi] [BUG]: shadow borders (Issue #378)

identify-border-overflow-application is generally (but not always!) used with Electron apps, because Electron apps generally try to draw over the invisible borders reserved by the operating system.

The most common applications have generally been handled in this provided filehttps://github.com/LGUG2Z/komorebi/blob/master/komorebi.generated.ps1.

In the case of Firefox and Windows Terminal, I don't think you should be using identify-border-overflow-application, which is probably what is resulting in the behaviour you have shown.

— Reply to this email directly, view it on GitHubhttps://github.com/LGUG2Z/komorebi/issues/378#issuecomment-1510058063, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGESTOKKVT5EBIU2QLKYCRLXBN3YNANCNFSM6AAAAAAWAEMFCM. You are receiving this because you authored the thread.Message ID: @.***>

Kibouo avatar Apr 16 '23 10:04 Kibouo

I believe this is fixed as of https://github.com/LGUG2Z/komorebi/commit/6a1ed3bcaa8d1f6e75535778fc5bd51d9cc3c019, if not before

LGUG2Z avatar May 12 '24 18:05 LGUG2Z