From af47c2457afdd5d352c496179b9832cea2be477b Mon Sep 17 00:00:00 2001
From: Randall Morey <randall@randallmorey.com>
Date: Tue, 28 Apr 2020 11:46:05 -0400
Subject: [PATCH] =?UTF-8?q?style:=20=F0=9F=92=84=20add=20button=20componen?=
 =?UTF-8?q?t=20with=20styles,=20tests,=20and=20story?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 addons/rose/addon/components/rose/button.js   | 13 +++
 addons/rose/addon/styles/rose/_index.scss     |  5 +-
 .../addon/styles/rose/components/_button.scss | 56 +++++++++++++
 .../addon/styles/rose/components/_index.scss  |  1 +
 .../addon/styles/rose/utilities/_button.scss  | 80 +++++++++++++++++++
 .../addon/styles/rose/utilities/_index.scss   |  1 +
 .../templates/components/rose/button.hbs      |  8 ++
 addons/rose/app/components/rose/button.js     |  1 +
 addons/rose/app/templates/index.hbs           | 10 +++
 addons/rose/stories/button.stories.js         | 19 -----
 addons/rose/stories/rose/button.stories.js    | 36 +++++++++
 .../components/rose/button-test.js            | 35 ++++++++
 12 files changed, 242 insertions(+), 23 deletions(-)
 create mode 100644 addons/rose/addon/components/rose/button.js
 create mode 100644 addons/rose/addon/styles/rose/components/_button.scss
 create mode 100644 addons/rose/addon/styles/rose/components/_index.scss
 create mode 100644 addons/rose/addon/styles/rose/utilities/_button.scss
 create mode 100644 addons/rose/addon/styles/rose/utilities/_index.scss
 create mode 100644 addons/rose/addon/templates/components/rose/button.hbs
 create mode 100644 addons/rose/app/components/rose/button.js
 delete mode 100644 addons/rose/stories/button.stories.js
 create mode 100644 addons/rose/stories/rose/button.stories.js
 create mode 100644 addons/rose/tests/integration/components/rose/button-test.js

diff --git a/addons/rose/addon/components/rose/button.js b/addons/rose/addon/components/rose/button.js
new file mode 100644
index 000000000..3b11f0a24
--- /dev/null
+++ b/addons/rose/addon/components/rose/button.js
@@ -0,0 +1,13 @@
+import Component from '@ember/component';
+import layout from '../../templates/components/rose/button';
+
+/**
+ * A simple HTML `<button>` element.
+ *
+ * @example
+ *  <Rose::Button @submit={{true}} @style="primary">Save</Rose::Button>
+ *  <Rose::Button @disabled={{true}} @style="secondary">Disabled</Rose::Button>
+ */
+export default Component.extend({
+  layout,
+});
diff --git a/addons/rose/addon/styles/rose/_index.scss b/addons/rose/addon/styles/rose/_index.scss
index 1353b5513..ab921a668 100644
--- a/addons/rose/addon/styles/rose/_index.scss
+++ b/addons/rose/addon/styles/rose/_index.scss
@@ -1,6 +1,3 @@
 @import 'reset';
 @import 'variables';
-
-html {
-  background: pink;
-}
+@import 'components';
diff --git a/addons/rose/addon/styles/rose/components/_button.scss b/addons/rose/addon/styles/rose/components/_button.scss
new file mode 100644
index 000000000..9e9909dd7
--- /dev/null
+++ b/addons/rose/addon/styles/rose/components/_button.scss
@@ -0,0 +1,56 @@
+@use '../utilities/button';
+
+@include button.button-style(
+  'primary',
+  (
+    color: var(--white),
+    background-color: var(--info),
+    border-color: var(--info-starker-2),
+    box-shadow: 0px 4px 4px rgba(var(--stark-components), var(--opacity-2)),
+    text-shadow: 0px 1px 1px rgba(var(--black-components), var(--opacity-2)),
+  ),
+  (
+    background-color: var(--info-subtler-1),
+    box-shadow: (
+      0px 4px 4px rgba(var(--stark-components), var(--opacity-3)),
+      0px 4px 8px rgba(var(--stark-components), var(--opacity-4)),
+    ),
+  ),
+  (
+    background-color: var(--info-starker-1),
+    box-shadow: none,
+  ),
+  (
+    opacity: var(--opacity-1),
+    box-shadow: none,
+  )
+);
+
+@include button.button-style(
+  'secondary',
+  (
+    color: var(--ui-gray-starker-3),
+    background-color: var(--ui-gray-subtler-5),
+    border-color: var(--ui-gray-subtler-2),
+    box-shadow: 0px 4px 4px rgba(var(--stark-components), var(--opacity-2)),
+  ),
+  (
+    color: var(--ui-gray-starker-4),
+    background-color: var(--subtle),
+    border-color: var(--ui-gray-starker-1),
+    box-shadow: (
+      0px 4px 4px rgba(var(--stark-components), var(--opacity-3)),
+      0px 4px 8px rgba(var(--stark-components), var(--opacity-4)),
+    ),
+  ),
+  (
+    color: var(--ui-gray-starker-4),
+    background-color: var(--ui-gray-subtler-3),
+    border-color: var(--ui-gray-starker-1),
+    box-shadow: none,
+  ),
+  (
+    opacity: var(--opacity-1),
+    box-shadow: none,
+  )
+);
diff --git a/addons/rose/addon/styles/rose/components/_index.scss b/addons/rose/addon/styles/rose/components/_index.scss
new file mode 100644
index 000000000..2d4322481
--- /dev/null
+++ b/addons/rose/addon/styles/rose/components/_index.scss
@@ -0,0 +1 @@
+@import 'button';
diff --git a/addons/rose/addon/styles/rose/utilities/_button.scss b/addons/rose/addon/styles/rose/utilities/_button.scss
new file mode 100644
index 000000000..b27db5a4e
--- /dev/null
+++ b/addons/rose/addon/styles/rose/utilities/_button.scss
@@ -0,0 +1,80 @@
+@use 'sass:map';
+
+@mixin button {
+  display: flex;
+  flex-direction: row;
+  padding: 5px 24px;
+  line-height: 1.846; // targeting 24px
+  border-radius: 2px;
+  border: 1px solid;
+  font-size: 0.8125rem;
+  font-weight: 600;
+}
+
+@mixin button-style($name, $default, $hover: &, $active: &, $disabled: &) {
+  .rose-button-#{$name} {
+    @include button;
+    border-color: map.get($default, border-color);
+    color: map.get($default, color);
+    background-color: map.get($default, background-color);
+    @if map.get($default, 'box-shadow') {
+      box-shadow: map.get($default, 'box-shadow');
+    }
+    @if map.get($default, 'text-shadow') {
+      text-shadow: map.get($default, 'text-shadow');
+    }
+
+    &:not(:disabled) {
+      @if $hover {
+        &:hover {
+          @if map.get($hover, 'border-color') {
+            border-color: map.get($hover, 'border-color');
+          }
+          @if map.get($hover, 'color') {
+            color: map.get($hover, 'color');
+          }
+          @if map.get($hover, 'background-color') {
+            background-color: map.get($hover, 'background-color');
+          }
+          @if map.get($hover, 'box-shadow') {
+            box-shadow: map.get($hover, 'box-shadow');
+          }
+          @if map.get($hover, 'text-shadow') {
+            text-shadow: map.get($hover, 'text-shadow');
+          }
+        }
+      }
+      @if $active {
+        &:active {
+          @if map.get($active, 'border-color') {
+            border-color: map.get($active, 'border-color');
+          }
+          @if map.get($active, 'color') {
+            color: map.get($active, 'color');
+          }
+          @if map.get($active, 'background-color') {
+            background-color: map.get($active, 'background-color');
+          }
+          @if map.get($active, 'box-shadow') {
+            box-shadow: map.get($active, 'box-shadow');
+          }
+          @if map.get($active, 'text-shadow') {
+            text-shadow: map.get($active, 'text-shadow');
+          }
+        }
+      }
+    }
+
+    @if $disabled {
+      &:disabled {
+        cursor: not-allowed;
+        @if map.get($disabled, 'opacity') {
+          opacity: map.get($disabled, 'opacity');
+        }
+        @if map.get($disabled, 'box-shadow') {
+          box-shadow: map.get($disabled, 'box-shadow');
+        }
+      }
+    }
+  }
+}
diff --git a/addons/rose/addon/styles/rose/utilities/_index.scss b/addons/rose/addon/styles/rose/utilities/_index.scss
new file mode 100644
index 000000000..502603456
--- /dev/null
+++ b/addons/rose/addon/styles/rose/utilities/_index.scss
@@ -0,0 +1 @@
+@forward 'button';
diff --git a/addons/rose/addon/templates/components/rose/button.hbs b/addons/rose/addon/templates/components/rose/button.hbs
new file mode 100644
index 000000000..6256543d0
--- /dev/null
+++ b/addons/rose/addon/templates/components/rose/button.hbs
@@ -0,0 +1,8 @@
+<button
+  ...attributes
+  type={{if @submit "submit" "button"}}
+  disabled={{if @disabled "disabled"}}
+  class={{if  @style (concat "rose-button-" @style)}}
+  >
+  {{yield}}
+</button>
diff --git a/addons/rose/app/components/rose/button.js b/addons/rose/app/components/rose/button.js
new file mode 100644
index 000000000..b10469216
--- /dev/null
+++ b/addons/rose/app/components/rose/button.js
@@ -0,0 +1 @@
+export { default } from 'rose/components/rose/button';
diff --git a/addons/rose/app/templates/index.hbs b/addons/rose/app/templates/index.hbs
index 122889dbb..64fb78428 100644
--- a/addons/rose/app/templates/index.hbs
+++ b/addons/rose/app/templates/index.hbs
@@ -1 +1,11 @@
 Dummy app index template
+
+<Rose::Button @style="primary">Primary</Rose::Button>
+<br>
+<Rose::Button @style="primary" @disabled={{true}}>Primary</Rose::Button>
+<br>
+
+<Rose::Button @style="secondary">Secondary</Rose::Button>
+<br>
+<Rose::Button @style="secondary" @disabled={{true}}>Secondary</Rose::Button>
+<br>
diff --git a/addons/rose/stories/button.stories.js b/addons/rose/stories/button.stories.js
deleted file mode 100644
index b49f9c049..000000000
--- a/addons/rose/stories/button.stories.js
+++ /dev/null
@@ -1,19 +0,0 @@
-import { hbs } from 'ember-cli-htmlbars';
-import { action } from '@storybook/addon-actions';
-import { text /*withKnobs, boolean, select */ } from '@storybook/addon-knobs';
-
-export default {
-  title: 'Button',
-};
-
-export const Normal = () => ({
-  template: hbs`
-    <button {{action onClick}}>
-      {{text}}
-    </button>
-  `,
-  context: {
-    text: text('text', 'Button Text'),
-    onClick: action('clicked'),
-  },
-});
diff --git a/addons/rose/stories/rose/button.stories.js b/addons/rose/stories/rose/button.stories.js
new file mode 100644
index 000000000..c74e67b40
--- /dev/null
+++ b/addons/rose/stories/rose/button.stories.js
@@ -0,0 +1,36 @@
+import { hbs } from 'ember-cli-htmlbars';
+import { action } from '@storybook/addon-actions';
+import { text, boolean, select } from '@storybook/addon-knobs';
+
+export default {
+  title: 'Rose::Button',
+};
+
+export const Normal = () => ({
+  template: hbs`
+    <Rose::Button
+      @submit={{submit}}
+      @disabled={{disabled}}
+      @style={{style}}
+      {{action onClick}}
+      >
+      {{text}}
+    </Rose::Button>
+  `,
+  context: {
+    text: text('text', 'Button 🌈'),
+    submit: boolean('submit', false),
+    disabled: boolean('disabled', false),
+    style: select(
+      'style',
+      {
+        primary: 'primary',
+        secondary: 'secondary',
+        warning: 'warning',
+        ghost: 'ghost',
+      },
+      'primary'
+    ),
+    onClick: action('clicked'),
+  },
+});
diff --git a/addons/rose/tests/integration/components/rose/button-test.js b/addons/rose/tests/integration/components/rose/button-test.js
new file mode 100644
index 000000000..89ba514fb
--- /dev/null
+++ b/addons/rose/tests/integration/components/rose/button-test.js
@@ -0,0 +1,35 @@
+import { module, test } from 'qunit';
+import { setupRenderingTest } from 'ember-qunit';
+import { render, find } from '@ember/test-helpers';
+import hbs from 'htmlbars-inline-precompile';
+
+module('Integration | Component | rose/button', function (hooks) {
+  setupRenderingTest(hooks);
+
+  test('it renders', async function (assert) {
+    await render(hbs`<Rose::Button>Button 🌈</Rose::Button>`);
+    assert.equal(this.element.textContent.trim(), 'Button 🌈');
+  });
+
+  test('it is type="button" by default', async function (assert) {
+    await render(hbs`<Rose::Button>Button</Rose::Button>`);
+    assert.equal(await find('button').type, 'button');
+  });
+
+  test('it is type="submit" when @submit={{true}}', async function (assert) {
+    await render(hbs`<Rose::Button @submit={{true}} />`);
+    assert.equal(await find('button').type, 'submit');
+  });
+
+  test('it is disabled when @disabled={{true}}', async function (assert) {
+    await render(hbs`<Rose::Button @disabled={{true}} />`);
+    assert.equal(await find('button').disabled, true);
+  });
+
+  test('it adds a style class', async function (assert) {
+    await render(hbs`<Rose::Button @style="primary" />`);
+    assert.ok(await find('button.rose-button-primary'));
+    await render(hbs`<Rose::Button @style="secondary" />`);
+    assert.ok(await find('button.rose-button-secondary'));
+  });
+});