-
Notifications
You must be signed in to change notification settings - Fork 12k
ng new --style=compass not working #1685
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
Comments
As far as I can tell, we've actually dropped support for compass per se. I'll update the README. |
Do you have any advice for this bug for |
@saulshanabrook |
@filipesilva I need to use css sprites (svg and png with angular-cli) any options for that? |
@neha-s16 it should work out of the box really. Let me know if it doesn't |
@filipesilva Thanks for prompt response. Which folder should I keep the svgs / images in for it to work and create sprites? I need to know the out folder as well since i would need to include it in styles.scss file. I am using the default project structure and default angular-cli.json. |
@neha-s16 if you're referencing them from css/sass files you can keep them wherever you feel like. The build system will pull them up and rename them and it should just work. If you need to link them from elsewhere or use it in html |
Doesn't look like its working. My images are in assets/images/common/normal and assets/images/common/retina. I used ng build to build the project and it copied all images in dist/ folders but did not generate the normal.png and retina.png sprites as I expected. |
I might be misunderstanding the issue then. Can you open a new issue with a repro so I can test please? |
#4054 New Issue. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
os: darwin x64
angular cli: 1.0.0-beta11-webpack.2
node: 4.2.1
To reproduce:
=> runtime error: (page fails to render)
Uncaught Error: Expected 'styles' to be an array of strings. (assertions.js 20)
Notes:
The text was updated successfully, but these errors were encountered: