Side by Side Diff: static/scss/layout/_navbar.scss
Issue 29401619:
Issue 4963 - Wrong size used for text logo on acceptableads.com (Closed)
Base URL: https://hg.adblockplus.org/web.acceptableads.com
Patch Set: Addressed comments
Created May 22, 2017, 7:08 p.m.
| Left: | ||
| Right: |
Use n/p to move between diff chunks;
N/P to move between comments.
« includes/navbar.tmpl ('K')
|
« static/css/main.css ('k')
|
static/scss/layout/_sidebar.scss » ('j')
|
static/scss/layout/_sidebar.scss » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| OLD | NEW |
|---|---|
| 1 // This file is part of acceptableads.org. | 1 // This file is part of acceptableads.org. |
| 2 // Copyright (C) 2016 Eyeo GmbH | 2 // Copyright (C) 2016 Eyeo GmbH |
| 3 // | 3 // |
| 4 // acceptableads.org is free software: you can redistribute it and/or modify | 4 // acceptableads.org is free software: you can redistribute it and/or modify |
| 5 // it under the terms of the GNU General Public License as published by | 5 // it under the terms of the GNU General Public License as published by |
| 6 // the Free Software Foundation, either version 3 of the License, or | 6 // the Free Software Foundation, either version 3 of the License, or |
| 7 // (at your option) any later version. | 7 // (at your option) any later version. |
| 8 // | 8 // |
| 9 // acceptableads.org is distributed in the hope that it will be useful, | 9 // acceptableads.org is distributed in the hope that it will be useful, |
| 10 // but WITHOUT ANY WARRANTY; without even the implied warranty of | 10 // but WITHOUT ANY WARRANTY; without even the implied warranty of |
| (...skipping 19 matching lines...) Expand all Loading... | |
| 30 } | 30 } |
| 31 | 31 |
| 32 .js #navbar | 32 .js #navbar |
| 33 { | 33 { |
| 34 position: fixed; | 34 position: fixed; |
| 35 z-index: 1; | 35 z-index: 1; |
| 36 top: 0; | 36 top: 0; |
| 37 left: 0; | 37 left: 0; |
| 38 } | 38 } |
| 39 | 39 |
| 40 .site-title | |
| 41 { | |
| 42 margin: 0; | |
| 43 font-size: 17px; | |
| 44 line-height: 64px; | |
| 45 } | |
| 46 | |
| 47 .site-title a | |
| 48 { | |
| 49 color: $primary-fg; | |
| 50 text-decoration: none; | |
| 51 font-weight: 300; | |
| 52 } | |
| 53 | |
| 54 .site-title .non-space | |
| 55 { | |
| 56 display: inline-block; | |
| 57 width: 1px; | |
| 58 } | |
| 59 | |
| 60 .site-title a strong | |
| 61 { | |
| 62 font-weight: 400; | |
| 63 } | |
| 64 | |
| 40 @media(min-width: $sidebar-breakpoint-x) | 65 @media(min-width: $sidebar-breakpoint-x) |
| 41 { | 66 { |
| 42 html[dir="rtl"] #navbar-logo | 67 html[dir="rtl"] #navbar-logo |
|
saroyanm
2017/06/01 13:09:54
"#navbar-logo" Item is a duplication, let's use ".
juliandoucette
2017/06/07 15:42:52
Done.
Good find.
| |
| 43 { | 68 { |
| 44 margin-right: $sidebar-width; | 69 margin-right: $sidebar-width; |
| 45 } | 70 } |
| 46 html[dir="ltr"] #navbar-logo | 71 html[dir="ltr"] #navbar-logo |
| 47 { | 72 { |
| 48 margin-left: $sidebar-width; | 73 margin-left: $sidebar-width; |
| 49 } | 74 } |
| 50 } | 75 } |
| 51 | 76 |
| 52 @media(max-width: $sidebar-breakpoint-x - 1) | 77 @media(max-width: $sidebar-breakpoint-x - 1) |
| 53 { | 78 { |
| 54 .js #navbar | 79 .js #navbar |
| 55 { | 80 { |
| 56 display: none; | 81 display: none; |
| 57 } | 82 } |
| 58 } | 83 } |
| OLD | NEW |
« includes/navbar.tmpl ('K')
|
« static/css/main.css ('k')
|
static/scss/layout/_sidebar.scss » ('j')
|
static/scss/layout/_sidebar.scss » ('J')
