[GitHub] [camel-website] AemieJ commented on a change in pull request #443: CAMEL-15360: fix HTML validation issues

[GitBox]

AemieJ commented on a change in pull request #443:
URL: https://github.com/apache/camel-website/pull/443#discussion_r463949285



##########
File path: layouts/partials/footer.html
##########
@@ -6,66 +6,50 @@
     </div>
     <footer>
         <div class="footer">
+            <figure class="logo">
+                <img class="logo" src="/_/img/logo-d.svg" alt="Apache Camel 
Logo"
+                    aria-label="white silhouette of a camel in front of a sand 
dune">
+            </figure>
+            <input id="footer-toggle-overview" type="checkbox" 
title="Show/Hide Overview section" />
             <dl>
-                <figure class="logo">
-                    <img class="logo" src="/_/img/logo-d.svg" alt="Apache 
Camel Logo"
-                        aria-label="white silhouette of a camel in front of a 
sand dune">
-                </figure>
-                <dt>Overview</dt>
-                <input type="checkbox" id="show-overview"/>
-                <label for="show-overview"><img class="show-menu" 
src="/_/img/show-footer-menu.svg"></label>
-                <label for="show-overview"><img class="hide-menu" 
src="/_/img/hide-footer-menu.svg"></label>
-                <div class="footer-menu">
-                    <dd><a href="/blog/">Blog</a></dd>
-                    <dd><a href="/docs/">Documentation</a></dd>
-                    <dd><a href="/community/support/">Community</a></dd>
-                    <dd><a href="/blog/">Download</a></dd>
-                </div>
+                <dt><label for="footer-toggle-overview">Overview</label><label 
for="footer-toggle-overview">&#65291;</label></dt>

Review comment:
       `<label for="footer-toggle-overview">&#65291;</label>`* is better 
included with a separate unique **id** to distinguish as it appears for desktop 
screen as well and the same case for each label with `&#65291;` in content.
   - `<label for="footer-toggle-overview" id="expand">&#65291;</label>`




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org