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 incorrect variable usage in SCSS #3991

Merged
merged 5 commits into from
Sep 2, 2020
Merged

Fix incorrect variable usage in SCSS #3991

merged 5 commits into from
Sep 2, 2020

Conversation

nyurik
Copy link
Contributor

@nyurik nyurik commented Sep 2, 2020

Summary

Fix incorrect variable usage in SCSS. Not sure how it got compiled.

Checklist

  • Check against all themes for compatibility in both light and dark modes
  • Checked in mobile
  • Checked in Chrome, Safari, Edge, and Firefox
  • Props have proper autodocs
  • Added documentation
  • Checked Code Sandbox works for the any docs examples
  • Added or updated jest tests
  • Checked for breaking changes and labeled appropriately
  • Checked for accessibility including keyboard-only and screenreader modes
  • A changelog entry exists and is marked appropriately

@nyurik nyurik requested a review from cjcenizal September 2, 2020 06:39
@kibanamachine
Copy link

Preview documentation changes for this PR: https://eui.elastic.co/pr_3991/

Copy link
Contributor

@snide snide left a comment

Choose a reason for hiding this comment

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

Don't forget a CL. Good spot.

@nyurik
Copy link
Contributor Author

nyurik commented Sep 2, 2020

dope, thanks @snide, updated. I keep forgetting it :(

CHANGELOG.md Outdated
@@ -10,6 +10,7 @@
- Fixed type mismatch between `EuiSelectable` options extended via `EuiSelectableOption` and internal option types ([#3983](https://github.com/elastic/eui/pull/3983))
- Fixed `EuiButton` CSS for RTL languages by using `margin-inline-[pos]` instead of `margin-[pos]` ([#3974](https://github.com/elastic/eui/pull/3974))
- Fix server-side rendering of `EuiBreadcrumbs` and `EuiCollapsibleNav` ([#3970](https://github.com/elastic/eui/pull/3970))
- Fix incorrect `euiCodeBlockNameColor` variable usage for `.hljs-name` in SCSS ([#3991](https://github.com/elastic/eui/pull/3991))
Copy link
Contributor

Choose a reason for hiding this comment

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

Because there was a release after you made this PR, your Changelog needs to be in the master section above.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I suspect changelog file doesn't like me... Or i need another cup of coffee... or both.

@kibanamachine
Copy link

Preview documentation changes for this PR: https://eui.elastic.co/pr_3991/

@cjcenizal cjcenizal added the bug label Sep 2, 2020
Copy link
Contributor

@cjcenizal cjcenizal left a comment

Choose a reason for hiding this comment

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

Code LGTM, didn't test locally. :)

CHANGELOG.md Show resolved Hide resolved
Copy link
Contributor

@cchaos cchaos left a comment

Choose a reason for hiding this comment

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

I added the Bug Fixes header, but once CI passes (again), good to merge. Thanks again @nyurik !

@kibanamachine
Copy link

Preview documentation changes for this PR: https://eui.elastic.co/pr_3991/

@nyurik nyurik merged commit 3d31ddb into master Sep 2, 2020
@nyurik nyurik deleted the nyurik-patch-1 branch September 2, 2020 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants