search

justintadlock / breadcrumb-trail

Official home of the Breadcrumb Trail plugin for WordPress.
GNU General Public License v2.0
136 stars 61 forks source link

Fixed 2 validation errors & 1 warning. + SEO #21

Closed azirer closed 8 years ago

azirer commented 8 years ago

Removed one validation warning because of role="navigation" and two errors because of tags inside ul. This way it completely validates at https://validator.nu/ https://validator.w3.org/nu/ and https://developers.google.com/structured-data/testing-tool/ Above tests have been made to pages in 3 level nested Breadcrumbs

Also changed h2 to p for SEO reasons because there is no need on any website to emphasize at the content of show_browse with an h2 tag. Headings should be used only for content that is important and related to the topic of the current page.

justintadlock commented 8 years ago

Please open individual tickets for any specific issues you've found. It will be looked over and discussed. If/When it's decided that there's an issue that should be fixed and how it should be fixed, a PR may then be accepted against the "dev" branch or one of the point branches (the master branch is the latest stable code).