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

Beginning of css variable implementation (--theme-background) #1306

Merged
merged 7 commits into from Dec 19, 2018
Merged

Beginning of css variable implementation (--theme-background) #1306

merged 7 commits into from Dec 19, 2018

Conversation

Link2Twenty
Copy link
Contributor

@Link2Twenty Link2Twenty commented Dec 11, 2018

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Documentation Update

Description

Added --theme-background leaving the default the same.
This will make no, visible, change to the styles currently.

Adding CSS variables means loading a CSS file containing overrides means you can change the whole sites theme.

:root {
  --theme-background: #303030
}

Related Tickets & Documents

#1104 and #1377

Added to documentation?

  • docs.dev.to
  • readme
  • no documentation needed

Added --theme-background leaving the default the same.
This will make no, visible, change to the styles currently.
Added --theme-background leaving the default the same.
This will make no, visible, change to the styles currently.
Added --theme-background leaving the default the same.
This will make no, visible, change to the styles currently.
Added --theme-background leaving the default the same.
This will make no, visible, change to the styles currently.
Added --theme-background leaving the default the same.
This will make no, visible, change to the styles currently.
Added --theme-background leaving the default the same.
This will make no, visible, change to the styles currently.
@pr-triage pr-triage bot added the PR: unreviewed bot applied label for PR's with no review label Dec 11, 2018
@Link2Twenty Link2Twenty changed the title Beginning of css variable implementation Beginning of css variable implementation (--theme-background) Dec 11, 2018
Copy link
Contributor

@rkichenama rkichenama left a comment

Choose a reason for hiding this comment

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

I like this; adding variable replacements piecemeal is likely a better path than attempting to replace all. Gives time to adopt the pattern

@pr-triage pr-triage bot added PR: reviewed-approved bot applied label for PR's where reviewer approves changes and removed PR: unreviewed bot applied label for PR's with no review labels Dec 11, 2018
@pr-triage pr-triage bot added PR: unreviewed bot applied label for PR's with no review and removed PR: reviewed-approved bot applied label for PR's where reviewer approves changes labels Dec 15, 2018
@Link2Twenty
Copy link
Contributor Author

When this one is merged I'll do a couple more.

@benhalpern benhalpern merged commit 67ee564 into forem:master Dec 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: unreviewed bot applied label for PR's with no review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants