Delta Between Two Patch Sets: 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
Left Patch Set:
Created April 3, 2017, 9:39 p.m.
Right Patch Set: Changed non-space to word-spacing
Created June 14, 2017, 12:02 p.m.
| Left: | ||
| Right: |
Use n/p to move between diff chunks;
N/P to move between comments.
« static/css/main.css ('K')
|
« static/css/main.css ('k')
|
static/scss/layout/_sidebar.scss » ('j')
|
no next file with change/comment »
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')
| LEFT | RIGHT |
|---|---|
| 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 #navbar h1 | 40 .site-title |
| 41 { | 41 { |
| 42 margin: 0; | 42 margin: 0; |
| 43 font-size: 17px; | 43 font-size: 17px; |
|
saroyanm
2017/04/28 10:19:48
Detail: in ticket it's mentioned 16px, I assume yo
juliandoucette
2017/05/22 19:18:33
Done.
| |
| 44 line-height: 64px; | 44 line-height: 64px; |
| 45 word-spacing: -2px; | |
| 45 } | 46 } |
| 46 | 47 |
| 47 #navbar h1 a | 48 .site-title a |
| 48 { | 49 { |
| 49 color: $primary-fg; | 50 color: $primary-fg; |
| 50 text-decoration: none; | 51 text-decoration: none; |
| 51 font-weight: 300; | 52 font-weight: 300; |
| 52 } | 53 } |
| 53 | 54 |
| 54 #navbar h1 a strong | 55 .site-title a strong |
| 55 { | 56 { |
| 56 font-weight: 400; | 57 font-weight: 400; |
| 57 margin-left: -2px; | |
|
saroyanm
2017/04/28 10:19:48
Is there a reason why you are not removing the spa
juliandoucette
2017/05/22 19:18:33
Acknowledged.
We determined that removing the spa
| |
| 58 } | 58 } |
| 59 | 59 |
| 60 @media(min-width: $sidebar-breakpoint-x) | 60 @media(min-width: $sidebar-breakpoint-x) |
| 61 { | 61 { |
| 62 html[dir="rtl"] #navbar-logo | 62 html[dir="rtl"] .site-title |
| 63 { | 63 { |
| 64 margin-right: $sidebar-width; | 64 margin-right: $sidebar-width; |
| 65 } | 65 } |
| 66 html[dir="ltr"] #navbar-logo | 66 html[dir="ltr"] .site-title |
| 67 { | 67 { |
| 68 margin-left: $sidebar-width; | 68 margin-left: $sidebar-width; |
| 69 } | 69 } |
| 70 } | 70 } |
| 71 | 71 |
| 72 @media(max-width: $sidebar-breakpoint-x - 1) | 72 @media(max-width: $sidebar-breakpoint-x - 1) |
| 73 { | 73 { |
| 74 .js #navbar | 74 .js #navbar |
| 75 { | 75 { |
| 76 display: none; | 76 display: none; |
| 77 } | 77 } |
| 78 } | 78 } |
| LEFT | RIGHT |
« static/css/main.css ('k')
|
static/scss/layout/_sidebar.scss » ('j')
|
Toggle Intra-line Diffs ('i')
|
Expand Comments ('e')
|
Collapse Comments ('c')
|
Toggle Comments ('s')
