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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

ThemeProvider: Add nesting support #19

Merged
merged 5 commits into from
Jun 29, 2018
Merged

Conversation

ItsJonQ
Copy link
Contributor

@ItsJonQ ItsJonQ commented Jun 29, 2018

ThemeProvider: Add nesting support 馃枌

This update adds nesting support to ThemeProvider, allowing you to
continously add onto the outer/parent Theme as required.

The scope feature was also abstracted from ThemeProvider into a dedicated
ScopeProvider. How it influences the compiled CSS remains the same.

Internally, the StyleSheet class instantiation has been changed into a
factory to produces a Singleton, used throughout the Providers and styled.

This update adds nesting support to `ThemeProvider`, allowing you to
continously add onto the outer/parent Theme as required.

The `scope` feature was also abstracted from `ThemeProvider` into a dedicated
`ScopeProvider`. How it influences the compiled CSS remains the same.

Internally, the `StyleSheet` class instantiation has been changed into a
factory to produces a Singleton, used throughout the Providers and `styled`.
@ItsJonQ ItsJonQ added the enhancement New feature or request label Jun 29, 2018
@ItsJonQ ItsJonQ self-assigned this Jun 29, 2018
@ItsJonQ
Copy link
Contributor Author

ItsJonQ commented Jun 29, 2018

Note: Add additional coverage for scope testing

@coveralls
Copy link

coveralls commented Jun 29, 2018

Pull Request Test Coverage Report for Build 115

  • 60 of 60 (100.0%) changed or added relevant lines in 7 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 110: 0.0%
Covered Lines: 237
Relevant Lines: 237

馃挍 - Coveralls

@ItsJonQ ItsJonQ merged commit b45f9d6 into master Jun 29, 2018
@ItsJonQ ItsJonQ deleted the theme-provider-nesting branch July 3, 2018 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants