slick-carousel/slick.scss is imported twice

[shobee]

Hi there,

rails-assets-slick-carousel-1.6.0/app/assets/stylesheets/slick-carousel.scss looks like this:

@import 'slick-carousel/slick.scss';
@import 'slick-carousel/slick.scss';

Not sure if this is the right place to address this and where this can be fixed?

[hut8]

Interesting. I will look into this tomorrow; I have some pretty urgent other stuff right now. Is this causing issues?

[hut8]

OK more information 😄

Reading the bower spec, specifically the history behind this pull request, I think that the slick-carousel folks have a bug in their bower.json. Specifically, I don't think they should be including slick.css in main. On a totally unrelated note, because of bower/bower#2232, they don't need to be including
the version tag either if you're planning on making a PR to that repo 😏

https://github.com/twbs/bootstrap/pull/16359/files -- looks like in 2015, Bootstrap removed the regular .css file and just includes .less in main. So I think that slick-carousel should follow suit. What are your thoughts on this?

[shobee]

Thanks for diving into this so quickly! I think it is obviously which way to go. So I am going to close this issue here and open a new at the slick-carousel repository.