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

refactor(demos): Split out and remove demos.scss #1285

Merged
merged 4 commits into from
Sep 13, 2017

Conversation

kfranqueiro
Copy link
Contributor

This gives each demo (or set of demos, in some cases) its own separate scss
file, allowing us to more clearly add demo-specific Sass to illustrate use of
the theme mixins we've started developing. It should also open up more
potential to speed up our compilation process.

This PR does not go about moving all demo-specific styles from the HTML files into the SCSS files; I figured that'd make the diff ridiculously large, so we can tackle that separately if/when we want to do it.

This gives each demo (or set of demos, in some cases) its own separate scss
file, allowing us to more clearly add demo-specific Sass to illustrate use of
the theme mixins we've started developing. It should also open up more
potential to speed up our compilation process.
@codecov-io
Copy link

codecov-io commented Sep 12, 2017

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1285   +/-   ##
======================================
  Coverage    99.9%   99.9%           
======================================
  Files          69      69           
  Lines        3314    3314           
  Branches      409     409           
======================================
  Hits         3311    3311           
  Misses          3       3

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 2db131d...de0905a. Read the comment docs.

Copy link
Contributor

@acdvorak acdvorak left a comment

Choose a reason for hiding this comment

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

LGTM!

@acdvorak acdvorak self-assigned this Sep 12, 2017
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.

None yet

3 participants