mtekk / Breadcrumb-NavXT

The longest maintained Breadcrumb plugin for WordPress
https://mtekk.us/code/breadcrumb-navxt
51 stars 25 forks source link

Wrap comma separator for post_terms in HTML tag #238

Open rutger1140 opened 4 years ago

rutger1140 commented 4 years ago

I've found out when using multiple post terms, there is a comma separator added automagically. This is fine, except for the fact that this element cannot be reached with CSS styling.

I would like to propose changing:

  //Everything but the first term needs a comma separator
  if($is_first == false)
  {
    $title .= ', ';
  }

to something like this:

  //Everything but the first term needs a comma separator
  if($is_first == false)
  {
    $title .= '<span class="separator">, </span>';
  }

When wrapping it in a HTML tag, it can be altered. (In my case hidden on small screens)