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">＋</label></dt>
Review comment:
`<label for="footer-toggle-overview">＋</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 `＋` in content.
- `<label for="footer-toggle-overview" id="expand">＋</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:
[email protected]
