-
Notifications
You must be signed in to change notification settings - Fork 30.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[GNOME Linux] No window decoration (shadow) if a window title bar style is custom #225073
Comments
window.titleBarStyle
is custom
window.titleBarStyle
is custom
window.titleBarStyle
is custom
window.titleBarStyle
is custom
window.titleBarStyle
is custom
Tested on |
I can confirm the missing window decorations (i.e. shadows) when running on GNOME/Wayland when using the custom title bar style (the default). It would be really nice if this worked, because
Regarding the comment on "Visual Studio Code - URL Handler", I still haven't figured out why there are two desktop files, but you can at least fix the issue with two task bar entries (and/or missing icons) by setting |
@cbrnr |
I think it's one of these upstream bugs and nothing VSCode could (or should) do itself about it: |
@jhasse |
However, Google Chrome does have borders and a shadow even though it is the basis of Electron (and it also has a custom title bar). So it might be worth looking into how they achieve this. |
@cbrnr |
The shadow with native titlebar are from the CSD implementation in Electron. For the custom titlebar Electron shares the implementation with Chrome and based on https://issues.chromium.org/issues/40158897 there is no work actively being tracked in upstream. Closing as not actionable on our end. |
@deepak1556 but wait, as I've mentioned previously, Chromium does have shadows when using a custom title at. Why not VS Code then? |
@cbrnr please refer to https://issues.chromium.org/issues/40158897#comment4 |
I don't understand. This issue is 3 years old and marked as obsolete, probably because it already works? |
@cbrnr
We can't say for sure until you learn the upstream source code. |
I'm pretty sure its in upstream, since I'm using Google Chrome and they don't add any patches AFAICS: https://aur.archlinux.org/packages/google-chrome |
Yes, you are right, they use packages compiled directly by the Google team. |
@deepak1556 can you reopen this issue please until it is clear if there's anything that could/should be done? |
The custom titlebar implementation in Electron https://github.com/electron/electron/blob/main/shell/browser/ui/views/opaque_frame_view.cc is pulled from https://source.chromium.org/chromium/chromium/src/+/main:chrome/browser/ui/views/frame/opaque_browser_frame_view.h which is the custom titlebar implementation for Chrome. The closest CL I could find related to this was https://chromium-review.googlesource.com/c/chromium/src/+/2901661 and it doesn't enable in OpaqueBrowserView. I will reopen to confirm the behavior with dPWA on linux |
Thanks @deepak1556, let me know if there is anything I can do to help! |
Observing this problem in KDE plasma 6.2.4 with X11. VSCode version 1.96.2 |
Does this issue occur when all extensions are disabled?: Yes
Steps to Reproduce:
window.titleBarStyle
options tocustom
in the settingsExpected behaviour
There should be a window decoration (shadow) as it is in XWayland compatibility mode:

Actual behaviour
There is no window decoration/shadow:

But If I set

window.titleBarStyle
tonative
, the window decoration/shadow appears:The text was updated successfully, but these errors were encountered: