patternfly-react-demo-app icon indicating copy to clipboard operation
patternfly-react-demo-app copied to clipboard

[suggestion] dynamically populated page breadcrumbs

Open tomer opened this issue 7 years ago • 1 comments

At the moment the Breadcrumbs are available only on Ipsum 1-A and Ipsum 1-B, an both are hardcoded to the pages. I'd suggest populating in dynamically by the routes, and willing to contribute the required code if you find this suggestion useful for the demo app purposes.

tomer avatar Jul 29 '18 08:07 tomer

hi @tomer - apologies for the late response (I am returning from PTO 😄 ). I think dynamic breadcrumbs would be a nice addition as long as the code is minimal. We are trying to keep this boilerplate easy to follow and to prevent any additional code which is not typically used (thus the removal of some of the Redux boilerplate recently). Anything which is additional would be nice to place in snippets / generators down the road.

cc: @dgutride any additional thoughts on this one?

priley86 avatar Aug 06 '18 14:08 priley86