Skip to content

feature/enable-scss - issue #895 #952

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

Merged
merged 11 commits into from
Jun 2, 2016
Merged

feature/enable-scss - issue #895 #952

merged 11 commits into from
Jun 2, 2016

Conversation

hankehly
Copy link
Contributor

@hankehly hankehly commented Jun 1, 2016

Define stylesheet implementation of choice without manually editing the seed (#895 ).

hankehly added 8 commits June 1, 2016 13:48
• SCSS stylesheets are ignored unless '--scss' flag is specified
• Renamed methods ending in 'css' to more generic 'stylesheets'
• Reduced duplication by reusing SCSS processing method for all stylesheets
• Added methods for finding SCSS files & their destinations based on wheth-
  er they are 'component' or 'external' stylesheets
• Renamed methods ending in 'css' to more generic 'stylesheets'
@joshwiens
Copy link
Contributor

@hank-ehly You will need to fix the linting errors as they fail CI

@40x
Copy link

40x commented Jun 1, 2016

+1 for this, much needed feature

@hankehly
Copy link
Contributor Author

hankehly commented Jun 1, 2016

@d3viant0ne

Understood. I'll deal with this asap.

@mgechev
Copy link
Owner

mgechev commented Jun 1, 2016

@hank-ehly great job! I restarted the AppVeyor build, tomorrow I'll review and merge.

@mgechev mgechev self-assigned this Jun 1, 2016
@Shyam-Chen
Copy link
Contributor

Put it in the wiki is better (IMO)

@mgechev mgechev merged commit e71cba7 into mgechev:master Jun 2, 2016
@TheDonDope
Copy link
Contributor

We should update the Wiki article https://github.com/mgechev/angular2-seed/wiki/Add-Sass-support now that the PR is merged.

@TheDonDope
Copy link
Contributor

@hank-ehly what is the other necessary minimum setup necessary to make use of the scss processing? Is renaming the .css files to .scss files enough? Do we still need to move the src/client/css/main.css and rename to src/client/assets/main.scss ? Thanks for the info!

@hankehly hankehly deleted the feature/enable-scss branch June 2, 2016 23:22
@hankehly
Copy link
Contributor Author

hankehly commented Jun 2, 2016

@TheDonDope

The minimum requirements to enable SCSS processing are:

@TheDonDope
Copy link
Contributor

@hank-ehly Thank you! I will update the Wiki accordingly with this info.

@hankehly
Copy link
Contributor Author

hankehly commented Jun 3, 2016

@TheDonDope

Thank you. I would like to note that setting the flag through project.config is currently not working. I apologize for not having researched further, but I believe it may be related to issue #823.

@TheDonDope
Copy link
Contributor

@hank-ehly You are welcome. Yes, i updated the wiki article (https://github.com/mgechev/angular2-seed/wiki/Enabling-SASS-support) and included that info. Could you have a quick review it, if i described it correctly? Thank you!

@hankehly
Copy link
Contributor Author

hankehly commented Jun 3, 2016

@TheDonDope

I just read the updated wiki page. Everything looks great!
Thank you for your time.

Hank

@chl03ks
Copy link
Contributor

chl03ks commented Jun 6, 2016

this configuration is not working for build.prod is not injecting the NPM dependencies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants