Select knob key/value ordering

addons/knobs/README.md

@@ -247,6 +247,25 @@ const value = select(label, options, defaultValue);
 
 > You can also provide options as an array like this: `['red', 'blue', 'yellow']`
 
+### selectV2
+
+In v4 this will replace `select`. The value from the select now uses the values from the `options` object.
+
+```js
+import { selectV2 } from '@storybook/addon-knobs';
+
+const label = 'Colors';
+const options = {
+  Red: 'red',
+  Blue: 'blue',
+  Yellow: 'yellow',
+  Rainbow: ['red', 'orange', 'etc'],
+  None: null,
+};
+const defaultValue = 'Red';
+
+const value = selectV2(label, options, defaultValue)
+
 ### date
 
 Allow you to get date (and time) from the user.

addons/knobs/src/angular/index.js

@@ -12,11 +12,12 @@ import {
   array,
   date,
   select,
+  selectV2,
   button,
   manager,
 } from '../base';
 
-export { knob, text, boolean, number, color, object, array, date, select, button };
+export { knob, text, boolean, number, color, object, array, date, select, selectV2, button };
 
 export const angularHandler = (channel, knobStore) => getStory => context =>
   prepareComponent({ getStory, context, channel, knobStore });

addons/knobs/src/base.js

@@ -1,3 +1,4 @@
+import deprecate from 'util-deprecate';
 import KnobManager from './KnobManager';
 
 export const manager = new KnobManager();
@@ -46,7 +47,14 @@ export function object(name, value) {
 }
 
 export function select(name, options, value) {
-  return manager.knob(name, { type: 'select', options, value });
+  return deprecate(
+    manager.knob(name, { type: 'select', options, value }),
+    'in v4 keys/values of the options argument are reversed'
+  );
+}
+
+export function selectV2(name, options, value) {
+  return manager.knob(name, { type: 'select', selectV2: true, options, value });
 }
 
 export function array(name, value, separator = ',') {

addons/knobs/src/components/__tests__/Select.js

@@ -0,0 +1,40 @@
+import React from 'react';
+import { shallow } from 'enzyme'; // eslint-disable-line
+import SelectType from '../types/Select';
+
+describe('Select', () => {
+  let knob;
+
+  beforeEach(() => {
+    knob = {
+      name: 'Colors',
+      value: '#00ff00',
+      options: {
+        Green: '#00ff00',
+        Red: '#ff0000',
+      },
+    };
+  });
+
+  it('displays value', () => {
+    const wrapper = shallow(<SelectType knob={knob} />);
+
+    const green = wrapper.find('option').first();
+    expect(green.text()).toEqual('#00ff00');
+    expect(green.prop('value')).toEqual('Green');
+  });
+
+  describe('selectV2', () => {
+    beforeEach(() => {
+      knob.selectV2 = true;
+    });
+
+    it('correctly maps option keys and values', () => {
+      const wrapper = shallow(<SelectType knob={knob} />);
+
+      const green = wrapper.find('option').first();
+      expect(green.text()).toEqual('Green');
+      expect(green.prop('value')).toEqual('#00ff00');
+    });
+  });
+});

addons/knobs/src/components/types/Select.js

@@ -18,20 +18,27 @@ const styles = {
 };
 
 class SelectType extends React.Component {
-  _makeOpt(key, val) {
+  _makeOpt(key, value, selectV2) {
     const opts = {
       key,
       value: key,
     };
 
-    return <option {...opts}>{val}</option>;
+    let display = value;
+
+    if (selectV2) {
+      opts.value = value;
+      display = key;
+    }
+
+    return <option {...opts}>{display}</option>;
   }
-  _options(values) {
+  _options({ options, selectV2 }) {
     let data = [];
-    if (Array.isArray(values)) {
-      data = values.map(val => this._makeOpt(val, val));
+    if (Array.isArray(options)) {
+      data = options.map(val => this._makeOpt(val, val));
     } else {
-      data = Object.keys(values).map(key => this._makeOpt(key, values[key]));
+      data = Object.keys(options).map(key => this._makeOpt(key, options[key], selectV2));
     }
 
     return data;
@@ -49,7 +56,7 @@ class SelectType extends React.Component {
         value={knob.value}
         onChange={e => onChange(e.target.value)}
       >
-        {this._options(knob.options)}
+        {this._options(knob)}
       </select>
     );
   }
@@ -64,6 +71,8 @@ SelectType.propTypes = {
   knob: PropTypes.shape({
     name: PropTypes.string,
     value: PropTypes.string,
+    options: PropTypes.oneOfType([PropTypes.array, PropTypes.object]),
+    selectV2: PropTypes.bool,
   }),
   onChange: PropTypes.func,
 };

addons/knobs/src/index.js

@@ -16,13 +16,16 @@ import {
   number,
   object,
   select,
+  selectV2,
   text,
 } from './base';
 
-export { knob, text, boolean, number, color, object, array, date, button, select };
+export { knob, text, boolean, number, color, object, array, date, button, select, selectV2 };
 
-deprecate(() => {},
-'Using @storybook/addon-knobs directly is discouraged, please use @storybook/addon-knobs/{{framework}}');
+deprecate(
+  () => {},
+  'Using @storybook/addon-knobs directly is discouraged, please use @storybook/addon-knobs/{{framework}}'
+);
 
 // generic higher-order component decorator for all platforms - usage is discouraged
 // This file Should be removed with 4.0 release

addons/knobs/src/react/index.js

@@ -13,11 +13,12 @@ import {
   array,
   date,
   select,
+  selectV2,
   button,
   manager,
 } from '../base';
 
-export { knob, text, boolean, number, color, object, array, date, select, button };
+export { knob, text, boolean, number, color, object, array, date, select, selectV2, button };
 
 export const reactHandler = (channel, knobStore) => getStory => context => {
   const initialContent = getStory(context);

addons/knobs/src/vue/index.js

@@ -10,11 +10,12 @@ import {
   array,
   date,
   select,
+  selectV2,
   button,
   manager,
 } from '../base';
 
-export { knob, text, boolean, number, color, object, array, date, select, button };
+export { knob, text, boolean, number, color, object, array, date, select, selectV2, button };
 
 export const vueHandler = (channel, knobStore) => getStory => context => ({
   data() {