search

XavierUniversity / V4

www.xavier.edu
1 stars 1 forks source link

Breadcrumbs #32

Open mcmullengreg opened 7 years ago

mcmullengreg commented 7 years ago

@lieslandr How confident do we feel about rewriting the breadcrumb function for CS.

My initial thought is that we can totally do it and we should write it with two "modes."

  1. Index page has "Xavier Home" h1 for "department name"
  2. Other pages have "Xavier Home" > Dept > Parent Page (if present, etc) > h1 for Page Name

Thoughts?

mcmullengreg commented 7 years ago

This is a ways off for dev, but I just remember it being an issue on the current version.

mcmullengreg commented 7 years ago

@lieslandr The current version of Campus Suite uses the navigation label (defined in edit nav), not the Page Title. Do we want to continue this trend or flip it to use the Title instead?

Should we do a better job of linking the title and the label?

mcmullengreg commented 7 years ago

H1 should be the pageTitle. Navigation label will not be affected.

mcmullengreg commented 7 years ago

Bad news bears, @lieslandr. PageTitle isn't stored anywhere that we can reference it for the breadcrumbs. We'll have to use "label" (at the very least) for the items in the breadcrumb trail. The h1 will be unaffected.

lieslandr commented 7 years ago

We can pull it out of the page, like we do now for the H1.

mcmullengreg commented 7 years ago

I'm talking about the parent pages within the breadcrumb trail. We'd have to traverse to those pages and grab the pageTitle.