Unified Diff: includes/components/navbar.html
Issue 29587584:
Issue 5635 - Implement website-default navbar component (Closed)
Base URL: https://hg.adblockplus.org/website-defaults
Patch Set: Rebased
Created Oct. 31, 2017, 2:07 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
pages/index.html » ('j')
|
static/js/navbar.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: includes/components/navbar.html |
| =================================================================== |
| new file mode 100644 |
| --- /dev/null |
| +++ b/includes/components/navbar.html |
| @@ -0,0 +1,20 @@ |
| +<div class="navbar"> |
| + <div class="container"> |
| + <h1 class="navbar-branding"> |
|
juliandoucette
2017/10/31 15:18:37
I don't know whether it's better to let the site t
juliandoucette
2017/10/31 15:30:44
AFAICT page heading should be <h1> and navbar bran
ire
2017/11/02 11:15:25
I think you're right. I haven't found a documented
|
| + <a href="index">Website Defaults</a> |
| + </h1> |
| + |
| + <button class="toggle-navbar-collapse"> |
| + <span class="open-label">Open</span> |
| + <span class="sr-only">/</span> |
| + <span class="close-label">Close</span> |
| + Menu |
| + </button> |
| + |
| + <div class="navbar-collapse"> |
| + <nav> |
| + <a href="index">Navbar Menu Link</a> |
| + </nav> |
| + </div> |
| + </div> |
| +</div> |
« no previous file with comments
|
« no previous file
|
pages/index.html » ('j')
|
static/js/navbar.js » ('J')
