doesn't play nice with Breadcrumb

[TrejGun]

Hi!

I have found a small issue

<Breadcrumb>{this.props.routes.map((item, i) =>
    <LinkContainer key={i} to={item.path}>
        <BreadcrumbItem>
            {item.component.displayName}
        </BreadcrumbItem>
    </LinkContainer>
)}</Breadcrumb>

this code shows warning in console

Warning: [react-bootstrap] `href` and `active` properties cannot be set at the same time

and all links are inactive

also can you please add this example to visual tests

[taion]

This is an upstream issue with the breadcrumb component. If you want this in the visual tests, make a PR.

[jquense]

is this upstream? the upstream behavior is not a bug persay...I guess it makes most sense to fix it there tho?

which is to say there was discussion around having the current behavior.

[taion]

There's not really anything we can do in <LinkContainer>... doesn't make sense to drop the href when active here.

[jquense]

hmm yeah I agree ok. fair enough!