Merge pull request #863 from sveltejs/short-method-names

use short method names

Author: Rich-Harris

src/generators/dom/Block.ts

@@ -226,53 +226,53 @@ export default class Block {
 		}
 
 		if (this.builders.create.isEmpty()) {
-			properties.addBlock(`create: @noop,`);
+			properties.addBlock(`c: @noop,`);
 		} else {
 			properties.addBlock(deindent`
-				create: function() {
+				c: function create() {
 					${this.builders.create}
-					${!this.builders.hydrate.isEmpty() && `this.hydrate();`}
+					${!this.builders.hydrate.isEmpty() && `this.h();`}
 				},
 			`);
 		}
 
 		if (this.generator.hydratable) {
 			if (this.builders.claim.isEmpty()) {
-				properties.addBlock(`claim: @noop,`);
+				properties.addBlock(`l: @noop,`);
 			} else {
 				properties.addBlock(deindent`
-					claim: function(nodes) {
+					l: function claim(nodes) {
 						${this.builders.claim}
-						${!this.builders.hydrate.isEmpty() && `this.hydrate();`}
+						${!this.builders.hydrate.isEmpty() && `this.h();`}
 					},
 				`);
 			}
 		}
 
 		if (!this.builders.hydrate.isEmpty()) {
 			properties.addBlock(deindent`
-				hydrate: function() {
+				h: function hydrate() {
 					${this.builders.hydrate}
 				},
 			`);
 		}
 
 		if (this.builders.mount.isEmpty()) {
-			properties.addBlock(`mount: @noop,`);
+			properties.addBlock(`m: @noop,`);
 		} else {
 			properties.addBlock(deindent`
-				mount: function(#target, anchor) {
+				m: function mount(#target, anchor) {
 					${this.builders.mount}
 				},
 			`);
 		}
 
 		if (this.hasUpdateMethod) {
 			if (this.builders.update.isEmpty()) {
-				properties.addBlock(`update: @noop,`);
+				properties.addBlock(`p: @noop,`);
 			} else {
 				properties.addBlock(deindent`
-					update: function(changed, ${this.params.join(', ')}) {
+					p: function update(changed, ${this.params.join(', ')}) {
 						${this.builders.update}
 					},
 				`);
@@ -282,20 +282,20 @@ export default class Block {
 		if (this.hasIntroMethod) {
 			if (hasIntros) {
 				properties.addBlock(deindent`
-					intro: function(#target, anchor) {
+					i: function intro(#target, anchor) {
 						if (${introing}) return;
 						${introing} = true;
 						${hasOutros && `${outroing} = false;`}
 
 						${this.builders.intro}
 
-						this.mount(#target, anchor);
+						this.m(#target, anchor);
 					},
 				`);
 			} else {
 				properties.addBlock(deindent`
-					intro: function(#target, anchor) {
-						this.mount(#target, anchor);
+					i: function intro(#target, anchor) {
+						this.m(#target, anchor);
 					},
 				`);
 			}
@@ -304,7 +304,7 @@ export default class Block {
 		if (this.hasOutroMethod) {
 			if (hasOutros) {
 				properties.addBlock(deindent`
-					outro: function(${this.alias('outrocallback')}) {
+					o: function outro(${this.alias('outrocallback')}) {
 						if (${outroing}) return;
 						${outroing} = true;
 						${hasIntros && `${introing} = false;`}
@@ -315,29 +315,30 @@ export default class Block {
 					},
 				`);
 			} else {
+				// TODO should this be a helper?
 				properties.addBlock(deindent`
-					outro: function(outrocallback) {
+					o: function outro(outrocallback) {
 						outrocallback();
 					},
 				`);
 			}
 		}
 
 		if (this.builders.unmount.isEmpty()) {
-			properties.addBlock(`unmount: @noop,`);
+			properties.addBlock(`u: @noop,`);
 		} else {
 			properties.addBlock(deindent`
-				unmount: function() {
+				u: function unmount() {
 					${this.builders.unmount}
 				},
 			`);
 		}
 
 		if (this.builders.destroy.isEmpty()) {
-			properties.addBlock(`destroy: @noop`);
+			properties.addBlock(`d: @noop`);
 		} else {
 			properties.addBlock(deindent`
-				destroy: function() {
+				d: function destroy() {
 					${this.builders.destroy}
 				}
 			`);

src/generators/dom/index.ts

@@ -236,23 +236,23 @@ export default function dom(
 		this._fragment = @create_main_fragment(this._state, this);
 
 		${generator.customElement ? deindent`
-			this._fragment.create();
-			this._fragment.${block.hasIntroMethod ? 'intro' : 'mount'}(this.shadowRoot, null);
+			this._fragment.c();
+			this._fragment.${block.hasIntroMethod ? 'i' : 'm'}(this.shadowRoot, null);
 
 			if (options.target) this._mount(options.target, options.anchor || null);
 		` : deindent`
 			if (options.target) {
 				${generator.hydratable
 					? deindent`
 						var nodes = @children(options.target);
-						options.hydrate ? this._fragment.claim(nodes) : this._fragment.create();
+						options.hydrate ? this._fragment.l(nodes) : this._fragment.c();
 						nodes.forEach(@detachNode);
 					` :
 					deindent`
 						${options.dev && `if (options.hydrate) throw new Error("options.hydrate only works if the component was compiled with the \`hydratable: true\` option");`}
-						this._fragment.create();
+						this._fragment.c();
 					`}
-				this._fragment.${block.hasIntroMethod ? 'intro' : 'mount'}(options.target, options.anchor || null);
+				this._fragment.${block.hasIntroMethod ? 'i' : 'm'}(options.target, options.anchor || null);
 
 				${(generator.hasComponents || generator.hasComplexBindings || templateProperties.oncreate || generator.hasIntroTransitions) && deindent`
 					${generator.hasComponents && `this._lock = true;`}

src/generators/dom/visitors/Component.ts

@@ -230,10 +230,10 @@ export default function visitComponent(
 		${beforecreate}
 	`);
 
-	block.builders.create.addLine(`${name}._fragment.create();`);
+	block.builders.create.addLine(`${name}._fragment.c();`);
 
 	block.builders.claim.addLine(
-		`${name}._fragment.claim( ${state.parentNodes} );`
+		`${name}._fragment.l( ${state.parentNodes} );`
 	);
 
 	block.builders.mount.addLine(