-
Notifications
You must be signed in to change notification settings - Fork 865
Breadcrumbs demo #135
Comments
Funny, I did exactly the same for the website that require a breadcrumb (some design don't, but usually I just hide it, so google is able to take it for SEO)... |
Nitpicking: shouldn't the |
I have tried to keep the theme independent of third-party plugins. Would also like to keep it that way. But what if we include those parts of the plugin that deals with breadcrumbs directly in the theme (via functions.php)? Is it a good or bad idea? |
I've used a couple of breadcrumb code (from foundation 5) in other wp sites that aren't dependent on any plugins. I'll see if I can add it here. http://paglalakbay.biz/custom-breadcrumbs-function/ This was prior to 5.4.7 but should still work with a few adjustments. |
I have some code that does the SEO that Yoast's plugin does and also works with FoundationPress, how can I send it? |
@themestechnology : Please create a new pull request: https://help.github.com/articles/using-pull-requests/ |
Did this ever get committed? |
Im on the move at the moment so I have not yet tested the code but this is basically what you need, I will do a commit in a few hours... https://github.com/themestechnology/FoundationPress-comments/blob/master/parts/breadcrumb-mettadata.php simply use get_template_part('parts/breadcrumb-mettadata'); in your footer |
I prefer to add the function into a custom-functions.php file in library |
Just use a do_action('FoundationPress_do_bredcrum'); to place it where you want |
There is now a pull request for that @olefredrik however I will leave it up to you do remove another breadcrumbs in the theme |
Something I add to FoundationPress - usually at the end of header.php - is a quick bit of code for breadcrumbs (internal links):
This adds a grid row, and prints the breadcrumbs applying the relevant classes [the div can also be within the
if
statement, just an example here to show which bit does what].On the one hand, this is a plugin dependency [albeit with a check], but on the other hand it's a great way to use/demo the Foundation breadcrumbs classes in a Wordpress site.
I can make a pull request if you think it's worth including in the core theme.
The text was updated successfully, but these errors were encountered: