Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

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: Created April 3, 2017, 9:39 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« includes/navbar.tmpl ('K') | « static/css/main.css ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
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
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
41 {
42 margin: 0;
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;
45 }
46
47 #navbar h1 a
48 {
49 color: $primary-fg;
50 text-decoration: none;
51 font-weight: 300;
52 }
53
54 #navbar h1 a strong
55 {
56 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 }
59
40 @media(min-width: $sidebar-breakpoint-x) 60 @media(min-width: $sidebar-breakpoint-x)
41 { 61 {
42 html[dir="rtl"] #navbar-logo 62 html[dir="rtl"] #navbar-logo
43 { 63 {
44 margin-right: $sidebar-width; 64 margin-right: $sidebar-width;
45 } 65 }
46 html[dir="ltr"] #navbar-logo 66 html[dir="ltr"] #navbar-logo
47 { 67 {
48 margin-left: $sidebar-width; 68 margin-left: $sidebar-width;
49 } 69 }
50 } 70 }
51 71
52 @media(max-width: $sidebar-breakpoint-x - 1) 72 @media(max-width: $sidebar-breakpoint-x - 1)
53 { 73 {
54 .js #navbar 74 .js #navbar
55 { 75 {
56 display: none; 76 display: none;
57 } 77 }
58 } 78 }
OLDNEW
« includes/navbar.tmpl ('K') | « static/css/main.css ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld