-
Notifications
You must be signed in to change notification settings - Fork 529
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
option to use Windows title bar on Linux #285
Comments
@Heath123 thanks for opening this up. For the moment this is a low-priority for me as I initially thought this would be straight-forward but #149 showed how many people tweaked their setups (and I'd inadvertently affected). I also heard in #243 that these client-side decorations didn't fit with their setup. I'd like to hear from others why the Windows-style theme here is preferred for them - others shared their setups in #149, like #149 (comment), so I'm curious if we can avoid having to juggle multiple options. |
Personally I like it because I think custom title bars look quite nice and professional, it saves a tiny bit of space (not really a big deal) and it's sort of funny to see a Windows-like window on Ubuntu, I don't know about other people, though. This is a really small thing and I don't care that much, so it makes sense that it's low-priority. I wonder if many other people want this or if it's mostly just me? |
I personally prefer the CSD, I use Ubuntu 20.04 + GNOME. |
This would be so awesome. Leaving it up to the OS is not my favorite choice as some Linux dists are absolutely ugly. |
One idea is to not put the close/minimize/maximize there and leave that up to the OS. That way we minimize annoyance on both sides. A less "windowsy" one would be better also. |
Wanted this feature too. The current one is ugly on Ubuntu. |
Is ugly on ANY distro/desktop environment |
@shiftkey the best solution here is to provide an option to enable/disable CSD and ship the app as-is (like vs code) |
Still no changes? :( |
The relevant code is linked in the other pull requests, but I've marked this as "help wanted" as I'd like someone to take a shot at making it configurable. |
Oh, that makes sense |
@shiftkey Maybe we could just additionally, make another build with Windows titlebar? |
@shiftkey Did a quick look. Seems that their configuration stores lies within the browser side of the boundary. But if we will need the config value before the browser window is even created, desktop/app/src/main-process/app-window.ts Lines 62 to 67 in 5d1d4b7
Maybe we could write the config value to a file? (like |
#139 added this, but was reverted (#149).
Proposed solution
Personally I (and probably some others) prefer the Windows-style title bar, so it would nice if there was a configuration option of some kind.
The text was updated successfully, but these errors were encountered: