I have day view located at "calendar/day/2012-10-19." calendar/day receives the active-trail class but the page with the argument does not.

CommentFileSizeAuthor
#6 support_views_arguments-1704846-6.patch1.18 KBAnonymous (not verified)

Comments

lencrockett’s picture

lencrockett commented

I just realized this same thing. I would love to see support for views arguments in trails as many sites are designed this way. Unfortunately as it is I cannot get much use out of an otherwise great module.

SeriousMatters’s picture

SeriousMatters commented

If the current page is /calendar/day/2012-10-19, these menu links get active-trail
/calendar
/calendar/day
/calendar/day/2012-10-19

I don't think menu link can have % in its path like /calendar/day/%. Please clarify what do you meant by 'views arguments support'.

Also note that list of clickable items created by Views is not Drupal menu and will not get active-trail at all.

brentratliff’s picture

brentratliff commented

/calendar/day/% is the exact case I'm talking about. That explains it I guess.

akalam’s picture

akalam commented
Issue summary: View changes

I think this is a views issue. See https://www.drupal.org/node/1689682

grathbone’s picture

grathbone commented

@desarrollo2.0 The Views issue that you reference is actually different and has been fixed for a while now.

If the Menu Trail by Path breadcrumb is disabled, the Views breadcrumb works perfectly fine with Contextual Filters. When MTbP breadcrumb is enabled, every breadcrumb item turns into the default View title.

In example:
Path: blog/articles/2015/05/07
Breadcrumb (MTbP disabled): Home > Blog > Articles > 2015 > May > 07
Breadcrumb (MTbP enabled): Home > Blog > Blog > Blog > Blog > Blog

Anonymous’s picture

Anonymous (not verified) commented
Version: 7.x-2.0-rc2 » 7.x-2.x-dev
Status: Active » Needs review
StatusFileSize
new support_views_arguments-1704846-6.patch1.18 KB

Here's a patch that seems to work in my situation but needs testing to make sure it works in other situations too.

davy-r’s picture

davy-r commented
Version: 7.x-2.x-dev » 7.x-3.x-dev
davy-r’s picture

davy-r commented

This should now be fixed in 7.x-3.x-dev.

davy-r’s picture

davy-r commented
Status: Needs review » Fixed

Just successfully tested

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.