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

fix: Move styles into app namespace to avoid ember-cli-sass addon.scss error #537

Closed
wants to merge 1 commit into from

Conversation

ohcibi
Copy link

@ohcibi ohcibi commented Apr 22, 2021

The ember-cli-sass plugin has a bug that throws an error about a missing
addon.sass file of the plugin when

  • The host app and one of it's addons both use ember-cli-sass addon and
  • The host app uses .sass extension and
  • The plugin uses .scss extension and
  • The plugin has a addon/styles directory

All 4 Conditions are required to reproduce. Best practice amongst ember-
cli-sass addon authors became to add stylesheets to the apps namespace
and require the developer to import these explicitly into the app's
styles. This solution allows both the app and the addon author to use
sass with whatever extension they prefer.

The ember-cli-sass plugin has a bug that throws an error about a missing
addon.sass file of the plugin when
* The host app and one of it's addons both use ember-cli-sass addon and
* The host app uses .sass extension and
* The plugin uses .scss extension and
* The plugin has a addon/styles directory

All 4 Conditions are required to reproduce. Best practice amongst ember-
cli-sass addon authors became to add stylesheets to the apps namespace
and require the developer to import these explicitly into the app's
styles. This solution allows both the app and the addon author to use
sass with whatever extension they prefer.
@ohcibi ohcibi changed the title Move styles into app namespace to avoid ember-cli-sass addon.scss error fix: Move styles into app namespace to avoid ember-cli-sass addon.scss error Apr 22, 2021
@ohcibi ohcibi marked this pull request as draft April 22, 2021 17:26
@ohcibi ohcibi marked this pull request as ready for review April 22, 2021 17:27
@bertdeblock
Copy link
Collaborator

@ohcibi, ember-cli-sass and sass are ditched in v0.13.2, so normally, this shouldn't be an issue anymore.

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.

2 participants