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

Added Solaized images #103

Merged
merged 1 commit into from
Jan 4, 2019
Merged

Conversation

craigem
Copy link
Contributor

@craigem craigem commented Jan 4, 2019

for both Dark and Light as suggested in this comment.

for both Dark and Light
@cdepillabout cdepillabout merged commit 0cd741d into cdepillabout:master Jan 4, 2019
@cdepillabout
Copy link
Owner

@craigem

If you also want to add a predefined solarized color scheme, please feel free to go ahead and send a PR.

I imagine you'd add it right below this line:

deriving (Eq, Show, Functor, Foldable)

You could basically just copy this code into the Termonad.Config.Colour module:

, palette = ExtendedPalette solarizedDark1 solarizedDark2
}
where
solarizedDark1 :: Vec N8 (Colour Double)
solarizedDark1 =
sRGB24 0 43 54 -- base03, background
:* sRGB24 220 50 47 -- red
:* sRGB24 133 153 0 -- green
:* sRGB24 181 137 0 -- yellow
:* sRGB24 38 139 210 -- blue
:* sRGB24 211 54 130 -- magenta
:* sRGB24 42 161 152 -- cyan
:* sRGB24 238 232 213 -- base2
:* EmptyVec
solarizedDark2 :: Vec N8 (Colour Double)
solarizedDark2 =
sRGB24 7 54 66 -- base02, background highlights
:* sRGB24 203 75 22 -- orange
:* sRGB24 88 110 117 -- base01, comments / secondary text
:* sRGB24 131 148 150 -- base0, body text / default code / primary content
:* sRGB24 147 161 161 -- base1, optional emphasised content
:* sRGB24 108 113 196 -- violet
:* sRGB24 101 123 131 -- base00
:* sRGB24 253 246 227 -- base3
:* EmptyVec

Just add a definition like solarizedDarkPalette and solarizedLightPalette. Then you can link to these two images you added in this PR in the haddocks. If you have trouble figuring it out you could send a WIP PR with what you have and we could go from there.

This would close #94.

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.

2 participants