Address feedback

Author: devversion

src/lib/core/ripple/ripple-ref.ts

@@ -6,21 +6,13 @@ import {RippleConfig, RippleRenderer} from './ripple-renderer';
 export class RippleRef {
 
   constructor(
-    private renderer: RippleRenderer,
+    private _renderer: RippleRenderer,
     public element: HTMLElement,
     public config: RippleConfig) {
   }
 
   /** Fades out the ripple element. */
   fadeOut() {
-    let rippleIndex = this.renderer.activeRipples.indexOf(this);
-
-    // Remove the ripple reference if added to the list of active ripples.
-    if (rippleIndex !== -1) {
-      this.renderer.activeRipples.splice(rippleIndex, 1);
-    }
-
-    // Regardless of being added to the list, fade-out the ripple element.
-    this.renderer.fadeOutRipple(this.element);
+    this._renderer.fadeOutRipple(this);
   }
 }

src/lib/core/ripple/ripple-renderer.ts

@@ -114,14 +114,22 @@ export class RippleRenderer {
     return rippleRef;
   }
 
-  /** Fades out a ripple element. */
-  fadeOutRipple(ripple: HTMLElement) {
-    ripple.style.transitionDuration = `${RIPPLE_FADE_OUT_DURATION}ms`;
-    ripple.style.opacity = '0';
+  /** Fades out a ripple reference. */
+  fadeOutRipple(ripple: RippleRef) {
+    let rippleEl = ripple.element;
+    let rippleIndex = this.activeRipples.indexOf(ripple);
+
+    // Remove the ripple reference if added to the list of active ripples.
+    if (rippleIndex !== -1) {
+      this.activeRipples.splice(rippleIndex, 1);
+    }
+
+    rippleEl.style.transitionDuration = `${RIPPLE_FADE_OUT_DURATION}ms`;
+    rippleEl.style.opacity = '0';
 
     // Once the ripple faded out, the ripple can be safely removed from the DOM.
     this.runTimeoutOutsideZone(() => {
-      ripple.parentNode.removeChild(ripple);
+      rippleEl.parentNode.removeChild(rippleEl);
     }, RIPPLE_FADE_OUT_DURATION);
   }