@import directive in an already @import'd file uses full url with hash as base

[dbergey]

My page's url looks like this, with a hash in the middle: http://mysite.com/#/controller/function/

In a <style> tag on the main page, I have (which works fine):

@import "/assets/main.less";

Inside main.less is another @import without a path:

@import "include.less"

The browser requests it as "http://mysite.com/#/controller/function/include.less", disregarding the hash. I have discovered that if I comment out the block:

// Stylesheets in IE don't always return the full path
if (! /^(https?|file):/.test(href)) {
    href = url.slice(0, url.lastIndexOf('/') + 1) + href;
}

...it works, and I have worked around it for now in Safari by adding a second condition thusly:

// Stylesheets in IE don't always return the full path
if (! /^(https?|file):/.test(href) && /msie/.test(navigator.userAgent)) {
    href = url.slice(0, url.lastIndexOf('/') + 1) + href;
}

...but that is of course not a good long-term solution, and probably doesn't fix the issue in IE.

Being unfamiliar with the less.js codebase, I'm not sure what or where the problem is. Help?

[neonstalwart]

it should probably work to just change the url like this when it's declared:
var url = window.location.href.replace(/^([^?#]+).*$/, '$1');

[dbergey]

Did it simpler:

var url = window.location.href.replace(/#.*$/, '');

Still would be nice to have it in the official build.

[neonstalwart]

yours is simpler... but needs to account for '?' in the url which will have the same problem as '#' ;)
var url = window.location.href.replace(/[?#].*$/, '');

Still would be nice to have it in the official build.

just fork, modify, and add a pull request. sometimes it takes a while for a response but @cloudhead eventually responds.

[dbergey]

I realized that after I posted it. Shall fork.

[lukeapage]

The pull request to fix this bug by @dbergey was committed.. therefore this particular bug should be fixed.. ? Any disagreements?