Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrap comma separator for post_terms in HTML tag #238

Open
rutger1140 opened this issue Apr 10, 2020 · 0 comments
Open

Wrap comma separator for post_terms in HTML tag #238

rutger1140 opened this issue Apr 10, 2020 · 0 comments

Comments

@rutger1140
Copy link

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)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant