search

gsuez / master-bootstrap-3

Joomla 3.x Template with Bootstrap 3
https://www.masterbootstrap.com/
142 stars 90 forks source link

Missing </div>, misordered </div> #16

Closed techgump closed 8 years ago

techgump commented 8 years ago

There is an issue with the html framework regarding the order of closing elements. In the head, it's missing a closing </ div >. And another closing </ div > for opening < div id="wrap" > closes inside of the < section >, despite the wrap div opening before < section >. Errors can be viewed with an html validator.

gsuez commented 8 years ago

Hi! from validator.w3.org, that issue is not listed.

techgump commented 8 years ago

Well, the validator may not explicitly be stating so, and I don't remember exactly what the errors stated, but from my findings they were due to closing tag issues. I removed all modules and content when testing, leaving just the template to render. The easiest one to verify just by looking at index.php of template, is the < div id="wrap" > which you will see opens just after the < body > tag, well before the < section > tag opens. But the closing tag occurs before the < /section > closing, hence there is no < /div> tag after the < /section>. Should be < body> < div id="wrap" > < header>... < /header> < section >... < /section> < /div > < /body >

but instead is: < body> < div id="wrap" > < header>... < /header> < section >... < /div > < /section> < /body >