Hi,

This theme looks great, but I think the breadcrumbs are in the wrong place. Thinking about the visual hierarchy of the page, it would make more sense to me if they came above/before the current page title, because that is the end of the trail. Does anyone agree?

CommentFileSizeAuthor
#2 gallery-content_png__PNG_Image__1007 × 792_pixels__-_Scaled__79__.png11.09 KBdarrenmothersele
#1 Screen Shot 2014-03-28 at 2.53.54 PM.png17.16 KBtkoleary
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tkoleary’s picture

tkoleary CreditAttribution: tkoleary commented
FileSize
Screen Shot 2014-03-28 at 2.53.54 PM.png17.16 KB

I do agree. Like this?

breadcrumb

darrenmothersele’s picture

darrenmothersele CreditAttribution: darrenmothersele commented
FileSize
gallery-content_png__PNG_Image__1007 × 792_pixels__-_Scaled__79__.png11.09 KB

Here's the related bit from Adminimum theme. Ignoring tab position, I think they got breadcrumbs in the right place...

tkoleary’s picture

tkoleary CreditAttribution: tkoleary commented
Assigned: Unassigned » tkoleary
Status: Active » Needs review

Breadcrumbs moved into branding and re-styled in dev branch.

sylus’s picture

sylus CreditAttribution: sylus commented
Status: Needs review » Fixed

I took a look at this and added the modifications to my subtheme.

Styling looks good and does provide a more consistent view of the menu trail.

Since this directly addresses the issues by the OP hope it is okay if I move to fixed status :)

tkoleary’s picture

tkoleary CreditAttribution: tkoleary commented

Sure! Not a problem.

tkoleary’s picture

tkoleary CreditAttribution: tkoleary commented
Status: Fixed » Closed (fixed)