vercel / hyper

A terminal built on web technologies
https://hyper.is
MIT License
43.38k stars 3.52k forks source link

White line / top border after Mojave upgrade? #3259

Open jesseleite opened 6 years ago

jesseleite commented 6 years ago

Issue

After upgrading to Mojave, I notice electron apps have a harsh white border at the top, which is especially noticeable on dark backgrounds. It happens in both dark and light mode. In this case, I have border: 0 !important; styling on my hyper window, so that you can see how apparent it is without the default hyper border...

screen shot 2018-10-17 at 10 58 56 am

I've also heard people say that all (even non-electron) windows have it, but native windows look way more natural than electron windows...

screen shot on 2018-10-17 at 09_15_29

I realize this might be actually on electron to fix (see https://github.com/electron/electron/issues/13164), but in that thread apparently the VS Code Insiders build has a 'fix', so I thought I'd share here in case it might be something that could be remedied on Hyper's end?

lclhst commented 6 years ago

This comes with Mojave, but you can disable this ugly line:

defaults write -app [application name] NSRequiresAquaSystemAppearance -bool No

jesseleite commented 6 years ago

Ran defaults write -app Hyper NSRequiresAquaSystemAppearance -bool No and restarted. No go, still see the line?

That said, end users of electron apps shouldn't have to change OS level settings to view electron windows as they were intended to be seen, wouldn't you agree?

karlhorky commented 6 years ago

@lclhst this was mentioned in the electron issue thread, and does not fix the issue completely: https://github.com/electron/electron/issues/13164#issuecomment-424309847

MariuzM commented 5 years ago

I think on one of the Mojave vids one Dev has talked about the gray white border and from my understanding thats a feature, a stupid feature if you ask me, that Dev and who approved need to be fired.