-
Notifications
You must be signed in to change notification settings - Fork 48
hero and navbar improvements #129
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from 4 commits
25a2578
bc19a15
3973c3f
d97eb62
d4ce849
bebfa51
f3f8013
e6c9a07
496b90b
a8552f5
658ce98
9225016
74b420d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,6 @@ | ||
| # macOS leftovers | ||
| .DS_Store | ||
| .hugo-build.lock | ||
|
|
||
| # Node.js-related assets (e.g. for CSS) | ||
| node_modules/ | ||
|
|
||
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,6 @@ | ||
| { | ||
| "private": true, | ||
| "devDependencies": { | ||
| "bulma": "^0.7.1" | ||
| "bulma": "^0.9.4" | ||
| } | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -7,15 +7,15 @@ | |
| {{- $githubUrl := printf "https://github.com/%s" $githubRepo }} | ||
| {{- $slackChannel := .Site.Params.slackChannel }} | ||
| {{- $projectPages := where .Site.RegularPages "Section" "" }} | ||
| <div class="navbar is-fixed-top{{ if $isHome }} is-dark{{ else }} has-shadow{{ end }}"> | ||
| <div id="mainnav" style="padding:10px" class="navbar is-transparent has-background-pattern is-fixed-top{{ if $isHome }} {{ else }} has-shadow{{ end }}"> | ||
| <div class="container"> | ||
| <div class="navbar-brand"> | ||
| <a class="navbar-item" href="{{ .Site.BaseURL }}"> | ||
| <img src="{{ $logo }}" alt="containerd navbar logo"> | ||
| <img src="{{ $logo }}" alt="containerd navbar logo" style="transform: scale(1.7);"> | ||
| </a> | ||
|
|
||
| {{- if not $isHome }} | ||
| <div class="navbar-item is-docs-title has-text-weight-bold is-hidden-mobile"> | ||
| <div class="navbar-item is-docs-title has-text-weight-bold has-text-white is-hidden-mobile"> | ||
| {{ .Title }} | ||
| </div> | ||
| {{- end }} | ||
|
|
@@ -29,12 +29,12 @@ | |
|
|
||
| <div class="navbar-menu" id="navbarMenu"> | ||
| <div class="navbar-end"> | ||
| <a class="navbar-item has-text-weight-bold" href="/downloads"> | ||
| <a class="navbar-item has-text-weight-bold has-text-white" href="/downloads"> | ||
| Downloads | ||
| </a> | ||
|
|
||
| <div class="navbar-item is-hoverable has-dropdown has-text-weight-bold" href="/docs"> | ||
| <a class="navbar-link"> | ||
| <a class="navbar-link has-text-white"> | ||
| Docs | ||
| </a> | ||
|
|
||
|
|
@@ -48,7 +48,7 @@ | |
| </div> <!-- .navbar-item --> | ||
|
|
||
| <div class="navbar-item is-hoverable has-dropdown has-text-weight-bold"> | ||
| <div class="navbar-link"> | ||
| <div class="navbar-link has-text-white"> | ||
| Project | ||
| </div> | ||
|
|
||
|
|
@@ -78,9 +78,9 @@ | |
| </span> | ||
| </a> | ||
|
|
||
| <a class="button is-borderless has-background-black has-text-white" href="{{ $githubUrl }}"> | ||
| <a class="button is-borderless has-background-white has-text-white" href="{{ $githubUrl }}"> | ||
| <span class="icon"> | ||
| <i class="fab fa-github" aria-hidden="true"></i> | ||
| <i class="fab fa-github" aria-hidden="true" style="color:black"></i> | ||
| </span> | ||
| </a> | ||
|
|
||
|
|
@@ -96,3 +96,16 @@ | |
| </div> <!-- .navbar-menu --> | ||
| </div> <!-- .container --> | ||
| </div> <!-- .navbar --> | ||
|
|
||
| <script src="https://code.jquery.com/jquery-3.5.1.slim.min.js" integrity="sha384-DfXdz2htPH0lsSSs5nCTpuj/zy4C+OGpamoFVy38MVBnE+IbbVYUew+OrCXaRkfj" crossorigin="anonymous"></script> | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This project already includes jQuery 3.6 (see under the themes folder). |
||
| <script src="https://cdn.jsdelivr.net/npm/popper.js@1.16.0/dist/umd/popper.min.js" integrity="sha384-Q6E9RHvbIyZFJoft+2mJbHaEWldlvI9IOYy5n3zV9zzTtmI3UksdQRVvoxMfooAo" crossorigin="anonymous"></script> | ||
| <script src="https://stackpath.bootstrapcdn.com/bootstrap/4.5.0/js/bootstrap.min.js" integrity="sha384-OgVRvuATP1z7JjHLkuOU7Xw704+h835Lr+6QL9UvYjZE3Ipu6Tp75j7Bh/kR0JKI" crossorigin="anonymous"></script> | ||
|
Comment on lines
+100
to
+102
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. CSS frameworks are resource heavy, and add a non-trivial overhead for maintainers. This website was built using bulma. Here you're adding full support for Bootstrap. We don't want that. Use either bulma or Bootstrap, not both. |
||
|
|
||
| <script> | ||
| $(function () { | ||
| $(document).scroll(function () { | ||
| var $nav = $("#mainnav"); | ||
| $nav.toggleClass("scrolled", $(this).scrollTop() > $nav.height()); | ||
| }); | ||
| }); | ||
| </script> | ||
|
Comment on lines
+104
to
+111
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: remove unnecessary space in class string, here and the other instances below.