Skip to content

Commit 5ca6893

Browse files
Rich-HarrisRich-Harris
authored
Merge branch 'master' into gh-781 
2 parents 75651bb + dc48ae6 commit 5ca6893

File tree

62 files changed

+1357
-1388
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

62 files changed

+1357
-1388
lines changed

‎src/generators/Generator.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -210,10 +210,10 @@ export default class Generator {
210210
}
211211

212212
if (globalWhitelist.has(name)) {
213-
code.prependRight(node.start, `( '${name}' in state ? state.`);
213+
code.prependRight(node.start, `('${name}' in state ? state.`);
214214
code.appendLeft(
215215
node.object ? node.object.end : node.end,
216-
` : ${name} )`
216+
` : ${name})`
217217
);
218218
} else {
219219
code.prependRight(node.start, `state.`);
@@ -337,7 +337,7 @@ export default class Generator {
337337

338338
if (defaultImport) {
339339
statements.push(
340-
`${name} = ( ${name} && ${name}.__esModule ) ? ${name}['default'] : ${name};`
340+
`${name} = (${name} && ${name}.__esModule) ? ${name}['default'] : ${name};`
341341
);
342342
}
343343
});
@@ -629,8 +629,8 @@ export default class Generator {
629629
// user code gets wrapped in an IIFE
630630
if (js.content.body.length) {
631631
const prefix = hasDefaultExport
632-
? `var ${this.alias('template')} = (function () {`
633-
: `(function () {`;
632+
? `var ${this.alias('template')} = (function() {`
633+
: `(function() {`;
634634
this.code
635635
.prependRight(js.content.start, prefix)
636636
.appendLeft(js.content.end, '}());');

‎src/generators/dom/Block.ts

Lines changed: 19 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ export default class Block {
135135
this.mount(name, parentNode);
136136

137137
if (isToplevel) {
138-
this.builders.unmount.addLine(`@detachNode( ${name} );`);
138+
this.builders.unmount.addLine(`@detachNode(${name});`);
139139
}
140140
}
141141

@@ -180,9 +180,9 @@ export default class Block {
180180

181181
mount(name: string, parentNode: string) {
182182
if (parentNode) {
183-
this.builders.mount.addLine(`@appendNode( ${name}, ${parentNode} );`);
183+
this.builders.mount.addLine(`@appendNode(${name}, ${parentNode});`);
184184
} else {
185-
this.builders.mount.addLine(`@insertNode( ${name}, #target, anchor );`);
185+
this.builders.mount.addLine(`@insertNode(${name}, #target, anchor);`);
186186
}
187187
}
188188

@@ -225,7 +225,7 @@ export default class Block {
225225
properties.addBlock(`create: @noop,`);
226226
} else {
227227
properties.addBlock(deindent`
228-
create: function () {
228+
create: function() {
229229
${this.builders.create}
230230
${!this.builders.hydrate.isEmpty() && `this.hydrate();`}
231231
},
@@ -237,7 +237,7 @@ export default class Block {
237237
properties.addBlock(`claim: @noop,`);
238238
} else {
239239
properties.addBlock(deindent`
240-
claim: function ( nodes ) {
240+
claim: function(nodes) {
241241
${this.builders.claim}
242242
${!this.builders.hydrate.isEmpty() && `this.hydrate();`}
243243
},
@@ -247,7 +247,7 @@ export default class Block {
247247

248248
if (!this.builders.hydrate.isEmpty()) {
249249
properties.addBlock(deindent`
250-
hydrate: function ( nodes ) {
250+
hydrate: function(nodes) {
251251
${this.builders.hydrate}
252252
},
253253
`);
@@ -257,7 +257,7 @@ export default class Block {
257257
properties.addBlock(`mount: @noop,`);
258258
} else {
259259
properties.addBlock(deindent`
260-
mount: function ( #target, anchor ) {
260+
mount: function(#target, anchor) {
261261
${this.builders.mount}
262262
},
263263
`);
@@ -268,7 +268,7 @@ export default class Block {
268268
properties.addBlock(`update: @noop,`);
269269
} else {
270270
properties.addBlock(deindent`
271-
update: function ( changed, ${this.params.join(', ')} ) {
271+
update: function(changed, ${this.params.join(', ')}) {
272272
${this.builders.update}
273273
},
274274
`);
@@ -278,20 +278,20 @@ export default class Block {
278278
if (this.hasIntroMethod) {
279279
if (hasIntros) {
280280
properties.addBlock(deindent`
281-
intro: function ( #target, anchor ) {
282-
if ( ${introing} ) return;
281+
intro: function(#target, anchor) {
282+
if (${introing}) return;
283283
${introing} = true;
284284
${hasOutros && `${outroing} = false;`}
285285
286286
${this.builders.intro}
287287
288-
this.mount( #target, anchor );
288+
this.mount(#target, anchor);
289289
},
290290
`);
291291
} else {
292292
properties.addBlock(deindent`
293-
intro: function ( #target, anchor ) {
294-
this.mount( #target, anchor );
293+
intro: function(#target, anchor) {
294+
this.mount(#target, anchor);
295295
},
296296
`);
297297
}
@@ -300,8 +300,8 @@ export default class Block {
300300
if (this.hasOutroMethod) {
301301
if (hasOutros) {
302302
properties.addBlock(deindent`
303-
outro: function ( ${this.alias('outrocallback')} ) {
304-
if ( ${outroing} ) return;
303+
outro: function(${this.alias('outrocallback')}) {
304+
if (${outroing}) return;
305305
${outroing} = true;
306306
${hasIntros && `${introing} = false;`}
307307
@@ -312,7 +312,7 @@ export default class Block {
312312
`);
313313
} else {
314314
properties.addBlock(deindent`
315-
outro: function ( outrocallback ) {
315+
outro: function(outrocallback) {
316316
outrocallback();
317317
},
318318
`);
@@ -323,7 +323,7 @@ export default class Block {
323323
properties.addBlock(`unmount: @noop,`);
324324
} else {
325325
properties.addBlock(deindent`
326-
unmount: function () {
326+
unmount: function() {
327327
${this.builders.unmount}
328328
},
329329
`);
@@ -333,16 +333,14 @@ export default class Block {
333333
properties.addBlock(`destroy: @noop`);
334334
} else {
335335
properties.addBlock(deindent`
336-
destroy: function () {
336+
destroy: function() {
337337
${this.builders.destroy}
338338
}
339339
`);
340340
}
341341

342342
return deindent`
343-
function ${this.name} ( ${this.params.join(', ')}, #component${this.key
344-
? `, ${localKey}`
345-
: ''} ) {
343+
function ${this.name}(${this.params.join(', ')}, #component${this.key ? `, ${localKey}` : ''}) {
346344
${this.variables.size > 0 &&
347345
`var ${Array.from(this.variables.keys())
348346
.map(key => {

‎src/generators/dom/index.ts

Lines changed: 37 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -91,9 +91,9 @@ export default function dom(
9191

9292
const condition = `isInitial || ${deps.map(dep => `changed.${dep}`).join(' || ')}`;
9393

94-
const statement = `if ( @differs( ( state.${key} = @template.computed.${key}( ${deps
94+
const statement = `if (@differs((state.${key} = @template.computed.${key}(${deps
9595
.map(dep => `state.${dep}`)
96-
.join(', ')} ) ), oldState.${key} ) ) changed.${key} = true;`;
96+
.join(', ')})), oldState.${key})) changed.${key} = true;`;
9797

9898
computationBuilder.addConditional(condition, statement);
9999
});
@@ -105,8 +105,8 @@ export default function dom(
105105

106106
if (generator.needsEncapsulateHelper) {
107107
builder.addBlock(deindent`
108-
function @encapsulateStyles ( node ) {
109-
@setAttribute( node, '${generator.stylesheet.id}', '' );
108+
function @encapsulateStyles(node) {
109+
@setAttribute(node, "${generator.stylesheet.id}", "");
110110
}
111111
`);
112112
}
@@ -118,11 +118,11 @@ export default function dom(
118118

119119
if (styles && generator.options.css !== false && !generator.customElement) {
120120
builder.addBlock(deindent`
121-
function @add_css () {
122-
var style = @createElement( 'style' );
121+
function @add_css() {
122+
var style = @createElement("style");
123123
style.id = '${generator.stylesheet.id}-style';
124124
style.textContent = ${styles};
125-
@appendNode( style, document.head );
125+
@appendNode(style, document.head);
126126
}
127127
`);
128128
}
@@ -152,30 +152,28 @@ export default function dom(
152152
const constructorBody = deindent`
153153
${options.dev && `this._debugName = '${debugName}';`}
154154
${options.dev && !generator.customElement &&
155-
`if ( !options || (!options.target && !options._root) ) throw new Error( "'target' is a required option" );`}
155+
`if (!options || (!options.target && !options._root)) throw new Error("'target' is a required option");`}
156156
this.options = options;
157157
${generator.usesRefs && `this.refs = {};`}
158158
this._state = ${templateProperties.data
159-
? `@assign( @template.data(), options.data )`
159+
? `@assign(@template.data(), options.data)`
160160
: `options.data || {}`};
161161
${generator.metaBindings}
162-
${computations.length && `this._recompute( {}, this._state, {}, true );`}
162+
${computations.length && `this._recompute({}, this._state, {}, true);`}
163163
${options.dev &&
164164
Array.from(generator.expectedProperties).map(
165165
prop =>
166-
`if ( !( '${prop}' in this._state ) ) console.warn( "${debugName} was created without expected data property '${prop}'" );`
166+
`if (!('${prop}' in this._state)) console.warn("${debugName} was created without expected data property '${prop}'");`
167167
)}
168168
${generator.bindingGroups.length &&
169-
`this._bindingGroups = [ ${Array(generator.bindingGroups.length)
170-
.fill('[]')
171-
.join(', ')} ];`}
169+
`this._bindingGroups = [${Array(generator.bindingGroups.length).fill('[]').join(', ')}];`}
172170
173171
this._observers = {
174-
pre: Object.create( null ),
175-
post: Object.create( null )
172+
pre: Object.create(null),
173+
post: Object.create(null)
176174
};
177175
178-
this._handlers = Object.create( null );
176+
this._handlers = Object.create(null);
179177
${templateProperties.ondestroy && `this._handlers.destroy = [@template.ondestroy]`}
180178
181179
this._root = options._root || this;
@@ -189,13 +187,13 @@ export default function dom(
189187
${css && `this.shadowRoot.innerHTML = \`<style>${options.dev ? `${css}\n/*# sourceMappingURL=${cssMap.toUrl()} */` : css}</style>\`;`}
190188
` :
191189
(generator.stylesheet.hasStyles && options.css !== false &&
192-
`if ( !document.getElementById( '${generator.stylesheet.id}-style' ) ) @add_css();`)
190+
`if (!document.getElementById("${generator.stylesheet.id}-style")) @add_css();`)
193191
}
194192
195-
${templateProperties.oncreate && `var oncreate = @template.oncreate.bind( this );`}
193+
${templateProperties.oncreate && `var oncreate = @template.oncreate.bind(this);`}
196194
197195
${(templateProperties.oncreate || generator.hasComponents || generator.hasComplexBindings || generator.hasIntroTransitions) && deindent`
198-
if ( !options._root ) {
196+
if (!options._root) {
199197
this._oncreate = [${templateProperties.oncreate && `oncreate`}];
200198
${(generator.hasComponents || generator.hasComplexBindings) && `this._beforecreate = [];`}
201199
${(generator.hasComponents || generator.hasIntroTransitions) && `this._aftercreate = [];`}
@@ -208,22 +206,22 @@ export default function dom(
208206
209207
${generator.slots.size && `this.slots = {};`}
210208
211-
this._fragment = @create_main_fragment( this._state, this );
209+
this._fragment = @create_main_fragment(this._state, this);
212210
213-
if ( options.target ) {
211+
if (options.target) {
214212
${generator.hydratable
215213
? deindent`
216-
var nodes = @children( options.target );
217-
options.hydrate ? this._fragment.claim( nodes ) : this._fragment.create();
218-
nodes.forEach( @detachNode );
214+
var nodes = @children(options.target);
215+
options.hydrate ? this._fragment.claim(nodes) : this._fragment.create();
216+
nodes.forEach(@detachNode);
219217
` :
220218
deindent`
221-
${options.dev && `if ( options.hydrate ) throw new Error( 'options.hydrate only works if the component was compiled with the \`hydratable: true\` option' );`}
219+
${options.dev && `if (options.hydrate) throw new Error("options.hydrate only works if the component was compiled with the \`hydratable: true\` option");`}
222220
this._fragment.create();
223221
`}
224222
${generator.customElement ?
225-
`this._mount( options.target, options.anchor || null );` :
226-
`this._fragment.${block.hasIntroMethod ? 'intro' : 'mount'}( options.target, options.anchor || null );`}
223+
`this._mount(options.target, options.anchor || null);` :
224+
`this._fragment.${block.hasIntroMethod ? 'intro' : 'mount'}(options.target, options.anchor || null);`}
227225
228226
${(generator.hasComponents || generator.hasComplexBindings || templateProperties.oncreate || generator.hasIntroTransitions) && deindent`
229227
${generator.hasComponents && `this._lock = true;`}
@@ -266,13 +264,13 @@ export default function dom(
266264
});
267265
}`}
268266
269-
attributeChangedCallback ( attr, oldValue, newValue ) {
267+
attributeChangedCallback(attr, oldValue, newValue) {
270268
this.set({ [attr]: newValue });
271269
}
272270
}
273271
274-
customElements.define('${generator.tag}', ${name});
275-
@assign( ${prototypeBase}, ${proto}, {
272+
customElements.define("${generator.tag}", ${name});
273+
@assign(${prototypeBase}, ${proto}, {
276274
_mount(target, anchor) {
277275
this._fragment.${block.hasIntroMethod ? 'intro' : 'mount'}(this.shadowRoot, null);
278276
target.insertBefore(this, anchor);
@@ -285,31 +283,31 @@ export default function dom(
285283
`);
286284
} else {
287285
builder.addBlock(deindent`
288-
function ${name} ( options ) {
286+
function ${name}(options) {
289287
${constructorBody}
290288
}
291289
292-
@assign( ${prototypeBase}, ${proto});
290+
@assign(${prototypeBase}, ${proto});
293291
`);
294292
}
295293

296294
builder.addBlock(deindent`
297295
${options.dev && deindent`
298-
${name}.prototype._checkReadOnly = function _checkReadOnly ( newState ) {
296+
${name}.prototype._checkReadOnly = function _checkReadOnly(newState) {
299297
${Array.from(generator.readonly).map(
300298
prop =>
301-
`if ( '${prop}' in newState && !this._updatingReadonlyProperty ) throw new Error( "${debugName}: Cannot set read-only property '${prop}'" );`
299+
`if ('${prop}' in newState && !this._updatingReadonlyProperty) throw new Error("${debugName}: Cannot set read-only property '${prop}'");`
302300
)}
303301
};
304302
`}
305303
306304
${computations.length ? deindent`
307-
${name}.prototype._recompute = function _recompute ( changed, state, oldState, isInitial ) {
305+
${name}.prototype._recompute = function _recompute(changed, state, oldState, isInitial) {
308306
${computationBuilder}
309307
}
310308
` : (!sharedPath && `${name}.prototype._recompute = @noop;`)}
311309
312-
${templateProperties.setup && `@template.setup( ${name} );`}
310+
${templateProperties.setup && `@template.setup(${name});`}
313311
`);
314312

315313
const usedHelpers = new Set();
@@ -342,7 +340,7 @@ export default function dom(
342340

343341
else if (format === 'cjs') {
344342
const SHARED = '__shared';
345-
let requires = `var ${SHARED} = require( ${JSON.stringify(sharedPath)} );`;
343+
let requires = `var ${SHARED} = require(${JSON.stringify(sharedPath)});`;
346344
used.forEach(name => {
347345
const alias = generator.alias(name);
348346
requires += `\nvar ${alias} = ${SHARED}.${name};`;
@@ -392,9 +390,7 @@ export default function dom(
392390
// special case
393391
const global = `_svelteTransitionManager`;
394392

395-
result += `\n\nvar ${generator.alias(
396-
'transitionManager'
397-
)} = window.${global} || ( window.${global} = ${code});`;
393+
result += `\n\nvar ${generator.alias('transitionManager')} = window.${global} || (window.${global} = ${code});`;
398394
} else {
399395
const alias = generator.alias(expression.id.name);
400396
if (alias !== expression.id.name)

0 commit comments

Comments
 (0)