Fix footer layout and outline

[kronnox]

Test URLs:

• Before: https://main--famous-smoke-cigaradvisor--hlxsites.hlx.page/cigaradvisor/
• After: https://bug-footer-layout--famous-smoke-cigaradvisor--hlxsites.hlx.page/cigaradvisor/

[aem-code-sync]

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

• Re-run PSI checks
• Re-sync branch

Commits

• fdde2f4 ✅ (latest)
• b89c8a0 ✅

[aem-code-sync]

Page	Scores	Audits	Google
/cigaradvisor/

[kronnox]

@bstopp After some research I came to the conclusion it would be best to keep the headings in the footer. I improved the HTML5 document outline to reduce risk of the headings being misinterpreted [1]. This is all according to specification [2], but a lot of screenreaders have never fully adopted the HTML5 document outline [3]. That's why we should still follow the general heading hierarchy of the site (added a hidden footer h2 heading to clean this up [4]).

[1] https://iloveseo.com/seo/john-mueller-explained-how-one-could-use-h2-tags-in-the-footer/
[2] https://html.spec.whatwg.org/multipage/sections.html#the-footer-element
[3] http://html5doctor.com/computer-says-no-to-html5-document-outline/
[4] http://html5doctor.com/outlines/