notyou / freebase-site

Automatically exported from code.google.com/p/freebase-site
0 stars 0 forks source link

Less parser error after updating to v1.0.36 #10

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
There seems to be a bug in the less parser when it encounters CSS rules that 
declare multiple properties on one line, and one of those properties is 
background-image (or inferred through CSS shorthand, such as 'background: 
url(/foo)'.

What steps will reproduce the problem?
1. http://devel.sandbox-freebase.com:8115/appeditor/MANIFEST/appeditor.mf.css

Here's the stack trace:

Error in MANIFEST

JS exception: [object Object]

at /freebase/site/manifest/less: 397

    * /freebase/site/manifest/less: 397
    * /freebase/site/manifest/less: 346
    * /freebase/site/manifest/MANIFEST: 106
    * /freebase/site/manifest/MANIFEST: 194
    * /freebase/site/manifest/MANIFEST: 371
    * /freebase/site/core/MANIFEST: 40
    * /freebase/site/appeditor/MANIFEST: 33

Original issue reported on code.google.com by daep...@google.com on 2 Nov 2010 at 5:45

GoogleCodeExporter commented 9 years ago
Fixed by quoting all of our url() definitions

Original comment by culbert...@google.com on 3 Nov 2010 at 12:14