From e6ffd49f9b17a64eec04ed8919b44c7c8b74a23b Mon Sep 17 00:00:00 2001
From: James Arroyo <jamesarroyo5@gmail.com>
Date: Mon, 6 Nov 2017 12:47:35 -0700
Subject: [PATCH 1/9] added an animation that fades the banner's body text in
 and out. Using web animations api

---
 ab-animated-banner.html | 71 ++++++++++++++++++++++++++++++++++++++++-
 1 file changed, 70 insertions(+), 1 deletion(-)

diff --git a/ab-animated-banner.html b/ab-animated-banner.html
index c7d6f6e..9b00531 100644
--- a/ab-animated-banner.html
+++ b/ab-animated-banner.html
@@ -33,7 +33,33 @@
           text-align: center;
       }
 
+      .banner .animation{
+          height: 100%;
+          display: flex;
+          flex-wrap: wrap;
+          justify-content:flex-start;
+
+      }
+
+      .animation span{
+          height: 40px;
+          margin: 13px 15px 0 0;
+      }
+
+      .animation .img{
+          height: 40px;
+          margin: 13px 15px 0 0;
+          width: 40px;
+          margin-left: 10px;
+      }
 
+      .animation .text{
+          width: 200px;
+          padding-top: 11px;
+          font-size: 14px;
+          font-weight: 800;
+          color: #525252;
+      }
 
     </style>
 
@@ -42,6 +68,10 @@
             <div class="innerLeft">
                 <h4>[[title]]</h4>
             </div>
+            <div id="bannerAnimation" class="animation">
+              <span class="img"></span>
+              <span class="text">[[bodyText]]</span>
+            </div>
         </div>
     </div>
 
@@ -53,19 +83,58 @@ <h4>[[title]]</h4>
      * @polymer
      */
     class AbAnimatedBanner extends Polymer.Element {
-      static get is() { return 'ab-animated-banner'; }
+
+      static get is() {
+        return 'ab-animated-banner';
+      }
+
       static get properties() {
         return {
           title: {
             type: String,
             value: 'Title'
           },
+          bodyText: {
+            type: String,
+            value: 'Body TExt'
+          },
           dataSet: {
             type: Array,
             value: ['Jimmy Arroyo', 'John Doe', 'Bob Smith']
           }
         };
       }
+
+      /**
+       * Cycles through an array of strings displaying them one by one
+       * with an animation
+       *
+       * @param {Array<String>} values An array of strings
+       */
+      _cycleAnimation(values) {
+        this.$.bannerAnimation.animate([
+          {transform: 'translateY(0)', opacity: 1, easing: 'ease-in'},
+          {transform: 'translateY(0)', opacity: 1},
+          {transform: 'translateY(0)', opacity: 1}, // Hack: added duplicate keyframe since web animations api doesnt allow a delay or duration on a specific keyframe in the sequence
+          {transform: 'translateY(0)', opacity: 0, easing: 'ease-out'}
+        ], {
+          duration: 4000,
+          iterations: Infinity,
+          direction: 'alternate',
+          fill: 'forwards'
+        });
+
+      }
+
+      ready() {
+        super.ready();
+
+        // Start cycle animation
+        this._cycleAnimation(['1', '2', '3', '4', '5']);
+      }
+
+
+
     }
 
     window.customElements.define(AbAnimatedBanner.is, AbAnimatedBanner);

From 28723bd1c94700869570c6c46b930c4eaeaed553 Mon Sep 17 00:00:00 2001
From: James Arroyo <jamesarroyo5@gmail.com>
Date: Tue, 7 Nov 2017 11:34:45 -0700
Subject: [PATCH 2/9] add DS_Store to the git ignore file

---
 .gitignore | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.gitignore b/.gitignore
index 8d4ae25..b13058c 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1 +1,2 @@
 bower_components
+.DS_Store

From 135bc6f0fa349be712270ae6882f3cccf95eabb8 Mon Sep 17 00:00:00 2001
From: James Arroyo <jamesarroyo5@gmail.com>
Date: Tue, 7 Nov 2017 11:37:35 -0700
Subject: [PATCH 3/9] added functionality to loop through provided strings and
 display them inside the banner with the animation

---
 ab-animated-banner.html | 59 ++++++++++++++++++++++++-----------------
 1 file changed, 34 insertions(+), 25 deletions(-)

diff --git a/ab-animated-banner.html b/ab-animated-banner.html
index 9b00531..4d03232 100644
--- a/ab-animated-banner.html
+++ b/ab-animated-banner.html
@@ -21,56 +21,48 @@
 
       }
 
-      .banner .innerLeft{
+      .banner .titleBox{
           height: 100%;
           width: 20%;
           float: left;
           background: rgba(253,202,0,1);
       }
 
-      .innerLeft h4{
+      .titleBox h4{
           font-size: 20px;
           text-align: center;
       }
 
-      .banner .animation{
+      .banner .body{
           height: 100%;
           display: flex;
           flex-wrap: wrap;
           justify-content:flex-start;
-
       }
 
-      .animation span{
+      .body span{
           height: 40px;
           margin: 13px 15px 0 0;
       }
 
-      .animation .img{
-          height: 40px;
-          margin: 13px 15px 0 0;
-          width: 40px;
-          margin-left: 10px;
-      }
-
-      .animation .text{
+      .body .text{
           width: 200px;
           padding-top: 11px;
           font-size: 14px;
           font-weight: 800;
           color: #525252;
+          margin-left: 10px;
       }
 
     </style>
 
     <div class="banner-container">
         <div class="banner">
-            <div class="innerLeft">
+            <div class="titleBox">
                 <h4>[[title]]</h4>
             </div>
-            <div id="bannerAnimation" class="animation">
-              <span class="img"></span>
-              <span class="text">[[bodyText]]</span>
+            <div id="bannerAnimation" class="body">
+              <span class="text">[[body]]</span>
             </div>
         </div>
     </div>
@@ -94,13 +86,17 @@ <h4>[[title]]</h4>
             type: String,
             value: 'Title'
           },
-          bodyText: {
+          body: {
             type: String,
-            value: 'Body TExt'
+            value: ''
           },
           dataSet: {
             type: Array,
             value: ['Jimmy Arroyo', 'John Doe', 'Bob Smith']
+          },
+          _arrayCyclePosition: {
+            type: Number,
+            value: -1
           }
         };
       }
@@ -111,26 +107,39 @@ <h4>[[title]]</h4>
        *
        * @param {Array<String>} values An array of strings
        */
-      _cycleAnimation(values) {
-        this.$.bannerAnimation.animate([
+      _cycleAnimation() {
+        // increment array and get remainder so loop is never ending.
+        this._arrayCyclePosition = (this._arrayCyclePosition + 1) % this.dataSet.length;
+
+        // update banner body with new value;
+        this.body = this.dataSet[this._arrayCyclePosition];
+
+        let bannerAnimation = this.$.bannerAnimation.animate([
           {transform: 'translateY(0)', opacity: 1, easing: 'ease-in'},
           {transform: 'translateY(0)', opacity: 1},
           {transform: 'translateY(0)', opacity: 1}, // Hack: added duplicate keyframe since web animations api doesnt allow a delay or duration on a specific keyframe in the sequence
           {transform: 'translateY(0)', opacity: 0, easing: 'ease-out'}
         ], {
           duration: 4000,
-          iterations: Infinity,
+          iterations: 1,
           direction: 'alternate',
           fill: 'forwards'
         });
 
-      }
+        var element = this;
+
+        // Continue cylcing banner.
+        bannerAnimation.onfinish = function() {
+          element._cycleAnimation();
+        };
+
+      };
 
       ready() {
         super.ready();
 
-        // Start cycle animation
-        this._cycleAnimation(['1', '2', '3', '4', '5']);
+        // Start animation cycle
+        this._cycleAnimation();
       }
 
 

From bf4dbc21610291a9c2ef88f6662acdc3b101d171 Mon Sep 17 00:00:00 2001
From: James Arroyo <jamesarroyo5@gmail.com>
Date: Tue, 7 Nov 2017 14:24:26 -0700
Subject: [PATCH 4/9] Documented and refactored some properties

---
 ab-animated-banner.html | 23 ++++++++++++++++++-----
 1 file changed, 18 insertions(+), 5 deletions(-)

diff --git a/ab-animated-banner.html b/ab-animated-banner.html
index 4d03232..ba01a75 100644
--- a/ab-animated-banner.html
+++ b/ab-animated-banner.html
@@ -82,19 +82,34 @@ <h4>[[title]]</h4>
 
       static get properties() {
         return {
+          /**
+          * The title displayed in the left side of the banner.
+          */
           title: {
             type: String,
             value: 'Title'
           },
+
+          /**
+          * The body text of the banner.
+          */
           body: {
             type: String,
             value: ''
           },
+
+          /**
+          * Set the array of strings to be displayed in the body of the banner.
+          */
           dataSet: {
             type: Array,
             value: ['Jimmy Arroyo', 'John Doe', 'Bob Smith']
           },
-          _arrayCyclePosition: {
+
+          /**
+          * The dataSet's position in which the banner's body is displaying.
+          */
+          _dataSetPosition: {
             type: Number,
             value: -1
           }
@@ -109,10 +124,10 @@ <h4>[[title]]</h4>
        */
       _cycleAnimation() {
         // increment array and get remainder so loop is never ending.
-        this._arrayCyclePosition = (this._arrayCyclePosition + 1) % this.dataSet.length;
+        this._dataSetPosition = (this._dataSetPosition + 1) % this.dataSet.length;
 
         // update banner body with new value;
-        this.body = this.dataSet[this._arrayCyclePosition];
+        this.body = this.dataSet[this._dataSetPosition];
 
         let bannerAnimation = this.$.bannerAnimation.animate([
           {transform: 'translateY(0)', opacity: 1, easing: 'ease-in'},
@@ -142,8 +157,6 @@ <h4>[[title]]</h4>
         this._cycleAnimation();
       }
 
-
-
     }
 
     window.customElements.define(AbAnimatedBanner.is, AbAnimatedBanner);

From f6ddc39483f82a821bdd7074e05b4987e98a2efa Mon Sep 17 00:00:00 2001
From: James Arroyo <jamesarroyo5@gmail.com>
Date: Tue, 7 Nov 2017 16:16:05 -0700
Subject: [PATCH 5/9] updated README with description, installation and getting
 started goodies.

---
 README.md | 45 ++++++++++++++++++++++++++++-----------------
 1 file changed, 28 insertions(+), 17 deletions(-)

diff --git a/README.md b/README.md
index ffe25b7..772f55f 100644
--- a/README.md
+++ b/README.md
@@ -1,33 +1,44 @@
 # \<ab-animated-banner\>
 
+A Polymer web component.
 
+The `ab-animated-banner` is a polymer element which displays a simple banner that cycles through an array of strings.
 
-## Install the Polymer-CLI
+## Installation
 
-First, make sure you have the [Polymer CLI](https://www.npmjs.com/package/polymer-cli) installed. Then run `polymer serve` to serve your application locally.
+```sh
+$ bower install -S ab-animated-banner
+```
 
-## Viewing Your Application
+## Getting Started
 
-```
-$ polymer serve
+Import the element.
+
+```html
+<link rel="import" href="/bower_components/ab-animated-banner/ab-animated-banner.html">
 ```
 
-## Building Your Application
+Add the element.
 
-```
-$ polymer build
+```html
+<ab-animated-banner> </ab-animated-banner>
 ```
 
-This will create builds of your application in the `build/` directory, optimized to be served in production. You can then serve the built versions by giving `polymer serve` a folder to serve from:
+## Contributing
+1. Fork it!
+2. Create your feature branch: `git checkout -b my-new-feature`
+3. Commit your changes: `git commit -am 'Add some feature'`
+4. Push to the branch: `git push origin my-new-feature`
+5. Submit a pull request :D
 
-```
-$ polymer serve build/default
-```
+## License
 
-## Running Tests
+The MIT License (MIT)
 
-```
-$ polymer test
-```
+Copyright (c) 2017 Jimmy Arroyo
+
+Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.
 
-Your application is already set up to be tested via [web-component-tester](https://github.com/Polymer/web-component-tester). Run `polymer test` to run your application's test suite locally.
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

From a164e74fdd4169b4fc1fb1d25cf2a9be3503153f Mon Sep 17 00:00:00 2001
From: James Arroyo <jamesarroyo5@gmail.com>
Date: Tue, 7 Nov 2017 16:35:19 -0700
Subject: [PATCH 6/9] updated README.md informing users to include
 webcomponents-lite for browsers that don't support web components

---
 README.md | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/README.md b/README.md
index 772f55f..6b3fc3f 100644
--- a/README.md
+++ b/README.md
@@ -12,9 +12,11 @@ $ bower install -S ab-animated-banner
 
 ## Getting Started
 
-Import the element.
+Include the [webcomponents.js](http://webcomponents.org/polyfills/) "lite" polyfill (for browsers who don't natively support web components), then import `ab-animated-banner.html`:
+
 
 ```html
+<script src="../bower_components/webcomponentsjs/webcomponents-lite.js"></script>
 <link rel="import" href="/bower_components/ab-animated-banner/ab-animated-banner.html">
 ```
 

From f8bba1fecb995423347d62163154cf7f697f01f5 Mon Sep 17 00:00:00 2001
From: James Arroyo <jamesarroyo5@gmail.com>
Date: Thu, 9 Nov 2017 14:11:31 -0700
Subject: [PATCH 7/9] Added data binding attribute 'banner-options' to the
 element which allows for customization of some of the elements properties

---
 ab-animated-banner.html | 42 ++++++++++++++++++++++++++++++-----------
 1 file changed, 31 insertions(+), 11 deletions(-)

diff --git a/ab-animated-banner.html b/ab-animated-banner.html
index ba01a75..11832d5 100644
--- a/ab-animated-banner.html
+++ b/ab-animated-banner.html
@@ -21,14 +21,14 @@
 
       }
 
-      .banner .titleBox{
+      .banner .header{
           height: 100%;
           width: 20%;
           float: left;
           background: rgba(253,202,0,1);
       }
 
-      .titleBox h4{
+      .header h4{
           font-size: 20px;
           text-align: center;
       }
@@ -58,8 +58,8 @@
 
     <div class="banner-container">
         <div class="banner">
-            <div class="titleBox">
-                <h4>[[title]]</h4>
+            <div class="header">
+                <h4>[[headerTitle]]</h4>
             </div>
             <div id="bannerAnimation" class="body">
               <span class="text">[[body]]</span>
@@ -82,12 +82,19 @@ <h4>[[title]]</h4>
 
       static get properties() {
         return {
+          /**
+          * Holds all attributes that can be passed to the banner.
+          */
+          bannerOptions: {
+            type: Object,
+            value: {}
+          },
           /**
           * The title displayed in the left side of the banner.
           */
-          title: {
+          headerTitle: {
             type: String,
-            value: 'Title'
+            value: ''
           },
 
           /**
@@ -101,9 +108,9 @@ <h4>[[title]]</h4>
           /**
           * Set the array of strings to be displayed in the body of the banner.
           */
-          dataSet: {
+          bodyArray: {
             type: Array,
-            value: ['Jimmy Arroyo', 'John Doe', 'Bob Smith']
+            value: []
           },
 
           /**
@@ -120,14 +127,13 @@ <h4>[[title]]</h4>
        * Cycles through an array of strings displaying them one by one
        * with an animation
        *
-       * @param {Array<String>} values An array of strings
        */
       _cycleAnimation() {
         // increment array and get remainder so loop is never ending.
-        this._dataSetPosition = (this._dataSetPosition + 1) % this.dataSet.length;
+        this._dataSetPosition = (this._dataSetPosition + 1) % this.bodyArray.length;
 
         // update banner body with new value;
-        this.body = this.dataSet[this._dataSetPosition];
+        this.body = this.bodyArray[this._dataSetPosition];
 
         let bannerAnimation = this.$.bannerAnimation.animate([
           {transform: 'translateY(0)', opacity: 1, easing: 'ease-in'},
@@ -150,9 +156,23 @@ <h4>[[title]]</h4>
 
       };
 
+      /**
+       * Deconstructs the bannerOptions and assigns them to the element.
+       *
+       * @param {Object} bannerOptions Options used to customize the animatedBanner element.
+       */
+      assignProperties(bannerOptions) {
+        // TODO: Check that values are of correct type and handle errors
+        this.bodyArray = bannerOptions.bodyArray;
+        this.headerTitle = bannerOptions.headerTitle;
+      };
+
       ready() {
         super.ready();
 
+        // Bind bannerOptions to bannerElement
+        this.assignProperties(this.bannerOptions);
+
         // Start animation cycle
         this._cycleAnimation();
       }

From 433b7e5fe6d8dbe8ada297b51d6e392dc7f0ab26 Mon Sep 17 00:00:00 2001
From: James Arroyo <jamesarroyo5@gmail.com>
Date: Thu, 9 Nov 2017 14:12:09 -0700
Subject: [PATCH 8/9] updated the demo to pass data to the banner so it
 displays data

---
 demo/index.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/demo/index.html b/demo/index.html
index 45110bc..37b723d 100644
--- a/demo/index.html
+++ b/demo/index.html
@@ -10,7 +10,7 @@
 </head>
 
 <body style="background-color: #efefef">
-  <ab-animated-banner></ab-animated-banner>
+  <ab-animated-banner banner-options='{"bodyArray": ["Jimmy", "Jane", "John"], "headerTitle": "Names"}'></ab-animated-banner>
 </body>
 
 </html>

From f3da5a8f3b84a6d080fd34cfb8d610afcf48f7ea Mon Sep 17 00:00:00 2001
From: James Arroyo <jamesarroyo5@gmail.com>
Date: Thu, 9 Nov 2017 14:17:36 -0700
Subject: [PATCH 9/9] changed 'body' property to '_body' to express that it is
 protected and not for public use

---
 ab-animated-banner.html | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/ab-animated-banner.html b/ab-animated-banner.html
index 11832d5..622b4ba 100644
--- a/ab-animated-banner.html
+++ b/ab-animated-banner.html
@@ -62,7 +62,7 @@
                 <h4>[[headerTitle]]</h4>
             </div>
             <div id="bannerAnimation" class="body">
-              <span class="text">[[body]]</span>
+              <span class="text">[[_body]]</span>
             </div>
         </div>
     </div>
@@ -97,14 +97,6 @@ <h4>[[headerTitle]]</h4>
             value: ''
           },
 
-          /**
-          * The body text of the banner.
-          */
-          body: {
-            type: String,
-            value: ''
-          },
-
           /**
           * Set the array of strings to be displayed in the body of the banner.
           */
@@ -113,6 +105,14 @@ <h4>[[headerTitle]]</h4>
             value: []
           },
 
+          /**
+          * The body text of the banner.
+          */
+          _body: {
+            type: String,
+            value: ''
+          },
+
           /**
           * The dataSet's position in which the banner's body is displaying.
           */
@@ -133,7 +133,7 @@ <h4>[[headerTitle]]</h4>
         this._dataSetPosition = (this._dataSetPosition + 1) % this.bodyArray.length;
 
         // update banner body with new value;
-        this.body = this.bodyArray[this._dataSetPosition];
+        this._body = this.bodyArray[this._dataSetPosition];
 
         let bannerAnimation = this.$.bannerAnimation.animate([
           {transform: 'translateY(0)', opacity: 1, easing: 'ease-in'},