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

Design light theme #434

Merged
merged 7 commits into from Oct 18, 2014
Merged

Design light theme #434

merged 7 commits into from Oct 18, 2014

Conversation

cormier
Copy link
Member

@cormier cormier commented Sep 1, 2014

Fixes #423

@torhve torhve force-pushed the design-light-theme branch 2 times, most recently from 9965bf1 to 1a3e676 Compare September 17, 2014 11:14
@cormier cormier force-pushed the design-light-theme branch 2 times, most recently from 9754be8 to 5642eb6 Compare September 27, 2014 21:46
@cormier cormier changed the title [WIP] Design light theme Design light theme Sep 27, 2014
@cormier
Copy link
Member Author

cormier commented Sep 27, 2014

@lorenzhs, @torhve, please merge if you find that this PR is appropriate.

@lorenzhs
Copy link
Member

@cormier can you rebase please?

@cormier cormier force-pushed the design-light-theme branch 2 times, most recently from 4e32609 to 6f96f91 Compare October 18, 2014 16:24
@cormier
Copy link
Member Author

cormier commented Oct 18, 2014

@lorenzhs : i've rebased.

@lorenzhs
Copy link
Member

is it good to merge?

@cormier
Copy link
Member Author

cormier commented Oct 18, 2014

It is !

@lorenzhs
Copy link
Member

Nice, I'll have a look later

<link href="css/glowingbear.css" rel="stylesheet" media="screen">
<link href="css/themes/dark.css" rel="stylesheet" media="screen">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this still correct with the parametrised input in line 40?

David Cormier and others added 7 commits October 18, 2014 14:42
* Remove style.css as those values should be themable as well.
* Update urls in index.html to link to dark theme by default
* make the background lighter
* set highlight colour to something readable
* improve readability of default grey colours
* make default text color readable
* fix some base colours
lorenzhs added a commit that referenced this pull request Oct 18, 2014
@lorenzhs lorenzhs merged commit 10dde0e into master Oct 18, 2014
@cormier cormier deleted the design-light-theme branch November 28, 2014 15:58
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.

Feature Request: Light mode
3 participants