Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

Black/dark design? #296

Closed
rugk opened this issue Aug 27, 2015 · 13 comments
Closed

Black/dark design? #296

rugk opened this issue Aug 27, 2015 · 13 comments
Assignees

Comments

@rugk
Copy link

rugk commented Aug 27, 2015

I'd like to use a black design for the markdown preview. (otherwise this is just a too strong contrast)

I've tried it with this simple CSS tweak, but it does not really looks such nice and it seems I have to adjust much more:

.markdown-preview {
  color: #ffffff;
  background-color: #000000;
}

E.g. tables are still white and I don't really need such a dark black.

In #293 (comment) I've seen @hemraker uses one.
I think at least you, @hemraker, have a nicer solution for this.

@hemraker
Copy link

I use One Dark Ui and syntax theme. Simple as that :)

untitled-2

@rugk
Copy link
Author

rugk commented Aug 27, 2015

Strange. I've also used One Dark syntax but another UI theme.
So after changing both to One Dark it worked (obviously), so I've tried and changed the UI theme back (leaving one dark as syntax like it always was) and it stayed dark... 😕

So I've changed the (build-in) syntax themes again and even the 'light' themes always displayed a dark background (although the quote part was light now).
So maybe the style was not applied as I (as far as I can remember) didn't changed it after installing this package. (It was really just black text on a white background)
Could this be the issue? So the style just was not applied...

@rugk rugk changed the title Black design? Black/dark design? Aug 27, 2015
@hemraker
Copy link

That sounds very likely. I don't think changing the theme overwrites the css you've written in styles.less but to be honest I haven't played too much around with it as I've been really satisfied with One Dark.

@rugk
Copy link
Author

rugk commented Aug 27, 2015

Ehm... you're right, yes.
Just noticed that I still had the overwrites in my stylesheet... Dump... 🙉

Now it looks better, but it seems that the UI theme is important. Only with One dark (UI) it was dark. With Atom dark it wasn't, no matter what syntax theme I choose.

@rugk
Copy link
Author

rugk commented Aug 27, 2015

And this really does not look nice:
markdownpreviewdesigntest
(code: https://gist.github.com/rugk/099e2b793d907c9ee46e)

@hemraker
Copy link

I couldn't agree more 😄
Seems that UI influences menus, toolbars and markdown-preview Syntax is just the editor.

Glad you got it fixed!!

@rugk
Copy link
Author

rugk commented Aug 27, 2015

Well... fixed. I still do not want to use the One dark.

But I've now opened an issue at the UI repo: atom/atom-dark-ui#50

@rugk rugk closed this as completed Aug 27, 2015
@simurai
Copy link
Contributor

simurai commented Aug 29, 2015

I think it's worth trying to make this package adapt to the syntax colors. Then not every theme has to do the same.

@simurai simurai reopened this Aug 29, 2015
@simurai simurai self-assigned this Aug 29, 2015
@rugk
Copy link
Author

rugk commented Aug 29, 2015

Okay, so back here.
Just make sure to not override this option.

@simurai
Copy link
Contributor

simurai commented Sep 1, 2015

Just make sure to not override this option.

Yes, that should stay.

WIP. One dark:

screen shot 2015-09-01 at 11 24 48 pm

Atom dark:

screen shot 2015-09-01 at 11 29 07 pm

@rugk
Copy link
Author

rugk commented Sep 19, 2015

The screenshots already look very promising. Any news here?

@simurai
Copy link
Contributor

simurai commented Sep 23, 2015

@rugk Ok, this should be in the next release.

@rugk
Copy link
Author

rugk commented Sep 26, 2015

Nice... 👍

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

No branches or pull requests

3 participants