Skip to content
This repository was archived by the owner on Dec 19, 2024. It is now read-only.

use nested @applies instead of copy pasta #80

Merged
merged 1 commit into from
Dec 2, 2015
Merged

Conversation

notwaldorf
Copy link
Contributor

Nested applies actually work, so let's do that instead. Also, this will allow us to be able to change the font magically for all paper elements, by overriding the right mixin:

<style is="custom-style">
    :root {
      --paper-font-common-base: {
        font-family: serif !important;
      };
    };
</style>

@abdonrd
Copy link
Member

abdonrd commented Dec 1, 2015

Yeah! Since when the nested applies works?

@notwaldorf
Copy link
Contributor Author

@abdonrd not sure, to be honest. for a while.

@cdata
Copy link
Contributor

cdata commented Dec 2, 2015

beautiful 💇 LGTM 👍

notwaldorf added a commit that referenced this pull request Dec 2, 2015
@notwaldorf notwaldorf merged commit d253ec2 into master Dec 2, 2015
@notwaldorf notwaldorf deleted the use-more-apply branch December 2, 2015 20:10
notwaldorf added a commit that referenced this pull request Dec 10, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants