Skip to content

Use includePaths to import Bourbon#14

Open
vlasscontreras wants to merge 2 commits into
dbrekalo:masterfrom
vlasscontreras:master
Open

Use includePaths to import Bourbon#14
vlasscontreras wants to merge 2 commits into
dbrekalo:masterfrom
vlasscontreras:master

Conversation

@vlasscontreras

@vlasscontreras vlasscontreras commented Apr 12, 2019

Copy link
Copy Markdown

When using simpleLightbox as dependency, and importing simpleLighbox.scss to the main stylesheet of the project, Sass returns an error about importing the '../node_modules/bourbon/app/assets/stylesheets/bourbon' because is a relative path and won't refer to the same file all the time.

Using includePaths allows to use @import 'bourbon'; without errors.

@ghost

ghost commented Jan 30, 2022

Copy link
Copy Markdown

This repo seems dead. Did you find any other alternatives @vlasscontreras ? thanks a lot
@dbrekalo still here my friend ?

thanks

@vlasscontreras

Copy link
Copy Markdown
Author

@bacloud22 No, I switched to Vue.js, the solution was different anyways.

This library is still useful when using the compiled version though.

@ghost

ghost commented Jan 30, 2022

Copy link
Copy Markdown

Thanks for the confirmation. I'm using it in a very old and dormant project that I would maybe... Maybe mount to the internet 😯..
Thanks for the work @vlasscontreras !!

Have a great day!

@ghost

ghost commented Jan 30, 2022

Copy link
Copy Markdown

It would be nice to have this PR merged if you agree.. importing and compiling the scss is really great.
Thanks

@vlasscontreras

Copy link
Copy Markdown
Author

I agree. However, I'm not one of the maintainers, so, a solution for you could be fork it and make it available though NPM.

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.

1 participant