Navigation Menu

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

A few things required to show the deprecation warning #118

Merged
merged 4 commits into from Jan 19, 2017

Conversation

blasten
Copy link
Contributor

@blasten blasten commented Jan 18, 2017

No description provided.

@blasten blasten changed the title Add mixin --iron-component-page-header-panel A few things required to show the deprecation warning Jan 18, 2017
@notwaldorf
Copy link
Contributor

Can you document this mixin in the #Styling section, pls? Also uhhh (and I'm sorry this will sound annoying) are we worried whether just randomly starting to distribute content is a breaking change?

@blasten
Copy link
Contributor Author

blasten commented Jan 19, 2017

@notwaldorf I actually didn't need the mixin! I have specified a selector for the distribution point to address your concern.

@notwaldorf
Copy link
Contributor

LGTM!

@blasten blasten merged commit 56c1735 into master Jan 19, 2017
@blasten blasten deleted the paper-header-mixin branch January 19, 2017 20:35
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.

None yet

3 participants