Breadcrumb HTML5 error #54

Open
kafran opened this Issue Jun 25, 2012 · 3 comments

Projects

None yet

3 participants

@kafran
kafran commented Jun 25, 2012

The generated HTML:

<ul class="breadcrumb">
     <li><a href="http://">Home</a></li><span class="divider">/</span>
     <a href="http://category/cat-a/" title="View all posts in Cat A">Cat A</a><span class="divider">/</span>
     <li class="active">Readability Test</li>
</ul>

Issues:

  • HTML does not accept a in
      , please put the 'delimiter' before , do not forget the blank spaces ;)
    • I don't know why the "Cat A" link is being generated without a
    • , ofcourse this is not accepted by W3 too.

    I tryed to fix it myself but I dont know php. =/
    Thank for your work.

@kafran
kafran commented Jun 25, 2012

Argg markdown =/

Issues:

  • HTML5 does not accept a <span> in <ul>, please put the 'delimiter' before </li>, do not forget the blank spaces ;)
  • I don't know why the "Cat A" link is being generated without a <li>, ofcourse this is not accepted by W3 too.

I tryed to fix it myself but I dont know php. =/
Thank for your work.

@diazemiliano

Hi can correct this editing the file "functions.php" in line: 398.

  1. Just go to Admin > Appearance > Editor
  2. Search the file "functions.php" in the template file list.
  3. With your browser find the string "divider" and will see:
$delimiter = '<span class="divider">/</span>';

Just change to:

$delimiter = '<li><span class="divider">/</span></li>';

Save and that's all. Good luck!!!

@rachelbaker
Owner

@diazemiliano

Thank you for posting a response with the code fix. I will add to the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment