Skip to content

feat: Added modern-lilac and modern-lilac2 themes #343

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

Merged
merged 7 commits into from
Oct 24, 2022

Conversation

FleetAdmiralJakob
Copy link
Contributor

@FleetAdmiralJakob FleetAdmiralJakob commented Oct 24, 2022

Description

I added the best theme ever!!!!!
Actually it's my personal theme, but I use it all the time and I think it belongs here too.

Type of change

  • New feature (added a non-breaking change which adds functionality)

How Has This Been Tested?

With the attribute thing in the link

Checklist:

  • I have checked to make sure no other pull requests are open for this issue
  • The code is properly formatted and is consistent with the existing code style

Copy link
Owner

@DenverCoder1 DenverCoder1 left a comment

Choose a reason for hiding this comment

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

Looks great!

It would also be nice if you could update the docs/themes.md as well on your branch.

image

https://user-images.githubusercontent.com/20955511/197569406-6ff144c3-1d6e-4500-9f0b-3112a6c62584.png

Here is an image URL you can use.

@FleetAdmiralJakob
Copy link
Contributor Author

FleetAdmiralJakob commented Oct 24, 2022

Hi, I thought it would be nice if there is a second theme where the text is white, so I made it under modern-lilac2. If you send me again such an image, I will add this theme also to the doc so that you can merge this PR.

@DenverCoder1
Copy link
Owner

The image is from the demo site

image

https://user-images.githubusercontent.com/20955511/197575977-029fc730-9c7e-4556-be7c-a727a1715fa7.png

@FleetAdmiralJakob
Copy link
Contributor Author

Did it, you can merge!!!!

Copy link
Owner

@DenverCoder1 DenverCoder1 left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks for the contribution 🎉

@DenverCoder1 DenverCoder1 changed the title New nice theme!!!! feat: Added modern-lilac and modern-lilac2 themes Oct 24, 2022
@DenverCoder1 DenverCoder1 merged commit 4acf33d into DenverCoder1:main Oct 24, 2022
@DenverCoder1 DenverCoder1 added enhancement New feature or request new theme adding a new theme labels Oct 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request new theme adding a new theme
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants