Merge pull request #179 from react-bootstrap/support-action

Support router action type

Author: jquense

package.json

@@ -32,8 +32,7 @@
   },
   "homepage": "https://github.com/react-bootstrap/react-router-bootstrap",
   "peerDependencies": {
-    "react": ">=0.14.0",
-    "react-router": ">=2.0.0"
+    "react": ">=0.14.0"
   },
   "devDependencies": {
     "babel-cli": "^6.5.2",

src/LinkContainer.js

@@ -1,16 +1,44 @@
 // This is largely taken from react-router/lib/Link.
 
 import React from 'react';
-import Link from 'react-router/lib/Link';
+
+function isLeftClickEvent(event) {
+  return event.button === 0;
+}
+
+function isModifiedEvent(event) {
+  return !!(
+    event.metaKey ||
+    event.altKey ||
+    event.ctrlKey ||
+    event.shiftKey
+  );
+}
+
+function createLocationDescriptor(to, query, hash, state) {
+  if (query || hash || state) {
+    return { pathname: to, query, hash, state };
+  }
+
+  return to;
+}
 
 const propTypes = {
   onlyActiveOnIndex: React.PropTypes.bool.isRequired,
   to: React.PropTypes.oneOfType([
     React.PropTypes.string,
     React.PropTypes.object,
   ]).isRequired,
+  query: React.PropTypes.string,
+  hash: React.PropTypes.string,
+  state: React.PropTypes.object,
+  action: React.PropTypes.oneOf([
+    'push',
+    'replace',
+  ]).isRequired,
   onClick: React.PropTypes.func,
   active: React.PropTypes.bool,
+  target: React.PropTypes.string,
   children: React.PropTypes.node.isRequired,
 };
 
@@ -20,15 +48,37 @@ const contextTypes = {
 
 const defaultProps = {
   onlyActiveOnIndex: false,
+  action: 'push',
 };
 
 class LinkContainer extends React.Component {
   onClick = (event) => {
-    if (this.props.children.props.onClick) {
-      this.props.children.props.onClick(event);
+    const {
+      to, query, hash, state, children, onClick, target, action,
+    } = this.props;
+
+    if (children.props.onClick) {
+      children.props.onClick(event);
+    }
+
+    if (onClick) {
+      onClick(event);
     }
 
-    Link.prototype.handleClick.call(this, event);
+    if (
+      target ||
+      event.defaultPrevented ||
+      isModifiedEvent(event) ||
+      !isLeftClickEvent(event)
+    ) {
+      return;
+    }
+
+    event.preventDefault();
+
+    this.context.router[action](
+      createLocationDescriptor(to, query, hash, state)
+    );
   };
 
   render() {