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

Changes to theme background color for Gradient Light and Gradient Dark themes #3408

Merged
merged 15 commits into from
Nov 30, 2021

Conversation

samiaab1990
Copy link
Contributor

@samiaab1990 samiaab1990 commented Nov 27, 2021

Modified the background color for Gradient Light and Gradient Dark themes

Changes

  • Changed background color of Gradient Light to: background-color: #f9ccff; background-image: linear-gradient(295deg, #f9ccff 0%, #e6bbf9 11%, #9ec6f9 32%, #55e6ee 60%, #91f5d1 74%, #f9ffbf 98%); from background: rgb(255,253,141); background: linear-gradient(142deg, rgba(255,253,141,1) 0%, rgba(252,183,255,1) 35%, rgba(144,236,255,1) 100%);

  • Changed background color of Gradient Dark to: background-color: #4e037d; background-image: linear-gradient(162deg, #4e037d 8%, #40006a 22%, #270075 60%, #032e79 92%); ; from
    background-color: #280760; background-image: linear-gradient(225deg, #280760 47%, #03135e 92%);

Checklist

  • Updated the changelog at CHANGES.md- added changes under Themes:

CHANGES.md Outdated Show resolved Hide resolved
CHANGES.md Outdated Show resolved Hide resolved
Comment on lines 9 to 11
display: block;
overflow-x: auto;
padding: 0.5em;
Copy link
Member

Choose a reason for hiding this comment

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

Do not add these, these are added during the build process.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed

@joshgoebel
Copy link
Member

Any further context for the "why" of this change?

@samiaab1990
Copy link
Contributor Author

Any further context for the "why" of this change?

Wanted to enhance the background of the original themes: additionally made the color of .hljs-string in the light theme a little darker so it's visible in contrast with the background-just some aesthetic changes I thought would look better.

@joshgoebel joshgoebel merged commit e1abb9b into highlightjs:main Nov 30, 2021
@joshgoebel
Copy link
Member

@samiaab1990 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