Could the identifier "none" be wrapped in a span tag so it would be possible to position it easily along with other crumbs, which are wrapped in a tags ?

<?php
        'crumb' => '<span class="custom-breadcrumbs-none">'. check_plain($obj['title']) .'</span>',
?>

Or, could we have a theme function for setting the crumb?

Please let me know which you prefer and I can provide a patch.

Thanks

CommentFileSizeAuthor
#4 1002568_custom_breadcrumbs_none.diff2.32 KBMGN
#3 custom_breadcrumbs-none.patch2.08 KBawolfey
#2 custom_breadcrumbs-none.patch2.08 KBawolfey

Comments

MGN’s picture

MGN commented
Version: 6.x-2.0-beta3 » 6.x-2.x-dev

Yes, this can be done. If you would like to write the patch, write it against the latest 6.x-2.x-dev. Have a look at the function _custom_breadcrumbs_identifiers_option() at the end of the custom_breadcrumbs.module file, and 'custom_breadcrumbs_identifiers' settings fieldset in custom_breadcrumbs.admin.inc.

awolfey’s picture

awolfey commented
Status: Active » Needs review
StatusFileSize
new custom_breadcrumbs-none.patch2.08 KB

Here's a first pass. I didn't do anything in _custom_breadcrumbs_identifiers_option() because the attributes are not available there.

awolfey’s picture

awolfey commented
StatusFileSize
new custom_breadcrumbs-none.patch2.08 KB

Oops. The class was missing a "s" for consitency.

MGN’s picture

MGN commented
StatusFileSize
new 1002568_custom_breadcrumbs_none.diff2.32 KB

Thanks for the patch. I am afraid I misled you. Since <none> is now handled by the custom_breadcrumbs_identifiers submodule, this feature should be handled by that module. Can you test the following patch and see if it accomplishes what you intended?

awolfey’s picture

awolfey commented

Yes, this looks good and works for me.

Thanks.

MGN’s picture

MGN commented
Status: Needs review » Fixed

Thanks for testing. This has been committed to 6.x-2.x-dev.

Status: Fixed » Closed (fixed)

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