Skip to content
This repository has been archived by the owner on May 19, 2020. It is now read-only.

LESS-1.3.0 #12

Closed
BenjiZombie opened this issue Mar 14, 2012 · 11 comments
Closed

LESS-1.3.0 #12

BenjiZombie opened this issue Mar 14, 2012 · 11 comments

Comments

@BenjiZombie
Copy link

Trying to compile a .LESS file that was working with LESS-1.2.2 with LESS-1.3.0 and I now get the following error:

ERROR:
type: Syntax
message: 'eval' is null or not an object
filename: ..\app\vendors\bootstrap\less\bootstrap.less
column: -1
extract: ,, * Bootstrap v2.0.2

It's working fine if I use LESS-1.3.0 directly in my html file, so I know the .LESS file has no error. Could this be related to the way the .swf file calls less?

Thankx

@duncansmart
Copy link
Owner

Might be related to this: https://twitter.com/cloudhead/status/178567217309421568 "dropping IE support on client" might also imply that IE's JScript engine is no longer supported. Not sure yet though.

@hereswhatidid
Copy link

Also getting this error but only on the latest (2.0.2) bootstrap.less file. Other .less files compile fine.

@arabold
Copy link

arabold commented Mar 17, 2012

Got it fixed by patching bootstrap's mixins.less

It seems that less.js 1.3.0 is failing to properly parse the following mixin definition if used with JScript and duncansmart's WSH script:

// IE7 inline-block
// ----------------
.ie7-inline-block() {
  *display: inline;
  *zoom: 1;
}

the trick is to add a dummy element like this:

// IE7 inline-block
// ----------------
.ie7-inline-block() {
  *display: inline;
  *zoom: 1;
  something: else;
}

Weird? Looks like there's something strange going on in less.js but I haven't had the time to dive into it yet.

@xokko
Copy link

xokko commented Mar 29, 2012

This works too, maybe less intruding:

// IE7 inline-block
// ----------------
.ie7-inline-block() {
  *display: inline;
  *zoom: 1;
  // comment to make it work with WinLESS
}

@cnanney
Copy link

cnanney commented Mar 29, 2012

👍 Thanks for simple fix to get latest Bootstrap working via WinLess.

@govza
Copy link

govza commented Apr 11, 2012

fix doesn't work with bootstrap 2.0.2

@deepesh1801
Copy link

fix doesn't work for bootstrap 2.0.3. Is their a fix available.

@xokko
Copy link

xokko commented May 10, 2012

It works. Just add the comment in mixins.less. And use newest version of WinLESS.

@theoryshaw
Copy link

hmm... @xokko's suggestion didn't work for me, unfort.

@a-ursino
Copy link

it works with bootstrap 2.0.4 and winless 1.5.3 (less.js 1.3.0)
thanks

@grantnorwood
Copy link

I only saw failures when importing the mixins.less file, as well as using arbitrary comment blocks or any pseudo element selectors. The extra comment fix from xokko seems to resolve the issue. (Running WinLESS 1.5.3, less.js 1.3.0, bootstrap 2.0.4)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests