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

Theme Request: Fairyfloss #287

Closed
JuneKelly opened this issue Jun 12, 2019 · 18 comments
Closed

Theme Request: Fairyfloss #287

JuneKelly opened this issue Jun 12, 2019 · 18 comments
Labels
is:theme Requests that a theme be added or removed

Comments

@JuneKelly
Copy link
Contributor

Main github repo: http://sailorhg.github.io/fairyfloss

Example screenshot:

image

There is a partial conversion for emacs here: sailorhg/fairyfloss#3, but it's got some weird colors going on in places like the magit UI.

@hlissner hlissner added the is:theme Requests that a theme be added or removed label Jun 28, 2019
@ema2159
Copy link
Contributor

ema2159 commented Jul 15, 2019

If Mr. Lissner lets me, I'll implement this one

@ema2159
Copy link
Contributor

ema2159 commented Jul 25, 2019

I have green lights now so expect to see this soon 😁

@ema2159
Copy link
Contributor

ema2159 commented Jul 28, 2019

Almost there!
fairy

@ema2159
Copy link
Contributor

ema2159 commented Jul 28, 2019

Done, it's been added. I almost got diabetes implementing this jaja! Please test it and tell me if you find any weird faces. I used it and didn't found any issues besides some magit faces which I fixed

@ema2159 ema2159 closed this as completed Jul 28, 2019
@JuneKelly
Copy link
Contributor Author

JuneKelly commented Jul 31, 2019

Very nice, thanks!

I'm testing on spacemacs, and it all looks very good so far. One small point, maybe the current-line highlight is a little too strong? Maybe that's just a personal preference.

image

Otherwise, this looks real nice, 🎉

@ema2159
Copy link
Contributor

ema2159 commented Jul 31, 2019

That's using hl-line? If it is, then yes, it's too strong. I'll check in VS Code and use the same color

@ema2159
Copy link
Contributor

ema2159 commented Jul 31, 2019

@ShaneKilkelly done, It's now lighter and smoother. Report here if you find more things like that!

@JuneKelly
Copy link
Contributor Author

Thanks!

@chase-dwelle
Copy link

Surprised that I like the theme as much as I do. One question though, I have this blue color ("blue" -- #268bd2 in the theme) that I have not seen in any of the screenshots that does not have enough contrast with the background (can be seen with the infile = open(rfile, 'rb') line in the attached

image

Personally I think something like #85f3bc, a pastel green, is easier to differentiate from the background:

image

Is this something that can be changed, or is the original "blue" color preferred by most?

@ema2159
Copy link
Contributor

ema2159 commented Aug 14, 2019

You're right. I don't quite remember where I took that blue from. Although a pastel green would do the work, I personally think it would be better to find a blue alternative, given that it's the blue variable what we're changing.

@ema2159
Copy link
Contributor

ema2159 commented Aug 14, 2019

@chase-dwelle how about this color I just made up (lol)?:
#55b3cc
It's still blueish while having a much better contrast .

@chase-dwelle
Copy link

Yes that works; here it is in the same screenshot.

image

@ema2159
Copy link
Contributor

ema2159 commented Aug 14, 2019

Done, it should be up in the next update

@chase-dwelle
Copy link

Found some other issues with font faces in org-mode, especially with scheduled faces. The inherited faces from the base theme end up with nearly the same color in scheduled faces and the background. I suggest borrowing from doom-dracula:

   (org-scheduled :foreground green)
   (org-scheduled-previously :foreground yellow)
   (org-scheduled-today :foreground orange) 

@ema2159
Copy link
Contributor

ema2159 commented Sep 21, 2019

Hi @chase-dwelle!, Would you please show me an screenshot?

@chase-dwelle
Copy link

chase-dwelle commented Sep 21, 2019

Here is my screenshot using the base fairy floss theme:

image

Borrowing the following definitions from doom-dracula gives the image below. I'm guessing that there are some font face decisions in the base theme that can lead to lack of readability in some of the darker themes, which is why doom-dracula has so defined faces.

   (org-scheduled :foreground green)
   (org-scheduled-previously :foreground yellow)
   (org-scheduled-today :foreground orange)

image

@ema2159
Copy link
Contributor

ema2159 commented Oct 3, 2019

@chase-dwelle sorry for the late reply. How about now?

@chase-dwelle
Copy link

Yes works well. Thanks you!

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
is:theme Requests that a theme be added or removed
Projects
None yet
Development

No branches or pull requests

4 participants