Merge pull request #601 from sveltejs/gh-574

apply t0 styles to nodes if css transition has delay

.gitignore

@@ -12,3 +12,4 @@ coverage.lcov
 test/sourcemaps/samples/*/output.js
 test/sourcemaps/samples/*/output.js.map
 _actual.*
+src/generators/dom/shared.ts

src/shared/transitions.js

@@ -31,13 +31,21 @@ export function wrapTransition ( node, fn, params, intro, outgroup ) {
 	var obj = fn( node, params );
 	var duration = obj.duration || 300;
 	var ease = obj.easing || linear;
+	var cssText;
 
 	// TODO share <style> tag between all transitions?
 	if ( obj.css ) {
 		var style = document.createElement( 'style' );
 	}
 
-	if ( intro && obj.tick ) obj.tick( 0 );
+	if ( intro ) {
+		if ( obj.css && obj.delay ) {
+			cssText = node.style.cssText;
+			node.style.cssText += obj.css( 0 );
+		}
+
+		if ( obj.tick ) obj.tick( 0 );
+	}
 
 	return {
 		t: intro ? 0 : 1,
@@ -70,6 +78,7 @@ export function wrapTransition ( node, fn, params, intro, outgroup ) {
 			program.end = program.start + program.duration;
 
 			if ( obj.css ) {
+				if ( obj.delay ) node.style.cssText = cssText;
 				generateKeyframes( program.a, program.b, program.delta, program.duration, ease, obj.css, node, style );
 			}
 

test/runtime/samples/transition-css-delay/_config.js

@@ -0,0 +1,12 @@
+export default {
+	test ( assert, component, target, window, raf ) {
+		component.set({ visible: true });
+		const div = target.querySelector( 'div' );
+		assert.strictEqual( div.style.opacity, '0' );
+
+		raf.tick( 50 );
+		assert.strictEqual( div.style.opacity, '' );
+
+		component.destroy();
+	}
+};

test/runtime/samples/transition-css-delay/main.html

@@ -0,0 +1,19 @@
+{{#if visible}}
+	<div transition:foo>delayed</div>
+{{/if}}
+
+<script>
+	export default {
+		transitions: {
+			foo: function ( node, params ) {
+				return {
+					delay: 50,
+					duration: 100,
+					css: t => {
+						return `opacity: ${t}`;
+					}
+				};
+			}
+		}
+	};
+</script>