Review breadcrumbs of the application to ensure consistency

syndesisio / syndesis

This project is archived. A flexible, customizable, open source platform that provides core integration capabilities as a service.

https://syndesis.io/

Apache License 2.0

597 stars 204 forks source link

Review breadcrumbs of the application to ensure consistency #2353

Closed dongniwang closed 5 years ago

dongniwang commented 6 years ago

This is a...


[x] Feature request
[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report  
[ ] Documentation issue or request

The problem

Breadcrumbs in various pages look different and have different level of information.

Expected behavior

Could we serve all the breadcrumb from a single source?
Should breadcrumb always starts with Home?

Screenshot

screen shot 2018-04-11 at 2 23 56 pm screen shot 2018-04-11 at 2 21 56 pm

mcoker commented 6 years ago

@gashcrumb @zregvart do you have an opinion on this? Maybe creating a breadcrumb that generates dynamically instead of creating them statically?

I would advocate leaving "home" in the breadcrumbs as that's standard/expected for breadcrumbs in general.

zregvart commented 6 years ago

Perhaps we can look at how other projects do this: from the screenshots here EAP console (uses patternfly) has a << Back as a start, in OpenShift console breadcrumbs start with the tab/section name. Not sure if that helps they don't seem to have a Home concept though.

I think we can leave the decision on this this with @syndesisio/uxd

I think we might be able to take some stuff from hawtio.

mcoker commented 6 years ago

For reference

patternfly breadcrumb guideline http://www.patternfly.org/pattern-library/navigation/breadcrumbs/#design
patternfly html snippet https://rawgit.com/patternfly/patternfly/master-dist/dist/tests/breadcrumbs.html
patternfly-react component https://rawgit.com/patternfly/patternfly-react/gh-pages/index.html?selectedKind=patternfly-react%2FNavigation%2FBreadcrumb&selectedStory=Breadcrumb&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybooks%2Fstorybook-addon-knobs

stale[bot] commented 5 years ago

This issue has been automatically marked as stale because it has not had any activity since 90 days. It will be closed if no further activity occurs within 7 days. Thank you for your contributions!

amysueg commented 5 years ago

@dongniwang ... Is there ever a good time to review breadcrumbs? ;-)

stale[bot] commented 5 years ago

This issue has been automatically marked as stale because it has not had any activity since 90 days. It will be closed if no further activity occurs within 7 days. Thank you for your contributions!