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: Use padding on navbar link, optimise svg
Created Nov. 30, 2017, 2 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,23 @@ |
| +<div class="navbar bg-primary"> |
| + <div class="container"> |
| + <a href="index" class="navbar-branding float-start"> |
| + <img src="/img/png/eyeo-logo.png" srcset="/img/svg/eyeo-logo.svg 2x" alt="eyeo"> |
| + </a> |
| + |
| + <button class="toggle-navbar-collapse float-end"> |
| + <span class="open-label">Open</span> |
|
juliandoucette
2017/12/01 16:30:43
It seems like a toggle button will be common to al
ire
2017/12/08 10:42:21
You're right. Done.
|
| + <span class="sr-only">/</span> |
|
juliandoucette
2017/12/01 16:30:43
I don't think that this makes sense.
- If .open-l
ire
2017/12/08 10:42:21
Yeah I think I didn't get this correctly. It's no
|
| + <span class="close-label">Close</span> |
| + Menu |
| + </button> |
| + |
| + <nav class="navbar-nav navbar-collapse"> |
| + <ul> |
| + <li><a href="index">Navbar Menu Link</a></li> |
| + <li><a href="index">Navbar Menu Link</a></li> |
| + <li><a href="index">Navbar Menu Link</a></li> |
| + <li><a href="index">Navbar Menu Link</a></li> |
| + </ul> |
| + </nav> |
| + </div> |
| +</div> |
« no previous file with comments
|
« no previous file
|
pages/index.html » ('j')
|
static/js/navbar.js » ('J')
