Skip to content

Conversation

@desig9stein
Copy link
Contributor

@desig9stein desig9stein commented Oct 11, 2019

Rename $disable-button-shadow to $button-shadow and allow the user to change the default elevation through the theming engine

Closes #5680

Additional information (check all that apply):

  • Bug fix
  • New functionality
  • Documentation
  • Demos
  • CI/CD

Checklist:

  • All relevant tags have been applied to this PR
  • This PR includes unit tests covering all the new code
  • This PR includes API docs for newly added methods/properties
  • This PR includes feature/README.MD updates for the feature docs
  • This PR includes general feature table updates in the root README.MD
  • This PR includes CHANGELOG.MD updates for newly added functionality
  • This PR contains breaking changes
  • This PR includes ng update migrations for the breaking changes
  • This PR includes behavioral changes and the feature specification has been updated with them

Rename $disable-button-shadow to $button-shadow and allow the user to
change the default elevation through the theming engine

Closes #5680

Signed-off-by: MPopov <desig9stein@gmail.com>
MPopov added 2 commits October 11, 2019 12:41
Signed-off-by: MPopov <desig9stein@gmail.com>
Signed-off-by: MPopov <desig9stein@gmail.com>
simeonoff
simeonoff previously approved these changes Oct 14, 2019
Copy link
Collaborator

@simeonoff simeonoff left a comment

Choose a reason for hiding this comment

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

@desig9stein migrations seem to fail. Can you please check?

@simeonoff simeonoff added ✅ status: verified Applies to PRs that have passed manual verification version: 8.2.x and removed ❌ status: awaiting-test PRs awaiting manual verification labels Oct 14, 2019
Signed-off-by: MPopov <desig9stein@gmail.com>
@desig9stein desig9stein requested a review from simeonoff October 14, 2019 11:58
@kdinev kdinev merged commit 3ff0b28 into 8.2.x Oct 15, 2019
@kdinev kdinev deleted the fix-carousel-shadow-8.2 branch October 15, 2019 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎠 carousel material-theme version: 8.2.x ✅ status: verified Applies to PRs that have passed manual verification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants