add the 'type' attribute on buttons to prevent auto-submission if the…

… calendar is contained in a form
tomkp · Jan 3, 2017 · 1ff63f7 · 1ff63f7
1 parent c1b9ba9
commit 1ff63f7
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/src/Calendar.js b/src/Calendar.js
@@ -159,13 +159,13 @@ export default React.createClass({
           nav = (
               <tr className="month-header">
                   <th className="nav previous">
-                      <button className="nav-inner" onClick={this.previous}>«</button>
+                      <button className="nav-inner" onClick={this.previous} type="button">«</button>
                   </th>
                   <th colSpan="5">
                       <span className="month">{month.format('MMMM')}</span> <span className="year">{month.format('YYYY')}</span>
                   </th>
                   <th className="nav next">
-                      <button className="nav-inner" onClick={this.next}>»</button>
+                      <button className="nav-inner" onClick={this.next} type="button">»</button>
                   </th>
               </tr>
           )

diff --git a/src/Day.js b/src/Day.js
@@ -28,8 +28,11 @@ export default React.createClass({
         }
         else {
             body = (
-                <button className="Day-inner"
-                    onClick={() => this.props.handleClick(this.props.date)}>
+                <button
+                    className="Day-inner"
+                    onClick={() => this.props.handleClick(this.props.date)}
+                    type="button"
+                >
                     {this.props.date.format('D')}
                 </button>
             )