search

choffmeister / roboto-fontface-bower

Bower package for the Roboto font-face
Apache License 2.0
153 stars 57 forks source link

Case-sensitivity in URLs #30

Closed thatguystone closed 7 years ago

thatguystone commented 8 years ago

It looks like the URLs created from sass + less are being created as something like, .../Roboto/Roboto-Light.woff2, instead of .../roboto/Roboto-Light.woff2. On a case-sensitive filesystem (Linux, in my case), this means that the files can't be found.

Any thoughts on the best way to address this? A few options I thought of:

  1. Rename roboto to Roboto, but I fear that would break other things.
  2. Use sass's to-lower-case function, but less lacks a similar function.
  3. Add an extra, properly-cased arg to the mixin, but that's just a lot of repeated crap.
sontek commented 8 years ago

I just ran into this issue as well.

Module not found: Error: Cannot resolve 'file' or 'directory' ../../../fonts/Roboto/Roboto-BlackItalic.ttf in /Users/sontek/src/app/static/sass

Because its looking for Roboto instead of roboto

skorenuk commented 8 years ago

I used webpack and sass loader and I resolved issue like this: $roboto-font-path: '~roboto-fontface/fonts' !default; then: @import "~roboto-fontface/css/roboto/sass/roboto-fontface.scss";

But then I got another errors Cannot resolve module 'roboto-fontface/fonts/Roboto/Roboto/Roboto-Black.eot' twice ...Roboto/Roboto/...

Error coz of this line https://github.com/choffmeister/roboto-fontface-bower/blob/develop/css/mixins.scss#L23

paralin commented 7 years ago

Fixed by #31

choffmeister commented 7 years ago

Thanks @Badou90 for fixing this bug.

paralin commented 7 years ago

@choffmeister Thank you for the quick turnaround, everything seems good now.