raviriley / agency-jekyll-theme

Jekyll version of the newest Agency Bootstrap theme, plus new features: Google Analytics, Markdown support, custom pages, and more!
https://raviriley.github.io/agency-jekyll-theme-starter/
MIT License
347 stars 689 forks source link

delete extraneous </li> #26

Closed ajaxify closed 3 years ago

ajaxify commented 3 years ago

This is a bug fix.

Summary

navheader.html had an extra closing </li> outside of the loop for the navbar.

Testing

Copied the navheader.html into my project's _includes/ directory and removed the line. Verified that this removed the literal </li> from my page's output.

raviriley commented 3 years ago

I changed the spacing so it's clear that the </li> is definitely not extraneous

raviriley commented 3 years ago

Something strange is happening on your repo, but if you inspect the code of the site deployed from the starter template, it does not have the issue that's happening on your site.