Skip to content
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

Closed
Heath123 opened this issue May 22, 2020 · 14 comments · Fixed by #912
Closed

option to use Windows title bar on Linux #285

Heath123 opened this issue May 22, 2020 · 14 comments · Fixed by #912

Comments

@Heath123
Copy link

#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.

@shiftkey
Copy link
Owner

@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.

@shiftkey shiftkey changed the title Feature request: option to use Windows title bar on Linux option to use Windows title bar on Linux May 22, 2020
@Heath123
Copy link
Author

Heath123 commented May 23, 2020

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?

@harry-cpp
Copy link

I personally prefer the CSD, I use Ubuntu 20.04 + GNOME.

@GrahamSH-LLK
Copy link

This would be so awesome. Leaving it up to the OS is not my favorite choice as some Linux dists are absolutely ugly.

@GrahamSH-LLK
Copy link

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.

@mon-jai
Copy link

mon-jai commented Apr 11, 2022

Wanted this feature too. The current one is ugly on Ubuntu.

@Miqueas
Copy link

Miqueas commented Apr 17, 2022

Wanted this feature too. The current one is ugly on Ubuntu.

Is ugly on ANY distro/desktop environment

@Miqueas
Copy link

Miqueas commented Apr 17, 2022

@shiftkey the best solution here is to provide an option to enable/disable CSD and ship the app as-is (like vs code)

@Miqueas
Copy link

Miqueas commented Apr 17, 2022

From my point of view, this is a waste of space, since everything in the server-side decorated title bar can be in the same place as the app menu bar

image

@Miqueas
Copy link

Miqueas commented May 8, 2022

Still no changes? :(

@shiftkey
Copy link
Owner

shiftkey commented May 9, 2022

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.

@Miqueas
Copy link

Miqueas commented May 9, 2022

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

@mon-jai
Copy link

mon-jai commented Jul 26, 2023

@shiftkey Maybe we could just additionally, make another build with Windows titlebar?

@mon-jai
Copy link

mon-jai commented Jul 26, 2023

@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,

} else if (__LINUX__) {
windowOptions.frame = false
windowOptions.icon = path.join(__dirname, 'static', 'icon-logo.png')
}
this.window = new BrowserWindow(windowOptions)

Maybe we could write the config value to a file? (like `${app.getPath('userData')}/electron-config.json`)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants
@shiftkey @harry-cpp @Heath123 @Miqueas @GrahamSH-LLK @mon-jai and others