as seperation

Resolved Philipp
(@philippmuenchen)

11 years, 1 month ago
Hi!

First of all thank you so much for this amazing and super useful PlugIn.

Since the last update I face the problem that breadcrumbs get corrupted because between the items of the trail is the separation <li> missing.
E.g.:
```
<ul class="breadcrumbs">

<li class="home"><a title="Home" href="http://world.com">Home</a></li>
<li class="current_item"><a title="Link 1." href="/link1/">Link 1</a>
<a title="Link 2." href="/link2/">Link 2</a>
</li>
</ul>
```
So there’s something missing between Link 1 and Link 2:
```
<ul class="breadcrumbs">

<li class="home"><a title="Home" href="http://world.com">Home</a>
<li class="current_item"><a title="Link 1." href="/link1/">Link 1</a>
</li><li>
<a title="Link 2." href="/link2/">Link 2</a>
</li>
</ul>
```
Furthermore I can’t use <li> as separation because the plugin deletes it automatically.

It would be great if you could help me!

Best!

Philipp

http://wordpress.org/extend/plugins/breadcrumb-navxt/

[Please post code & markup between backticks or use the code button. Your posted code may now have been permanently damaged by the forum’s parser.]

Viewing 12 replies - 1 through 12 (of 12 total)

Plugin Author John Havlik
(@mtekk)

11 years, 1 month ago

This is related to the same thing discussed in this thread: http://wordpress.org/support/topic/custom-mark-up-not-being-saved?replies=9

The short answer is that the breadcrumb templates and the breadcrumb separator were never meant to contain li elements. The official way has always been to use bcn_display_list() rather than bcn_display().

Thread Starter Philipp
(@philippmuenchen)

11 years, 1 month ago

Hi mtekk!

Thank you for your answer. I’m already using bcn_display_list() and that’s ehy ot’s confusing me that there are no li elements between the single items.
bcn_display_list() generates the code I posted before.

Thank you!

Philipp

Plugin Author John Havlik
(@mtekk)

11 years, 1 month ago

Ok, that’s a different issue then, and I agree that’s very odd. I’ll see if I can reproduce that on my testbed and let you know what I find.

Plugin Author John Havlik
(@mtekk)

11 years, 1 month ago

So after looking into this a bit more, I’m trying to see how you ever got the original breadcrumb trail. There shouldn’t be any entries after the “current_item” entry. Is this issue visible on a local testbed or on an Internet facing site (where I can take a look at the generated HTML)? Finally, it may be helpful if you did a settings export for Breadcrumb NavXT and either sent it to me, or placed it in pastebin or equivalent service and linked to it here (the forum mods will not like it if you posted the export file here as it’s too large to sit in the forums).

Thread Starter Philipp
(@philippmuenchen)

11 years, 1 month ago

Hi mtekk,

thank you very much for your answer. The site is live and available here: http://goo.gl/o0AjF

I haven’t found a export function in the NavXT-Settings. Is that the right place to search for?! 🙂

Thank you for you amazing work!

Philipp

Plugin Author John Havlik
(@mtekk)

11 years, 1 month ago

You can export your settings from the help menu in the Breadcrumb NavXT settings page. Click on the help tab in the upper right hand corner of the page, in the left hand menu on the dropdown, click Import/Export/Reset. You should be now on the Import/Export/Reset form, press the Export button.

Looking at what’s being generated on the site, I’m definitely going to need to see what the settings look like (I’m seeing a few really goofy things).

Thread Starter Philipp
(@philippmuenchen)

11 years, 1 month ago

Hi mtekk,

thanks again for your reply. 🙂 You’ll find the exported settings here: http://pastebin.com/tt20a7XF

I really hope that we’ll find the problem. 🙂

Thanks and cheers!

Philipp

Plugin Author John Havlik
(@mtekk)

11 years, 1 month ago

Thanks for posting that, after looking at your settings I’m seeing a few things that look wrong.

For your custom post types about-n7w, n7w-worldtour, n7w-page, n7w-ancient, n7w-event, and n7w-finalist the breadcrumb template for each has a static link within it. Not sure exactly what you were trying to achieve with those static links, probably should use the post root option for those types if you need to inject a page hierarchy in front of those breadcrumbs.

It also appears that several of the custom post settings are getting represented multiple times in the export. I am not sure if that is something with your setup or a bug in the code. I will look into that to determine where the issue lies.
Thread Starter Philipp
(@philippmuenchen)

11 years, 1 month ago
Thank you again for your answer.

The “Root Page” option isn’t a solution for me because here I’m just able to choose a “regular” page. But I set up different custom post types with pages to keep the overview.
So I manually added the links into the NavXT-Template of every single post type to achieve logical breadcrumbs. That worked fine until you added the
- -filter into the Plugin. 😉
Thread Starter Philipp
(@philippmuenchen)

11 years, 1 month ago

Hi mtekk,

I just switched off breadcrumbs because I wasn’t able to get my old settings back after downgrading the Plugin. Is there a function or something else which protects the Plugin to remove the “li” tags. That would be very useful to use the breadcrumbs as I used to.

Thank you again for your help and amazing work!

Philipp

Plugin Author John Havlik
(@mtekk)

11 years, 1 month ago

The function that removes the li tags is wp_kses, there are two calls to it in the plugin. In the next version I’ll likely add the li tag into the acceptable tags list for wp_kses. For the time being, either modify the accepted tags list/array (both instances) or remove wp_kses calls. The next version will also have a filter call for that array so end users can easily add more acceptable tags without modifying the plugin (this should have been there before, can’t remember why I left it out).

Thread Starter Philipp
(@philippmuenchen)

11 years ago

You’re the best! Thank you! I got everything as I wanted. 😉

Viewing 12 replies - 1 through 12 (of 12 total)

The topic ‘as seperation’ is closed to new replies.