Problem/Motivation

The current page is not added to the breadcrumb

Proposed resolution

This patch adds support for optionally display the current page item in the breadcrumb.

Remaining tasks

Needs review by the module's mantainer

User interface changes

The option can be turned on/off on the module's configuration page

API changes

None

CommentFileSizeAuthor
menu_trail_by_path-display_current_page_in_breadcrumb-1.0.patch3.2 KBralf57

Comments

subhojit777’s picture

subhojit777
he/him
Primary language Bengali
commented
Status: Active » Needs review

The patch is working. But the current page was not appearing as link, is it supposed to be like this? The menutrails module was rendering the current page as link in breadcrumb.

SeriousMatters’s picture

SeriousMatters commented
Category: task » feature

I like this feature. But is it i18n safe?

if ($link_path == current_path()) {
  $breadcrumbs[] = '<span>' . $title . '</span>';
}
nielsonm’s picture

nielsonm commented

A lot of themes have this as a setting, for example the markup overrides in Adaptive Theme have the ability to add both the current page and the homepage in the breadcrumb. I'd say that it's a theme issue.

davy-r’s picture

davy-r commented
Issue summary: View changes
Status: Needs review » Active
davy-r’s picture

davy-r commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, menu_trail_by_path-display_current_page_in_breadcrumb-1.0.patch, failed testing.

The last submitted patch, menu_trail_by_path-display_current_page_in_breadcrumb-1.0.patch, failed testing.

The last submitted patch, menu_trail_by_path-display_current_page_in_breadcrumb-1.0.patch, failed testing.

The last submitted patch, menu_trail_by_path-display_current_page_in_breadcrumb-1.0.patch, failed testing.

davy-r’s picture

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

  • davy-r committed c8a75ac on 7.x-3.x 
    Issue #1809980 by ralf57, davy-r: Support for current page item in the...

  • davy-r committed 2ce45e2 on 7.x-3.x 
    Issue #1809980 by ralf57, davy-r: Support for current page item in the...
davy-r’s picture

davy-r commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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