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

Box Shadow Examples #94

Merged
merged 6 commits into from
Apr 26, 2019
Merged

Box Shadow Examples #94

merged 6 commits into from
Apr 26, 2019

Conversation

daigof
Copy link
Contributor

@daigof daigof commented Apr 25, 2019

  • It made sense to add this to Constants section like Colors examples
  • I also refactored/cleaned up the Colors Story

@daigof daigof requested a review from snags88 April 25, 2019 19:28
@snags88 snags88 had a problem deploying to curology-radiance-pr-94 April 25, 2019 19:29 Failure
@snags88 snags88 had a problem deploying to curology-radiance-pr-94 April 25, 2019 19:40 Failure
@snags88 snags88 temporarily deployed to curology-radiance-pr-94 April 25, 2019 19:44 Inactive
@snags88
Copy link
Contributor

snags88 commented Apr 25, 2019

Could we do a similar thing as Colors where we categorize the box shadow examples by context (e.g. legacyColors, brandColors, etc.). I think for box shadows, we have few different ones like Base config, alias config etc

@daigof
Copy link
Contributor Author

daigof commented Apr 25, 2019

@snags88 yep done

@snags88 snags88 temporarily deployed to curology-radiance-pr-94 April 25, 2019 20:18 Inactive
@snags88
Copy link
Contributor

snags88 commented Apr 25, 2019

@daigof i'm not seeing the values or the box shadow on the story now:
Screen Shot 2019-04-25 at 1 50 18 PM

@daigof
Copy link
Contributor Author

daigof commented Apr 26, 2019

ughh sorry, fixed

@daigof daigof merged commit 467357d into master Apr 26, 2019
@daigof daigof deleted the box-shadow-examples branch April 26, 2019 20:17
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.

2 participants