mirror_css/bootstrap
1
0
Fork 0
mirror of https://github.com/twbs/bootstrap.git synced 2025-09-27 22:09:04 +02:00
Code Issues Releases Wiki Activity

Customizer: Catch and properly report compile errors that Less.js's toCSS() can throw.

Browse Source 
Currently these errors are only reported in the JavaScript console, not in the Customizer UI.
Refs #15296

[skip sauce]
[skip validator]
This commit is contained in:
Chris Rebert
2014-12-05 12:58:47 -08:00
parent 7e758f60fc
commit 37183edfd5
1 changed files with 14 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
docs/assets/js/src/customizer.js
View File

@@ -24,7 +24,8 @@ window.onload = function () { // wait for load in a dumb way because B-0
'<div class="container">' + '<div class="container">' +
'<a href="#bsCustomizerAlert" data-dismiss="alert" class="close pull-right" aria-label="Close" role="button"><span aria-hidden="true">&times;</span></a>' + '<a href="#bsCustomizerAlert" data-dismiss="alert" class="close pull-right" aria-label="Close" role="button"><span aria-hidden="true">&times;</span></a>' +
'<p class="bs-customizer-alert-text"><span class="glyphicon glyphicon-warning-sign" aria-hidden="true"></span><span class="sr-only">Warning:</span>' + msg + '</p>' + '<p class="bs-customizer-alert-text"><span class="glyphicon glyphicon-warning-sign" aria-hidden="true"></span><span class="sr-only">Warning:</span>' + msg + '</p>' +
(err.extract ? '<pre class="bs-customizer-alert-extract">' + err.extract.join('\n') + '</pre>' : '') + (err.message ? $('<p></p>').text('Error: ' + err.message)[0].outerHTML : '') +
(err.extract ? $('<pre class="bs-customizer-alert-extract"></pre>').text(err.extract.join('\n'))[0].outerHTML : '') +
'</div>' + '</div>' +
'</div>').appendTo('body').alert() '</div>').appendTo('body').alert()
throw err throw err
@@ -258,12 +259,17 @@ window.onload = function () { // wait for load in a dumb way because B-0
filename: baseFilename + '.css' filename: baseFilename + '.css'
}) })
parser.parse(lessSource, function (err, tree) { parser.parse(lessSource, function (parseErr, tree) {
if (err) { if (parseErr) {
return promise.reject(err) return promise.reject(parseErr)
} }
try {
intoResult[baseFilename + '.css'] = cw + tree.toCSS() intoResult[baseFilename + '.css'] = cw + tree.toCSS()
intoResult[baseFilename + '.min.css'] = cw + tree.toCSS({ compress: true }) intoResult[baseFilename + '.min.css'] = cw + tree.toCSS({ compress: true })
}
catch (compileErr) {
return promise.reject(compileErr)
}
promise.resolve() promise.resolve()
}) })
@@ -306,7 +312,7 @@ window.onload = function () { // wait for load in a dumb way because B-0
} }
promise.resolve(result) promise.resolve(result)
}).fail(function (err) { }).fail(function (err) {
showError('<strong>Ruh roh!</strong> Could not parse less files.', err) showError('<strong>Ruh roh!</strong> Problem parsing or compiling Less files.', err)
promise.reject() promise.reject()
}) })