ctubbsii commented on code in PR #398:
URL: https://github.com/apache/accumulo-website/pull/398#discussion_r1309160221
##########
pages/downloads.md:
##########
@@ -86,15 +86,14 @@ available in a future update to this site.
{% assign lnkFile = 'accumulo-' | append: linkVers | append: '-' | append:
srcbin | append: '.tar.gz' %}
{% assign lnkSuffix = '/accumulo/' | append: linkVers | append: '/' | append:
lnkFile %}
<div class="row btn-group" style="margin-left: 20px; margin-bottom: 5px;
display: block">
- <div class="col btn-group">
+ <div class="col-auto btn-group">
<a {{btnDownloadStyle}} href="{{closerLink}}{{lnkSuffix}}"
link-suffix="{{lnkSuffix}}">{{lnkFile}}{{glyphSave}}</a>
- </div><div class="col btn-group">
Review Comment:
You removed the separate div, but if I remember correctly, this was here to
ensure the entire block wrapped as a group on screen width changes. Or maybe it
was because it was for padding/spacing tweaks. I don't think it was extraneous,
though. Same situation in the others below here.
##########
_includes/nav.html:
##########
@@ -1,64 +1,62 @@
-<nav class="navbar navbar-default navbar-fixed-top">
+<nav class="navbar navbar-expand-lg navbar-light fixed-top bg-light">
<div class="container">
- <div class="navbar-header">
- <button type="button" class="navbar-toggle" data-toggle="collapse"
data-target="#navbar-items">
- <span class="sr-only">Toggle navigation</span>
- <span class="icon-bar"></span>
- <span class="icon-bar"></span>
- <span class="icon-bar"></span>
- </button>
- <a href="{{ site.baseurl }}/"><img id="nav-logo" alt="Apache Accumulo"
class="img-responsive" src="{{ site.baseurl }}/images/accumulo-logo.png"
width="200"
- /></a>
- </div>
+ <a class="navbar-brand" href="{{ site.baseurl }}/">
+ <img alt="Apache Accumulo" id="nav-logo" src="{{ site.baseurl
}}/images/accumulo-logo.png" width="200">
+ </a>
+ <button class="navbar-toggler" type="button" data-bs-toggle="collapse"
data-bs-target="#navbar-items">
+ <span class="navbar-toggler-icon"></span>
+ </button>
<div class="collapse navbar-collapse" id="navbar-items">
- <ul class="nav navbar-nav">
- <li class="nav-link"><a href="{{ site.baseurl
}}/downloads">Download</a></li>
- <li class="nav-link"><a href="{{ site.baseurl }}/tour">Tour</a></li>
- <li class="dropdown">
- <a class="dropdown-toggle" data-toggle="dropdown"
href="#">Releases<span class="caret"></span></a>
+ <ul class="navbar-nav me-auto">
+ <li class="nav-item"><a class="nav-link" href="{{ site.baseurl
}}/downloads">Download</a></li>
+ <li class="nav-item"><a class="nav-link" href="{{ site.baseurl
}}/tour">Tour</a></li>
+ <li class="nav-item dropdown">
+ <a class="nav-link dropdown-toggle" href="#" role="button"
data-bs-toggle="dropdown">Releases</a>
<ul class="dropdown-menu">
- <li><a href="{{ site.baseurl }}/release/accumulo-3.0.0/">3.0.0
(Latest non-LTM)</a></li>
- <li><a href="{{ site.baseurl }}/release/accumulo-2.1.2/">2.1.2
(Latest LTM)</a></li>
- <li><a href="{{ site.baseurl }}/release/accumulo-1.10.3/">1.10.3
(Legacy LTM)</a></li>
- <li><a href="{{ site.baseurl }}/release/">Archive</a></li>
+ <li><a class="dropdown-item" href="{{ site.baseurl
}}/release/accumulo-3.0.0/">3.0.0 (Latest non-LTM)</a></li>
+ <li><a class="dropdown-item" href="{{ site.baseurl
}}/release/accumulo-2.1.2/">2.1.2 (Latest LTM)</a></li>
+ <li><a class="dropdown-item" href="{{ site.baseurl
}}/release/accumulo-1.10.3/">1.10.3 (Legacy LTM)</a></li>
+ <li><a class="dropdown-item" href="{{ site.baseurl
}}/release/">Archive</a></li>
</ul>
</li>
- <li class="dropdown">
- <a class="dropdown-toggle" data-toggle="dropdown"
href="#">Documentation<span class="caret"></span></a>
+ <li class="nav-item dropdown">
+ <a class="nav-link dropdown-toggle" href="#" role="button"
data-bs-toggle="dropdown">Documentation</a>
<ul class="dropdown-menu">
- <li><a href="{{ site.docs_baseurl }}">User Manual (2.x)</a></li>
- <li><a href="{{ site.baseurl }}/docs/2.x/apidocs">Javadocs
(2.x)</a></li>
- <li><a href="{{ site.baseurl }}/api">Public API</a></li>
- <li><a href="{{ site.baseurl }}/quickstart-1.x">Quickstart
(1.x)</a></li>
- <li><a href="{{ site.baseurl }}/accumulo2-maven-plugin">Accumulo
Maven Plugin</a></li>
- <li><a href="{{ site.baseurl
}}/1.10/accumulo_user_manual.html">User Manual (1.10)</a></li>
- <li><a href="{{ site.baseurl }}/1.10/apidocs">Javadocs
(1.10)</a></li>
- <li><a href="{{ site.baseurl }}/external-docs">External
Docs</a></li>
- <li><a href="{{ site.baseurl }}/docs-archive/">Archive</a></li>
+ <li><a class="dropdown-item" href="{{ site.docs_baseurl }}">User
Manual (2.x)</a></li>
+ <li><a class="dropdown-item" href="{{ site.baseurl
}}/docs/2.x/apidocs">Javadocs (2.x)</a></li>
+ <li><a class="dropdown-item" href="{{ site.baseurl }}/api">Public
API</a></li>
+ <li><a class="dropdown-item" href="{{ site.baseurl
}}/quickstart-1.x">Quickstart (1.x)</a></li>
+ <li><a class="dropdown-item" href="{{ site.baseurl
}}/accumulo2-maven-plugin">Accumulo Maven Plugin</a></li>
+ <li><a class="dropdown-item" href="{{ site.baseurl
}}/1.10/accumulo_user_manual.html">User Manual (1.10)</a></li>
+ <li><a class="dropdown-item" href="{{ site.baseurl
}}/1.10/apidocs">Javadocs (1.10)</a></li>
+ <li><a class="dropdown-item" href="{{ site.baseurl
}}/external-docs">External Docs</a></li>
+ <li><a class="dropdown-item" href="{{ site.baseurl
}}/docs-archive/">Archive</a></li>
</ul>
</li>
- <li class="dropdown">
- <a class="dropdown-toggle" data-toggle="dropdown"
href="#">Community<span class="caret"></span></a>
+ <li class="nav-item dropdown">
+ <a class="nav-link dropdown-toggle" href="#" role="button"
data-bs-toggle="dropdown">Community</a>
<ul class="dropdown-menu">
- <li><a href="{{ site.baseurl }}/contact-us">Contact Us</a></li>
- <li><a href="{{ site.baseurl }}/how-to-contribute">How To
Contribute</a></li>
- <li><a href="{{ site.baseurl }}/people">People</a></li>
- <li><a href="{{ site.baseurl }}/related-projects">Related
Projects</a></li>
+ <li><a class="dropdown-item" href="{{ site.baseurl
}}/contact-us">Contact Us</a></li>
+ <li><a class="dropdown-item" href="{{ site.baseurl
}}/how-to-contribute">How To Contribute</a></li>
+ <li><a class="dropdown-item" href="{{ site.baseurl
}}/people">People</a></li>
+ <li><a class="dropdown-item" href="{{ site.baseurl
}}/related-projects">Related Projects</a></li>
</ul>
</li>
- <li class="nav-link"><a href="{{ site.baseurl
}}/search">Search</a></li>
+ <li class="nav-item"><a class="nav-link" href="{{ site.baseurl
}}/search">Search</a></li>
</ul>
- <ul class="nav navbar-nav navbar-right">
- <li class="dropdown">
- <a class="dropdown-toggle" data-toggle="dropdown" href="#"><img
alt="Apache Software Foundation"
src="https://www.apache.org/foundation/press/kit/feather.svg"; width="15"/><span
class="caret"></span></a>
+ <ul class="navbar-nav ms-auto">
+ <li class="nav-item dropdown">
+ <a class="nav-link dropdown-toggle" href="#" role="button"
data-bs-toggle="dropdown">
+ <img alt="Apache Software Foundation"
src="https://www.apache.org/foundation/press/kit/feather.svg"; width="15"/>
+ </a>
<ul class="dropdown-menu">
- <li><a href="https://www.apache.org";>Apache Homepage <i
class="fa-solid fa-up-right-from-square"></i></a></li>
- <li><a href="https://www.apache.org/licenses/";>License <i
class="fa-solid fa-up-right-from-square"></i></a></li>
- <li><a
href="https://www.apache.org/foundation/sponsorship";>Sponsorship <i
class="fa-solid fa-up-right-from-square"></i></a></li>
- <li><a href="https://www.apache.org/security";>Security <i
class="fa-solid fa-up-right-from-square"></i></a></li>
- <li><a href="https://www.apache.org/foundation/thanks";>Thanks <i
class="fa-solid fa-up-right-from-square"></i></a></li>
- <li><a
href="https://www.apache.org/foundation/policies/conduct";>Code of Conduct <i
class="fa-solid fa-up-right-from-square"></i></a></li>
- <li><a
href="https://www.apache.org/events/current-event.html";>Current Event <i
class="fa-solid fa-up-right-from-square"></i></a></li>
+ <li><a class="dropdown-item" href="https://www.apache.org";>Apache
Homepage <i class="fa-solid fa-up-right-from-square"></i></a></li>
Review Comment:
Wouldn't the `<li>` be the dropdown item? Not the link inside it?
--
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.
To unsubscribe, e-mail: notifications-unsubscr...@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
![https://www.apache.org/foundation/press/kit/feather.svg"](https://www.apache.org/foundation/press/kit/feather.svg"){kind=link}