From 47d2f1893f39352f30a5a6ad535eaaf369b654d9 Mon Sep 17 00:00:00 2001
From: Zander Otavka <AlexanderOtavka@users.noreply.github.com>
Date: Sat, 12 Aug 2017 15:54:50 -0700
Subject: [PATCH] Fix mixed up type definitions for Dropdown

`valueKey` and `labelKey` were listed as theme props.  What's worse, `valueKey` was marked as required, when in fact it should not be present at all on the theme.
---
 components/dropdown/Dropdown.d.ts | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/components/dropdown/Dropdown.d.ts b/components/dropdown/Dropdown.d.ts
index 4d14eeb7..3ed67817 100644
--- a/components/dropdown/Dropdown.d.ts
+++ b/components/dropdown/Dropdown.d.ts
@@ -30,10 +30,6 @@ export interface DropdownTheme {
    * Used for the the label element.
    */
   label?: string;
-  /**
-   * Used for setting the label from source
-   */
-  labelKey?: string;
   /**
    * Used when dropdown has required attribute.
    */
@@ -58,10 +54,6 @@ export interface DropdownTheme {
    * Used for the list of values.
    */
   values?: string;
-  /**
-   * Used for setting the value from source
-   */
-  valueKey: string;
 }
 
 export interface DropdownProps extends ReactToolbox.Props {
@@ -88,6 +80,10 @@ export interface DropdownProps extends ReactToolbox.Props {
    * The text string to use for the floating label element.
    */
   label?: string;
+  /**
+   * Used for setting the label from source
+   */
+  labelKey?: string;
   /**
    * Name for the input field.
    */
@@ -104,6 +100,11 @@ export interface DropdownProps extends ReactToolbox.Props {
    * Callback function that is fired when the component is focused.
    */
   onFocus?: Function;
+  /**
+   * If true, the dropdown has a required attribute.
+   * @default false
+   */
+  required?: boolean;
   /**
    * Array of data objects with the data to represent in the dropdown.
    */
@@ -121,10 +122,9 @@ export interface DropdownProps extends ReactToolbox.Props {
    */
   value?: string | number;
   /**
-   * If true, the dropdown has a required attribute.
-   * @default false
+   * Used for setting the value from source
    */
-  required?: boolean;
+  valueKey?: string;
 }
 
 export class Dropdown extends React.Component<DropdownProps, {}> { }