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

Add simpleblack theme #398

Merged
merged 5 commits into from Oct 31, 2019
Merged

Add simpleblack theme #398

merged 5 commits into from Oct 31, 2019

Conversation

lucasprag
Copy link
Contributor

@lucasprag lucasprag commented Oct 29, 2019

Hi, great work on this plugin =)

I had this theme sitting in my own configuration for awhile and I was wondering if it would be nice to add to lightline's available themes.

This is how it looks like:
simpleblack

It was based on my colorscheme called simpleblack

I'm open to any feedback.

Thanks :)

@lucasprag
Copy link
Contributor Author

@lucasprag lucasprag commented Oct 29, 2019

I can't assign you as a reviewer, so I'm pinging you @itchyny =)

let s:blue2 = [ '#91AADF', '110' ]

let s:green = [ '#57BA37', '71' ]
let s:gold = [ '#F0D50C', '220' ]
Copy link
Owner

@itchyny itchyny Oct 30, 2019

Choose a reason for hiding this comment

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

Can you make the hex consistently lowercase?

Copy link
Contributor Author

@lucasprag lucasprag Oct 30, 2019

Choose a reason for hiding this comment

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

Sure, thanks for reviewing @itchyny =)

@lucasprag lucasprag requested a review from itchyny Oct 30, 2019
@itchyny
Copy link
Owner

@itchyny itchyny commented Oct 31, 2019

Thank you.

@itchyny itchyny merged commit 53bd327 into itchyny:master Oct 31, 2019
1 check passed
@lucasprag
Copy link
Contributor Author

@lucasprag lucasprag commented Oct 31, 2019

Cool, thanks =)

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.

None yet

2 participants