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

Add background color to main-content #110

Merged
merged 2 commits into from
Apr 8, 2016
Merged

Add background color to main-content #110

merged 2 commits into from
Apr 8, 2016

Conversation

jlord
Copy link
Contributor

@jlord jlord commented Apr 6, 2016

Tiny PR but wanted to make sure it was ok/got seen.

I tried the app on Windows today and noticed the background was dark and it seemed to me that it was because we don't explicitly give that part of the app a background color, so this PR adds it.

Funny, when you open dev tools the background goes white...

cc @simurai

dark-screen
dev-tools

@simurai
Copy link
Contributor

simurai commented Apr 7, 2016

Hah.. yes! 👍

Should we add a short comment? So we don't remove it by accident again? It could also be moved to html {} so that the whole background is white.

@jlord
Copy link
Contributor Author

jlord commented Apr 8, 2016

Yeah that sounds good 👍

@jlord jlord merged commit d4628fb into master Apr 8, 2016
@jlord jlord deleted the main-bg-color branch April 8, 2016 01:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants