Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Adapt-1808: Use TW box shadows for v7 #799

Merged
merged 3 commits into from
Feb 11, 2021
Merged

Adapt-1808: Use TW box shadows for v7 #799

merged 3 commits into from
Feb 11, 2021

Conversation

yvonnetangsu
Copy link
Member

@yvonnetangsu yvonnetangsu commented Feb 11, 2021

READY FOR REVIEW

Summary

  • Get rid of our custom box shadows and use TW default
  • Update our button styles to use TW md drop shadow because that's closer to our previous default
  • Update preview page

Needed By (Date)

  • When does this need to be merged by?

Urgency

  • How critical is this PR?

Steps to Test

  1. See the shadows: https://deploy-preview-799--decanter-v7.netlify.app/#drop-shadows
  2. Look at code

Affected Projects or Products

  • Does this PR impact any particular projects, products, or modules?

Associated Issues and/or People

  • JIRA ticket
  • Other PRs
  • Any other contextual information that might be helpful (e.g., description of a bug that this PR fixes, new functionality that it adds, etc.)
  • Anyone who should be notified? (@mention them here)

See Also

@yvonnetangsu yvonnetangsu changed the base branch from master to v7 February 11, 2021 01:21
@yvonnetangsu
Copy link
Member Author

@sherakama Met with Kerri so this one is ready too

Copy link
Member

@sherakama sherakama left a comment

Choose a reason for hiding this comment

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

GTG

@sherakama sherakama changed the title Adapt-1808 Use TW box shadows for v7 Adapt-1808: Use TW box shadows for v7 Feb 11, 2021
@sherakama sherakama merged commit 3401bc5 into v7 Feb 11, 2021
@sherakama sherakama deleted the ADAPT-1808-shadow branch February 11, 2021 18:19
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants