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

Fix bug where css variable nesting was broken #386

Merged
merged 1 commit into from
May 20, 2019

Conversation

tylercloke
Copy link
Contributor

@tylercloke tylercloke commented May 17, 2019

🔩 Description

This commit 58632b3#diff-c206cc432a36a0164b93bff1503dc802L30 accidentally changed css variable nesting incorrectly in a few spots from this:

var(--non-default, var(--default));

To this improper format:

var(--non-default), var(--default);

I've fixed it in all the places I found in that commit.

Screen Shot 2019-05-17 at 3 21 50 PM

Screen Shot 2019-05-17 at 3 26 47 PM

Screen Shot 2019-05-17 at 3 35 07 PM

⛓️ Related Resources

✅ Checklist

  • Necessary tests added/updated?
  • Necessary docs added/updated?
  • Code actually executed?
  • Vetting performed (unit tests, lint, etc.)?

@tylercloke tylercloke changed the title Fix bug where css variable nested was broken Fix bug where css variable nesting was broken May 17, 2019
@tylercloke tylercloke force-pushed the tc/fix-nested-css-var-overrides branch 2 times, most recently from 2c28756 to 1c1bcd6 Compare May 17, 2019 22:28
…-hsla references

This commit 58632b3#diff-c206cc432a36a0164b93bff1503dc802L30 accidentally changed css variable nesting incorrectly in a few spots from this:

var(--non-default, var(--default))

To this improper format:

var(--non-default), var(--default)

I've fixed it in all the places I found in that commit.

Signed-off-by: Tyler Cloke <[email protected]>
@tylercloke tylercloke force-pushed the tc/fix-nested-css-var-overrides branch from 1c1bcd6 to bc93f3f Compare May 17, 2019 22:36
@msorens msorens added automate-ui bug 🐛 Something isn't working ui labels May 17, 2019
Copy link
Contributor

@msorens msorens left a comment

Choose a reason for hiding this comment

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

✔️ Create User, Create Token, and Create Project restored with sizing and backgrounding.
✔️ Node integrations shadow gone.

Copy link
Contributor

@susanev susanev left a comment

Choose a reason for hiding this comment

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

i reviewed all the pages i know about and nothing stood out as being different
thanks again for fixing this soooo quickly 🎉

@susanev susanev merged commit 0721894 into master May 20, 2019
@chef-ci chef-ci deleted the tc/fix-nested-css-var-overrides branch May 20, 2019 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automate-ui bug 🐛 Something isn't working ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants