From abfd6ca3e177de12889a7c019d14868c94927b96 Mon Sep 17 00:00:00 2001
From: Chris Oyler <chris@zurb.com>
Date: Mon, 1 Feb 2016 17:09:15 -0800
Subject: [PATCH] updates docs for dropdowns

---
 docs/pages/dropdown.md    |  2 +-
 js/foundation.dropdown.js | 20 ++++++++------------
 2 files changed, 9 insertions(+), 13 deletions(-)

diff --git a/docs/pages/dropdown.md b/docs/pages/dropdown.md
index 73a7ec88e..6c93a64c8 100644
--- a/docs/pages/dropdown.md
+++ b/docs/pages/dropdown.md
@@ -37,7 +37,7 @@ To create the dropdown trigger, add `data-toggle` to a `<button>`. The value of
 
 
 <button class="button" type="button" data-toggle="example-dropdown-1">Hoverable Dropdown</button>
-<div class="dropdown-pane" id="example-dropdown-1" data-dropdown data-hover="true">
+<div class="dropdown-pane" id="example-dropdown-1" data-dropdown data-hover="true" data-hover-pane="true">
   Just some junk that needs to be said. Or not. Your choice.
 </div>
 ```
diff --git a/js/foundation.dropdown.js b/js/foundation.dropdown.js
index b87c8cc43..22e388cf8 100644
--- a/js/foundation.dropdown.js
+++ b/js/foundation.dropdown.js
@@ -31,7 +31,7 @@
 
   Dropdown.defaults = {
     /**
-     * Amount of time to delay opening a submenu on hover event.
+     * Amount of time, in milliseconds to delay opening a submenu on hover event.
      * @option
      * @example 250
      */
@@ -43,19 +43,19 @@
      */
     hover: false,
     /**
-     * Don't close dropdown when hovering over dropdown pane
+     * Prevent the dropdown pane from closing while hovering over it
      * @option
-     * @example true
+     * @example false
      */
     hoverPane: false,
     /**
-     * Number of pixels between the dropdown pane and the triggering element on open.
+     * Number of pixels between the dropdown pane and the triggering element on open, on the Y-axis
      * @option
      * @example 1
      */
     vOffset: 1,
     /**
-     * Number of pixels between the dropdown pane and the triggering element on open.
+     * Number of pixels between the dropdown pane and the triggering element on open, on the X-axis.
      * @option
      * @example 1
      */
@@ -67,13 +67,13 @@
      */
     positionClass: '',
     /**
-     * Allow the plugin to trap focus to the dropdown pane if opened with keyboard commands.
+     * Allow the plugin to trap focus to the dropdown pane if opened with keyboard commands. Useful if your dropdown pane contains form elements.
      * @option
      * @example false
      */
     trapFocus: false,
     /**
-     * Allow the plugin to set focus to the first focusable element within the pane, regardless of method of opening.
+     * Allow the plugin to set focus to the first focusable element within the pane, regardless of method of opening. Will cause the page to scroll your dropdown pane into view if opening via JS.
      * @option
      * @example true
      */
@@ -100,7 +100,7 @@
       'data-yeti-box': $id,
       'aria-haspopup': true,
       'aria-expanded': false
-      // 'data-resize': $id
+
     });
 
     this.options.positionClass = this.getPositionClass();
@@ -331,9 +331,6 @@
      * @event Dropdown#show
      */
      this.$element.trigger('show.zf.dropdown', [this.$element]);
-    //why does this not work correctly for this plugin?
-    // Foundation.reflow(this.$element, 'dropdown');
-    // Foundation._reflow(this.$element.attr('data-dropdown'));
   };
 
   /**
@@ -363,7 +360,6 @@
       this.usedPositions.length = 0;
     }
     this.$element.trigger('hide.zf.dropdown', [this.$element]);
-    // Foundation.reflow(this.$element, 'dropdown');
   };
   /**
    * Toggles the dropdown pane's visibility.
-- 
2.47.2