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

Transparent title bar #2

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

remysucre
Copy link

Make the title bar transparent for a cleaner look. Tested on MacOS.

Make the title bar transparent for a cleaner look. Tested on MacOS.
@rougier
Copy link
Owner

rougier commented May 18, 2020

Thanks. Does that work if you're not on mac ? (I'm on mac and I completely removed the titlebar)

@remysucre
Copy link
Author

It should, those parameters are not specific to Mac. I can't test now because I don't have a linux desktop atm.

@justinbarclay
Copy link

As far as I know the ns-* namespace is specific to MacOS. (Where ns here stands for NeXTSTEP)

@sivark
Copy link

sivark commented Jun 2, 2020

FWIW, I use the following to avoid any window decoration including borders and titlebar (tested on linux):

(set-frame-parameter nil 'undecorated t)

rougier pushed a commit that referenced this pull request Nov 6, 2020
Make org table entries easier to see.
@DivineDominion
Copy link
Contributor

DivineDominion commented Nov 6, 2021

macOS NSWindow title bar colors depend on system settings -- or, more precise, the app's appearance, which is derived from the system unless it opts out.

Dark theme with light system appearance:

Screen Shot 2021-11-07 at 00 56 35

Dark theme with dark system appearance:

Screen Shot 2021-11-07 at 00 57 01

So there are caveats when this is enabled by default.

I'd suggest adding this only if the themes auto-set for ns-system-appearance-change-functions (that are hooks that are called when switching from light to dark and back again, and during the initial appearance setting when the app launches).

Otherwise this could look weird if users don't understand what's going on.

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

Successfully merging this pull request may close these issues.

5 participants