Skip to content

Commit

Permalink
Fix basename in examples better (#3610)
Browse files Browse the repository at this point in the history
  • Loading branch information
taion authored and timdorr committed Jul 5, 2016
1 parent 6e50ece commit 00c44b7
Show file tree
Hide file tree
Showing 17 changed files with 41 additions and 35 deletions.
4 changes: 2 additions & 2 deletions examples/active-links/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import React from 'react'
import { render } from 'react-dom'
import { Router, Route, IndexRoute, Link, IndexLink, browserHistory } from 'react-router'

import withBasename from '../withBasename'
import withExampleBasename from '../withExampleBasename'

const ACTIVE = { color: 'red' }

Expand Down Expand Up @@ -59,7 +59,7 @@ const About = () => (
)

render((
<Router history={withBasename(browserHistory, __dirname)}>
<Router history={withExampleBasename(browserHistory, __dirname)}>
<Route path="/" component={App}>
<IndexRoute component={Index}/>
<Route path="/about" component={About}/>
Expand Down
4 changes: 2 additions & 2 deletions examples/animations/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import ReactCSSTransitionGroup from 'react-addons-css-transition-group'
import { render } from 'react-dom'
import { browserHistory, Router, Route, IndexRoute, Link } from 'react-router'

import withBasename from '../withBasename'
import withExampleBasename from '../withExampleBasename'

import './app.css'

Expand Down Expand Up @@ -49,7 +49,7 @@ const Page2 = () => (
)

render((
<Router history={withBasename(browserHistory, __dirname)}>
<Router history={withExampleBasename(browserHistory, __dirname)}>
<Route path="/" component={App}>
<IndexRoute component={Index}/>
<Route path="page1" component={Page1} />
Expand Down
4 changes: 2 additions & 2 deletions examples/auth-flow-async-with-query-params/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import React, { createClass } from 'react'
import { render } from 'react-dom'
import { Router, Route, IndexRoute, browserHistory, Link, withRouter } from 'react-router'

import withBasename from '../withBasename'
import withExampleBasename from '../withExampleBasename'

function App(props) {
return (
Expand Down Expand Up @@ -88,7 +88,7 @@ function serverAuth(authToken) {
}

render((
<Router history={withBasename(browserHistory, __dirname)}>
<Router history={withExampleBasename(browserHistory, __dirname)}>
<Route path="/" component={App}>
<IndexRoute component={Form} />
<Route path="page" component={Page} onEnter={requireCredentials}/>
Expand Down
4 changes: 2 additions & 2 deletions examples/auth-flow/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import React from 'react'
import { render } from 'react-dom'
import { browserHistory, Router, Route, Link, withRouter } from 'react-router'

import withBasename from '../withBasename'
import withExampleBasename from '../withExampleBasename'
import auth from './auth'

const App = React.createClass({
Expand Down Expand Up @@ -127,7 +127,7 @@ function requireAuth(nextState, replace) {
}

render((
<Router history={withBasename(browserHistory, __dirname)}>
<Router history={withExampleBasename(browserHistory, __dirname)}>
<Route path="/" component={App}>
<Route path="login" component={Login} />
<Route path="logout" component={Logout} />
Expand Down
7 changes: 5 additions & 2 deletions examples/auth-with-shared-root/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,12 @@ import React from 'react'
import { render } from 'react-dom'
import { browserHistory, Router } from 'react-router'

import withBasename from '../withBasename'
import withExampleBasename from '../withExampleBasename'
import routes from './config/routes'

render((
<Router history={withBasename(browserHistory, __dirname)} routes={routes}/>
<Router
history={withExampleBasename(browserHistory, __dirname)}
routes={routes}
/>
), document.getElementById('example'))
4 changes: 2 additions & 2 deletions examples/breadcrumbs/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import React from 'react'
import { render } from 'react-dom'
import { browserHistory, Router, Route, Link } from 'react-router'

import withBasename from '../withBasename'
import withExampleBasename from '../withExampleBasename'

import './app.css'

Expand Down Expand Up @@ -60,7 +60,7 @@ Orders.title = 'Orders'
Orders.path = '/orders'

render((
<Router history={withBasename(browserHistory, __dirname)}>
<Router history={withExampleBasename(browserHistory, __dirname)}>
<Route path={App.path} component={App}>
<Route path={Products.path} component={Products} />
<Route path={Orders.path} component={Orders} />
Expand Down
4 changes: 2 additions & 2 deletions examples/confirming-navigation/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import React from 'react'
import { render } from 'react-dom'
import { browserHistory, Router, Route, Link, withRouter } from 'react-router'

import withBasename from '../withBasename'
import withExampleBasename from '../withExampleBasename'

const App = React.createClass({
render() {
Expand Down Expand Up @@ -76,7 +76,7 @@ const Form = withRouter(
)

render((
<Router history={withBasename(browserHistory, __dirname)}>
<Router history={withExampleBasename(browserHistory, __dirname)}>
<Route path="/" component={App}>
<Route path="dashboard" component={Dashboard} />
<Route path="form" component={Form} />
Expand Down
4 changes: 2 additions & 2 deletions examples/dynamic-segments/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import React from 'react'
import { render } from 'react-dom'
import { browserHistory, Router, Route, Link, Redirect } from 'react-router'

import withBasename from '../withBasename'
import withExampleBasename from '../withExampleBasename'

const App = ({ children }) => (
<div>
Expand Down Expand Up @@ -33,7 +33,7 @@ const Task = ({ params: { userID, taskID } }) => (
)

render((
<Router history={withBasename(browserHistory, __dirname)}>
<Router history={withExampleBasename(browserHistory, __dirname)}>
<Route path="/" component={App}>
<Route path="user/:userID" component={User}>
<Route path="tasks/:taskID" component={Task} />
Expand Down
9 changes: 6 additions & 3 deletions examples/huge-apps/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@ import React from 'react'
import { render } from 'react-dom'
import { Router, browserHistory } from 'react-router'

import withBasename from '../withBasename'
import withExampleBasename from '../withExampleBasename'
import './stubs/COURSES'

const route = {
const rootRoute = {
childRoutes: [ {
path: '/',
component: require('./components/App'),
Expand All @@ -20,7 +20,10 @@ const route = {
}

render((
<Router history={withBasename(browserHistory, __dirname)} routes={route} />
<Router
history={withExampleBasename(browserHistory, __dirname)}
routes={rootRoute}
/>
), document.getElementById('example'))

// I've unrolled the recursive directory loop that is happening above to get a
Expand Down
4 changes: 2 additions & 2 deletions examples/master-detail/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import React from 'react'
import { render, findDOMNode } from 'react-dom'
import { browserHistory, Router, Route, IndexRoute, Link, withRouter } from 'react-router'

import withBasename from '../withBasename'
import withExampleBasename from '../withExampleBasename'
import ContactStore from './ContactStore'

import './app.css'
Expand Down Expand Up @@ -151,7 +151,7 @@ const NotFound = React.createClass({
})

render((
<Router history={withBasename(browserHistory, __dirname)}>
<Router history={withExampleBasename(browserHistory, __dirname)}>
<Route path="/" component={App}>
<IndexRoute component={Index} />
<Route path="contact/new" component={NewContact} />
Expand Down
4 changes: 2 additions & 2 deletions examples/nested-animations/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import ReactCSSTransitionGroup from 'react-addons-css-transition-group'
import { render } from 'react-dom'
import { browserHistory, Router, Route, Link } from 'react-router'

import withBasename from '../withBasename'
import withExampleBasename from '../withExampleBasename'

import './app.css'

Expand Down Expand Up @@ -74,7 +74,7 @@ const Tab2 = () => (
)

render((
<Router history={withBasename(browserHistory, __dirname)}>
<Router history={withExampleBasename(browserHistory, __dirname)}>
<Route path="/" component={App}>
<Route path="page1" component={Page1}>
<Route path="tab1" component={Tab1} />
Expand Down
4 changes: 2 additions & 2 deletions examples/passing-props-to-children/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import React from 'react'
import { render } from 'react-dom'
import { browserHistory, Router, Route, Link, withRouter } from 'react-router'

import withBasename from '../withBasename'
import withExampleBasename from '../withExampleBasename'

import './app.css'

Expand Down Expand Up @@ -78,7 +78,7 @@ const Taco = React.createClass({
})

render((
<Router history={withBasename(browserHistory, __dirname)}>
<Router history={withExampleBasename(browserHistory, __dirname)}>
<Route path="/" component={App}>
<Route path="taco/:name" component={Taco} />
</Route>
Expand Down
4 changes: 2 additions & 2 deletions examples/pinterest/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import React from 'react'
import { render } from 'react-dom'
import { browserHistory, Router, Route, IndexRoute, Link } from 'react-router'

import withBasename from '../withBasename'
import withExampleBasename from '../withExampleBasename'

const PICTURES = [
{ id: 0, src: 'http://placekitten.com/601/601' },
Expand Down Expand Up @@ -147,7 +147,7 @@ const Picture = React.createClass({
})

render((
<Router history={withBasename(browserHistory, __dirname)}>
<Router history={withExampleBasename(browserHistory, __dirname)}>
<Route path="/" component={App}>
<IndexRoute component={Index}/>
<Route path="/pictures/:id" component={Picture}/>
Expand Down
4 changes: 2 additions & 2 deletions examples/query-params/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import React from 'react'
import { render } from 'react-dom'
import { browserHistory, Router, Route, Link } from 'react-router'

import withBasename from '../withBasename'
import withExampleBasename from '../withExampleBasename'

const User = ({ params: { userID }, location: { query } }) => {
let age = query && query.showAge ? '33' : ''
Expand All @@ -27,7 +27,7 @@ const App = ({ children }) => (
)

render((
<Router history={withBasename(browserHistory, __dirname)}>
<Router history={withExampleBasename(browserHistory, __dirname)}>
<Route path="/" component={App}>
<Route path="user/:userID" component={User} />
</Route>
Expand Down
4 changes: 2 additions & 2 deletions examples/route-no-match/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import React from 'react'
import { render } from 'react-dom'
import { browserHistory, Router, Route, Link } from 'react-router'

import withBasename from '../withBasename'
import withExampleBasename from '../withExampleBasename'

class User extends React.Component {
render() {
Expand Down Expand Up @@ -46,7 +46,7 @@ class PageNotFound extends React.Component {
}

render((
<Router history={withBasename(browserHistory, __dirname)}>
<Router history={withExampleBasename(browserHistory, __dirname)}>
<Route path="/" component={App}>
<Route path="user/:userID" component={User} />
</Route>
Expand Down
4 changes: 2 additions & 2 deletions examples/sidebar/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import React from 'react'
import { render } from 'react-dom'
import { browserHistory, Router, Route, Link } from 'react-router'

import withBasename from '../withBasename'
import withExampleBasename from '../withExampleBasename'
import data from './data'

import './app.css'
Expand Down Expand Up @@ -84,7 +84,7 @@ const App = ({ content, sidebar }) => (
)

render((
<Router history={withBasename(browserHistory, __dirname)}>
<Router history={withExampleBasename(browserHistory, __dirname)}>
<Route path="/" component={App}>
<Route path="category/:category" components={{ content: Category, sidebar: CategorySidebar }}>
<Route path=":item" component={Item} />
Expand Down
4 changes: 2 additions & 2 deletions examples/withBasename.js → examples/withExampleBasename.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import useBasename from 'history/lib/useBasename';
import useBasename from 'history/lib/useBasename'

// This helper is for setting basename on examples with minimal boilerplate. In
// an actual application, you would build a custom history to set basename.
export default function withBasename(history, dirname) {
export default function withExampleBasename(history, dirname) {
return useBasename(() => history)({ basename: `/${dirname}` })
}

0 comments on commit 00c44b7

Please sign in to comment.