search

SomMeri / less4j

Less language is an extension of css and less4j compiles it into regular css. Less adds several dynamic features into css: variables, expressions, nested rules, and so on. Less was designed to be compatible with css and any correct css file is also correct less file.
145 stars 47 forks source link

Inconsistency with less.js #351

Open gdelhumeau opened 7 years ago

gdelhumeau commented 7 years ago

Hello.

We use LESS4j for a year and we are happy with it. Thanks for the great work.

I just have noticed an inconsistency:

@web-font-path: "https://fonts.googleapis.com/css?family=Source+Sans+Pro:300,400,700";

.web-font(@path) {
  @import url("@{path}");
}
.web-font(@web-font-path);

Results with lessjs:

@import url("https://fonts.googleapis.com/css?family=Source+Sans+Pro:300,400,700");

Results with less4j:

ERROR 959:3 The file @{path}.less does not exist.
 958: .web-font(@path) {
 959:   @import url("@{path}");
 960: }

    at com.github.sommeri.less4j.core.ThreadUnsafeLessCompiler.compile(ThreadUnsafeLessCompiler.java:87)
    at com.github.sommeri.less4j.core.DefaultLessCompiler.compile(DefaultLessCompiler.java:56)

Thanks, Guillaume Delhumeau