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

chore(tabs): removed .mdc-toolbar specific selectors #1979

Merged
merged 25 commits into from
Jan 23, 2018

Conversation

moog16
Copy link
Contributor

@moog16 moog16 commented Jan 17, 2018

Partial fix of: #1152

BREAKING CHANGE: removal of .mdc-toolbar selector forces clients to
customize tab-bars within toolbars that require a different ink color.

@moog16 moog16 changed the base branch from chore/tabs/color-sass-mixins to master January 19, 2018 19:23
@codecov-io
Copy link

codecov-io commented Jan 19, 2018

Codecov Report

Merging #1979 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1979   +/-   ##
=======================================
  Coverage   99.43%   99.43%           
=======================================
  Files          84       84           
  Lines        3717     3717           
  Branches      485      485           
=======================================
  Hits         3696     3696           
  Misses         21       21

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 53d412a...81fdf94. Read the comment docs.

@lynnmercier lynnmercier self-requested a review January 22, 2018 23:36
@lynnmercier lynnmercier self-assigned this Jan 22, 2018
Copy link
Contributor

@lynnmercier lynnmercier left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -71,7 +71,7 @@ In order to implement a component via a foundation, take the following steps:

1. Include the component's CSS on the page any way you wish
2. Add an instance property to your component which will be set to the proper foundation class.
We'll calls this `mdcFoundation`.
We'll call this `mdcFoundation`.
Copy link
Contributor

Choose a reason for hiding this comment

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

This does not belong in this PR

@moog16 moog16 merged commit b32d013 into master Jan 23, 2018
@moog16 moog16 deleted the chore/tabs/remove-toolbar-specific-styles branch January 23, 2018 00:39
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.

3 participants