search

atom / one-light-ui

Atom One light UI theme
MIT License
73 stars 62 forks source link

Missing border on OS X #25

Closed rnarian closed 9 years ago

rnarian commented 9 years ago

Is there any way to fix the missing border on OS X without messing with the presentation on Windows?

missing-border-1

I temporarily fixed it with the following css in my styles.less file:

.theme-one-light-ui {
  border-top: 1px solid rgba(0,0,0,.2);
}

missing-border-2

simurai commented 9 years ago

Yeah, it does look better with a border. I think there is a platform-* class that gets added to the body and could be used.

rnarian commented 9 years ago

Awesome! This is what I use in my styles.less now:

.platform-darwin atom-workspace.theme-one-light-ui {
  border-top: 1px solid rgba(0,0,0,.2);
}

.platform-darwin.is-blurred atom-workspace.theme-one-light-ui {
  border-top: 1px solid rgba(0,0,0,.1);
}

I would've sent a pull request, but I don't really know where to put it.

simurai commented 9 years ago

Ok, added to styles/atom.less. Thanks for your suggestion.