From: Chris Oyler <chris@zurb.com>
Date: Tue, 15 Dec 2015 18:57:16 +0000 (-0800)
Subject: cleans up some unnecessary code in abide
X-Git-Tag: v6.1.0~4^2^2~18
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=cfd70b47e7b50d33ae74f3e17453a8f3b422d51b;p=thirdparty%2Ffoundation%2Ffoundation-sites.git

cleans up some unnecessary code in abide
---

diff --git a/docs/pages/abide.md b/docs/pages/abide.md
index e5b119df5..81b55f244 100644
--- a/docs/pages/abide.md
+++ b/docs/pages/abide.md
@@ -19,8 +19,17 @@ These input types create a text field: `text`, `date`, `datetime`, `datetime-loc
   </div>
   <div class="row">
     <div class="small-12 columns">
-      <label>Input Label
-        <input type="text" placeholder=".small-12.columns" aria-describedby="exampleHelpText" required pattern="number">
+      <label>Number Required
+        <input type="text" placeholder="1234" aria-describedby="exampleHelpText" required pattern="number">
+        <span class="form-error">
+          Yo, you had better fill this out.
+        </span>
+      </label>
+      <p class="help-text" id="exampleHelpText">Here's how you use this input field!</p>
+    </div>
+    <div class="small-12 columns">
+      <label>Password Required
+        <input type="password" placeholder="yeti4preZ" aria-describedby="exampleHelpText" required pattern="password">
         <span class="form-error">
           Yo, you had better fill this out.
         </span>
@@ -30,12 +39,12 @@ These input types create a text field: `text`, `date`, `datetime`, `datetime-loc
   </div>
   <div class="row">
     <div class="medium-6 columns">
-      <label>Input Label
-        <input type="text" placeholder=".medium-6.columns" pattern="url">
+      <label>URL Required
+        <input type="text" placeholder="www.google.com" pattern="url">
       </label>
     </div>
     <div class="medium-6 columns">
-      <label>Input Label
+      <label>European Cars, Choose One
         <select id="select" required>
           <option value=""></option>
           <option value="volvo">Volvo</option>
diff --git a/js/foundation.abide.js b/js/foundation.abide.js
index ccf09a593..38dac44ae 100644
--- a/js/foundation.abide.js
+++ b/js/foundation.abide.js
@@ -97,6 +97,7 @@
       .find('input, textarea, select')
         .off('.abide')
         .on('blur.fndtn.abide change.fndtn.abide', function (e) {
+          console.log(e);
           if (self.options.validateOn === 'fieldChange') {
             self.validateInput($(e.target), self.$element);
           }
@@ -125,17 +126,11 @@
    * @param {Object} element - jQuery object to check for required attribute
    * @returns {Boolean} Boolean value depends on whether or not attribute is checked or empty
    */
+   var counter = 0;
   Abide.prototype.requiredCheck = function($el) {
     switch ($el[0].type) {
       case 'text':
-        if ($el.attr('required') && !$el.val()) {
-          // requirement check does not pass
-          return false;
-        } else {
-          return true;
-        }
-        break;
-        case 'password':
+      case 'password':
         if ($el.attr('required') && !$el.val()) {
           // requirement check does not pass
           return false;
@@ -144,13 +139,14 @@
         }
         break;
       case 'checkbox':
-        if ($el.attr('required') && !$el.is(':checked')) {
-          return false;
-        } else {
-          return true;
-        }
-        break;
+        // if ($el.attr('required') && !$el.is(':checked')) {
+        //   return false;
+        // } else {
+        //   return true;
+        // }
+        // break;
       case 'radio':
+      console.log($el[0].type, counter++);
         if ($el.attr('required') && !$el.is(':checked')) {
           return false;
         } else {