From 4cbfbafb387dfd96f4d6414e86bbb4ae501e189e Mon Sep 17 00:00:00 2001
From: Ivan Diaz <ivan@opensupports.com>
Date: Sun, 16 Aug 2015 16:16:53 -0300
Subject: [PATCH] [Ivan Diaz] - Add Button and widget

---
 app/js/App.js                                 |   4 +-
 .../{components => components-app}/Footer.js  |   3 -
 .../{components => components-app}/Header.js  |   3 -
 .../components-core/__tests__/button-test.js  |  35 ++
 app/js/components-core/button.js              |  44 ++
 app/js/components-core/widget.js              |  28 ++
 app/js/components/.gitkeep                    |   0
 app/js/components/__tests__/footer-test.js    |  17 -
 app/js/pages/component-demo-page.js           |  41 +-
 app/styles/_reset.scss                        | 427 ++++++++++++++++++
 app/styles/_vars.scss                         |   2 +
 .../{elements => app-components}/_footer.scss |   0
 .../{elements => app-components}/_header.scss |   0
 app/styles/core-components/_button.scss       |  16 +
 app/styles/core-components/_widget.scss       |   6 +
 app/styles/elements/.gitkeep                  |   0
 app/styles/main.scss                          |   7 +-
 package.json                                  |   4 +
 18 files changed, 597 insertions(+), 40 deletions(-)
 rename app/js/{components => components-app}/Footer.js (91%)
 rename app/js/{components => components-app}/Header.js (91%)
 create mode 100644 app/js/components-core/__tests__/button-test.js
 create mode 100644 app/js/components-core/button.js
 create mode 100644 app/js/components-core/widget.js
 delete mode 100644 app/js/components/.gitkeep
 delete mode 100644 app/js/components/__tests__/footer-test.js
 create mode 100644 app/styles/_reset.scss
 rename app/styles/{elements => app-components}/_footer.scss (100%)
 rename app/styles/{elements => app-components}/_header.scss (100%)
 create mode 100644 app/styles/core-components/_button.scss
 create mode 100644 app/styles/core-components/_widget.scss
 delete mode 100644 app/styles/elements/.gitkeep

diff --git a/app/js/App.js b/app/js/App.js
index 249d82d7..af81a6d5 100644
--- a/app/js/App.js
+++ b/app/js/App.js
@@ -6,8 +6,8 @@ import {RouteHandler}     from 'react-router';
 
 import CurrentUserActions from './actions/CurrentUserActions';
 import CurrentUserStore   from './stores/CurrentUserStore';
-import Header             from './components/Header';
-import Footer             from './components/Footer';
+import Header             from './components-app/Header';
+import Footer             from './components-app/Footer';
 
 var App = React.createClass({
 
diff --git a/app/js/components/Footer.js b/app/js/components-app/Footer.js
similarity index 91%
rename from app/js/components/Footer.js
rename to app/js/components-app/Footer.js
index d2d769ad..3bc42db2 100644
--- a/app/js/components/Footer.js
+++ b/app/js/components-app/Footer.js
@@ -7,9 +7,6 @@ var Footer = React.createClass({
   render() {
     return (
       <footer>
-
-        Footer
-
       </footer>
     );
   }
diff --git a/app/js/components/Header.js b/app/js/components-app/Header.js
similarity index 91%
rename from app/js/components/Header.js
rename to app/js/components-app/Header.js
index 9f4a1fbd..e9c44faa 100644
--- a/app/js/components/Header.js
+++ b/app/js/components-app/Header.js
@@ -7,9 +7,6 @@ var Header = React.createClass({
   render() {
     return (
       <header>
-
-        Header
-
       </header>
     );
   }
diff --git a/app/js/components-core/__tests__/button-test.js b/app/js/components-core/__tests__/button-test.js
new file mode 100644
index 00000000..1fd7a257
--- /dev/null
+++ b/app/js/components-core/__tests__/button-test.js
@@ -0,0 +1,35 @@
+/**
+ * Created by ivan on 16/08/15.
+ */
+jest.dontMock('../button.js');
+
+import React from 'react/addons';
+import Button from '../button.js';
+var TestUtils = React.addons.TestUtils;
+
+describe('Button', () => {
+	it('should render children', () => {
+		var button = TestUtils.renderIntoDocument(
+			<Button>
+				testcontent
+			</Button>
+		);
+
+		expect(button.getDOMNode().textContent).toEqual('testcontent');
+	});
+
+	it('should add passed types to class', () => {
+		var types = [
+			'primary'
+		];
+
+		types.forEach((type) => {
+			var button = TestUtils.renderIntoDocument(
+				<Button type={type}>
+					testcontent
+				</Button>
+			);
+			expect(button.getDOMNode().getAttribute('class')).toContain('button-' + type);
+		});
+	});
+});
\ No newline at end of file
diff --git a/app/js/components-core/button.js b/app/js/components-core/button.js
new file mode 100644
index 00000000..6ae24dca
--- /dev/null
+++ b/app/js/components-core/button.js
@@ -0,0 +1,44 @@
+/**
+ * Created by ivan on 16/08/15.
+ */
+'use strict';
+
+import React from 'react/addons';
+import classNames from 'classnames';
+
+var Button = React.createClass({
+
+	propTypes: {
+		children: React.PropTypes.node,
+		type: React.PropTypes.oneOf([
+			'primary'
+		])
+	},
+
+	getDefaultProps() {
+		return {
+			type: 'primary'
+		}
+	},
+
+	render() {
+		return (
+			<button className={this.getClass()}>
+				{this.props.children}
+			</button>
+		);
+	},
+
+	getClass() {
+		var classes = {
+			'button': true
+		};
+
+		classes['button-' + this.props.type] = (this.props.type);
+		classes[this.props.className] = (this.props.className);
+
+		return classNames(classes);
+	}
+});
+
+export default Button;
\ No newline at end of file
diff --git a/app/js/components-core/widget.js b/app/js/components-core/widget.js
new file mode 100644
index 00000000..098bccf7
--- /dev/null
+++ b/app/js/components-core/widget.js
@@ -0,0 +1,28 @@
+import React from 'react/addons';
+import classNames from 'classnames';
+
+var Widget = React.createClass({
+	propTypes: {
+		children: React.PropTypes.node.isRequired
+	},
+
+	render() {
+		return (
+			<div className={this.getClass()}>
+				{this.props.children}
+			</div>
+		);
+	},
+
+	getClass() {
+		var classes = {
+			'widget': true
+		};
+
+		classes[this.props.className] = (this.props.className);
+
+		return classNames(classes);
+	}
+});
+
+export default Widget;
\ No newline at end of file
diff --git a/app/js/components/.gitkeep b/app/js/components/.gitkeep
deleted file mode 100644
index e69de29b..00000000
diff --git a/app/js/components/__tests__/footer-test.js b/app/js/components/__tests__/footer-test.js
deleted file mode 100644
index 818688be..00000000
--- a/app/js/components/__tests__/footer-test.js
+++ /dev/null
@@ -1,17 +0,0 @@
-jest.dontMock('../Footer.js');
-
-import React from 'react/addons';
-import Footer from '../Footer.js';
-var TestUtils = React.addons.TestUtils;
-
-describe('Footer', () => {
-	it('should 2 equal 2', () => {
-		var footer = TestUtils.renderIntoDocument(
-			<Footer />
-		);
-		var label = TestUtils.findRenderedDOMComponentWithTag(
-			footer, 'footer');
-
-		expect(label.getDOMNode().textContent).toEqual('Footer');
-	});
-});
\ No newline at end of file
diff --git a/app/js/pages/component-demo-page.js b/app/js/pages/component-demo-page.js
index 504b0e34..fb8c26ef 100644
--- a/app/js/pages/component-demo-page.js
+++ b/app/js/pages/component-demo-page.js
@@ -4,32 +4,47 @@ import React         from 'react/addons';
 import {Link}        from 'react-router';
 import DocumentTitle from 'react-document-title';
 
-import Footer        from '../components/Footer.js';
+import Button from '../components-core/button.js';
 
-var HomePage = React.createClass({
+var DemoPage = React.createClass({
 
 	propTypes: {
 		currentUser: React.PropTypes.object.isRequired
 	},
 
+	elements: [
+		{
+			title: 'Primary Button',
+			render: (
+				<Button type="primary">Sign up</Button>
+			)
+		}
+	],
+
 	render() {
 		return (
 			<DocumentTitle title="Demo Page">
 				<section className="home-page">
-
-					<div>
-						FooterDemo
-					</div>
-
-					<div>
-						<Footer />
-					</div>
-
+					{this.renderElements()}
 				</section>
 			</DocumentTitle>
 		);
-	}
+	},
 
+	renderElements: function () {
+		return this.elements.map((element) => {
+			return (
+			<div className="demo-element">
+				<h4>
+					{element.title}
+				</h4>
+				<div class="demo-element--example">
+					{element.render}
+				</div>
+			</div>
+			);
+		});
+	}
 });
 
-export default HomePage;
\ No newline at end of file
+export default DemoPage;
\ No newline at end of file
diff --git a/app/styles/_reset.scss b/app/styles/_reset.scss
new file mode 100644
index 00000000..81c6f31e
--- /dev/null
+++ b/app/styles/_reset.scss
@@ -0,0 +1,427 @@
+/*! normalize.css v3.0.2 | MIT License | git.io/normalize */
+
+/**
+ * 1. Set default font family to sans-serif.
+ * 2. Prevent iOS text size adjust after orientation change, without disabling
+ *    user zoom.
+ */
+
+html {
+  font-family: sans-serif; /* 1 */
+  -ms-text-size-adjust: 100%; /* 2 */
+  -webkit-text-size-adjust: 100%; /* 2 */
+}
+
+/**
+ * Remove default margin.
+ */
+
+body {
+  margin: 0;
+}
+
+/* HTML5 display definitions
+   ========================================================================== */
+
+/**
+ * Correct `block` display not defined for any HTML5 element in IE 8/9.
+ * Correct `block` display not defined for `details` or `summary` in IE 10/11
+ * and Firefox.
+ * Correct `block` display not defined for `main` in IE 11.
+ */
+
+article,
+aside,
+details,
+figcaption,
+figure,
+footer,
+header,
+hgroup,
+main,
+menu,
+nav,
+section,
+summary {
+  display: block;
+}
+
+/**
+ * 1. Correct `inline-block` display not defined in IE 8/9.
+ * 2. Normalize vertical alignment of `progress` in Chrome, Firefox, and Opera.
+ */
+
+audio,
+canvas,
+progress,
+video {
+  display: inline-block; /* 1 */
+  vertical-align: baseline; /* 2 */
+}
+
+/**
+ * Prevent modern browsers from displaying `audio` without controls.
+ * Remove excess height in iOS 5 devices.
+ */
+
+audio:not([controls]) {
+  display: none;
+  height: 0;
+}
+
+/**
+ * Address `[hidden]` styling not present in IE 8/9/10.
+ * Hide the `template` element in IE 8/9/11, Safari, and Firefox < 22.
+ */
+
+[hidden],
+template {
+  display: none;
+}
+
+/* Links
+   ========================================================================== */
+
+/**
+ * Remove the gray background color from active links in IE 10.
+ */
+
+a {
+  background-color: transparent;
+}
+
+/**
+ * Improve readability when focused and also mouse hovered in all browsers.
+ */
+
+a:active,
+a:hover {
+  outline: 0;
+}
+
+/* Text-level semantics
+   ========================================================================== */
+
+/**
+ * Address styling not present in IE 8/9/10/11, Safari, and Chrome.
+ */
+
+abbr[title] {
+  border-bottom: 1px dotted;
+}
+
+/**
+ * Address style set to `bolder` in Firefox 4+, Safari, and Chrome.
+ */
+
+b,
+strong {
+  font-weight: bold;
+}
+
+/**
+ * Address styling not present in Safari and Chrome.
+ */
+
+dfn {
+  font-style: italic;
+}
+
+/**
+ * Address variable `h1` font-size and margin within `section` and `article`
+ * contexts in Firefox 4+, Safari, and Chrome.
+ */
+
+h1 {
+  font-size: 2em;
+  margin: 0.67em 0;
+}
+
+/**
+ * Address styling not present in IE 8/9.
+ */
+
+mark {
+  background: #ff0;
+  color: #000;
+}
+
+/**
+ * Address inconsistent and variable font size in all browsers.
+ */
+
+small {
+  font-size: 80%;
+}
+
+/**
+ * Prevent `sub` and `sup` affecting `line-height` in all browsers.
+ */
+
+sub,
+sup {
+  font-size: 75%;
+  line-height: 0;
+  position: relative;
+  vertical-align: baseline;
+}
+
+sup {
+  top: -0.5em;
+}
+
+sub {
+  bottom: -0.25em;
+}
+
+/* Embedded content
+   ========================================================================== */
+
+/**
+ * Remove border when inside `a` element in IE 8/9/10.
+ */
+
+img {
+  border: 0;
+}
+
+/**
+ * Correct overflow not hidden in IE 9/10/11.
+ */
+
+svg:not(:root) {
+  overflow: hidden;
+}
+
+/* Grouping content
+   ========================================================================== */
+
+/**
+ * Address margin not present in IE 8/9 and Safari.
+ */
+
+figure {
+  margin: 1em 40px;
+}
+
+/**
+ * Address differences between Firefox and other browsers.
+ */
+
+hr {
+  -moz-box-sizing: content-box;
+  box-sizing: content-box;
+  height: 0;
+}
+
+/**
+ * Contain overflow in all browsers.
+ */
+
+pre {
+  overflow: auto;
+}
+
+/**
+ * Address odd `em`-unit font size rendering in all browsers.
+ */
+
+code,
+kbd,
+pre,
+samp {
+  font-family: monospace, monospace;
+  font-size: 1em;
+}
+
+/* Forms
+   ========================================================================== */
+
+/**
+ * Known limitation: by default, Chrome and Safari on OS X allow very limited
+ * styling of `select`, unless a `border` property is set.
+ */
+
+/**
+ * 1. Correct color not being inherited.
+ *    Known issue: affects color of disabled elements.
+ * 2. Correct font properties not being inherited.
+ * 3. Address margins set differently in Firefox 4+, Safari, and Chrome.
+ */
+
+button,
+input,
+optgroup,
+select,
+textarea {
+  color: inherit; /* 1 */
+  font: inherit; /* 2 */
+  margin: 0; /* 3 */
+}
+
+/**
+ * Address `overflow` set to `hidden` in IE 8/9/10/11.
+ */
+
+button {
+  overflow: visible;
+}
+
+/**
+ * Address inconsistent `text-transform` inheritance for `button` and `select`.
+ * All other form control elements do not inherit `text-transform` values.
+ * Correct `button` style inheritance in Firefox, IE 8/9/10/11, and Opera.
+ * Correct `select` style inheritance in Firefox.
+ */
+
+button,
+select {
+  text-transform: none;
+}
+
+/**
+ * 1. Avoid the WebKit bug in Android 4.0.* where (2) destroys native `audio`
+ *    and `video` controls.
+ * 2. Correct inability to style clickable `input` types in iOS.
+ * 3. Improve usability and consistency of cursor style between image-type
+ *    `input` and others.
+ */
+
+button,
+html input[type="button"], /* 1 */
+input[type="reset"],
+input[type="submit"] {
+  -webkit-appearance: button; /* 2 */
+  cursor: pointer; /* 3 */
+}
+
+/**
+ * Re-set default cursor for disabled elements.
+ */
+
+button[disabled],
+html input[disabled] {
+  cursor: default;
+}
+
+/**
+ * Remove inner padding and border in Firefox 4+.
+ */
+
+button::-moz-focus-inner,
+input::-moz-focus-inner {
+  border: 0;
+  padding: 0;
+}
+
+/**
+ * Address Firefox 4+ setting `line-height` on `input` using `!important` in
+ * the UA stylesheet.
+ */
+
+input {
+  line-height: normal;
+}
+
+/**
+ * It's recommended that you don't attempt to style these elements.
+ * Firefox's implementation doesn't respect box-sizing, padding, or width.
+ *
+ * 1. Address box sizing set to `content-box` in IE 8/9/10.
+ * 2. Remove excess padding in IE 8/9/10.
+ */
+
+input[type="checkbox"],
+input[type="radio"] {
+  box-sizing: border-box; /* 1 */
+  padding: 0; /* 2 */
+}
+
+/**
+ * Fix the cursor style for Chrome's increment/decrement buttons. For certain
+ * `font-size` values of the `input`, it causes the cursor style of the
+ * decrement button to change from `default` to `text`.
+ */
+
+input[type="number"]::-webkit-inner-spin-button,
+input[type="number"]::-webkit-outer-spin-button {
+  height: auto;
+}
+
+/**
+ * 1. Address `appearance` set to `searchfield` in Safari and Chrome.
+ * 2. Address `box-sizing` set to `border-box` in Safari and Chrome
+ *    (include `-moz` to future-proof).
+ */
+
+input[type="search"] {
+  -webkit-appearance: textfield; /* 1 */
+  -moz-box-sizing: content-box;
+  -webkit-box-sizing: content-box; /* 2 */
+  box-sizing: content-box;
+}
+
+/**
+ * Remove inner padding and search cancel button in Safari and Chrome on OS X.
+ * Safari (but not Chrome) clips the cancel button when the search input has
+ * padding (and `textfield` appearance).
+ */
+
+input[type="search"]::-webkit-search-cancel-button,
+input[type="search"]::-webkit-search-decoration {
+  -webkit-appearance: none;
+}
+
+/**
+ * Define consistent border, margin, and padding.
+ */
+
+fieldset {
+  border: 1px solid #c0c0c0;
+  margin: 0 2px;
+  padding: 0.35em 0.625em 0.75em;
+}
+
+/**
+ * 1. Correct `color` not being inherited in IE 8/9/10/11.
+ * 2. Remove padding so people aren't caught out if they zero out fieldsets.
+ */
+
+legend {
+  border: 0; /* 1 */
+  padding: 0; /* 2 */
+}
+
+/**
+ * Remove default vertical scrollbar in IE 8/9/10/11.
+ */
+
+textarea {
+  overflow: auto;
+}
+
+/**
+ * Don't inherit the `font-weight` (applied by a rule above).
+ * NOTE: the default cannot safely be changed in Chrome and Safari on OS X.
+ */
+
+optgroup {
+  font-weight: bold;
+}
+
+/* Tables
+   ========================================================================== */
+
+/**
+ * Remove most spacing between table cells.
+ */
+
+table {
+  border-collapse: collapse;
+  border-spacing: 0;
+}
+
+td,
+th {
+  padding: 0;
+}
\ No newline at end of file
diff --git a/app/styles/_vars.scss b/app/styles/_vars.scss
index d0678ade..937b25bb 100644
--- a/app/styles/_vars.scss
+++ b/app/styles/_vars.scss
@@ -7,6 +7,8 @@ $blue: #1f8de2;
 $green: #1fe27b;
 $red: #e21f3f;
 
+$primary-red: #DD5555;
+
 // spacing
 $full-space: 40px;
 $half-space: 20px;
diff --git a/app/styles/elements/_footer.scss b/app/styles/app-components/_footer.scss
similarity index 100%
rename from app/styles/elements/_footer.scss
rename to app/styles/app-components/_footer.scss
diff --git a/app/styles/elements/_header.scss b/app/styles/app-components/_header.scss
similarity index 100%
rename from app/styles/elements/_header.scss
rename to app/styles/app-components/_header.scss
diff --git a/app/styles/core-components/_button.scss b/app/styles/core-components/_button.scss
new file mode 100644
index 00000000..6fede974
--- /dev/null
+++ b/app/styles/core-components/_button.scss
@@ -0,0 +1,16 @@
+.button {
+
+  &:focus {
+    outline: none;
+  }
+
+  &-primary {
+    background-color: $primary-red;
+    border: solid transparent;
+    border-radius: 4px;
+    color: white;
+    height: 47px;
+    text-transform: uppercase;
+    width: 239px;
+  }
+}
\ No newline at end of file
diff --git a/app/styles/core-components/_widget.scss b/app/styles/core-components/_widget.scss
new file mode 100644
index 00000000..fea43bb7
--- /dev/null
+++ b/app/styles/core-components/_widget.scss
@@ -0,0 +1,6 @@
+.widget {
+  background-color: white;
+  border-radius: 4px;
+  min-height: 50px;
+  width: 324px;
+}
\ No newline at end of file
diff --git a/app/styles/elements/.gitkeep b/app/styles/elements/.gitkeep
deleted file mode 100644
index e69de29b..00000000
diff --git a/app/styles/main.scss b/app/styles/main.scss
index 2fce83f2..d288a93d 100644
--- a/app/styles/main.scss
+++ b/app/styles/main.scss
@@ -1,6 +1,9 @@
+@import 'reset';
 @import 'vars';
 @import 'typography';
 @import 'base';
 
-@import 'elements/header';
-@import 'elements/footer';
\ No newline at end of file
+@import 'app-components/header';
+@import 'app-components/footer';
+@import 'core-components/button';
+@import 'core-components/widget';
\ No newline at end of file
diff --git a/package.json b/package.json
index 068b906a..9e174bf6 100644
--- a/package.json
+++ b/package.json
@@ -68,5 +68,9 @@
     "unmockedModulePathPatterns": [
       "react"
     ]
+  },
+  "dependencies": {
+    "app-module-path": "^1.0.3",
+    "classnames": "^2.1.3"
   }
 }