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

Unified Diff: static/css/main.css

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: Changed non-space to word-spacing Created June 14, 2017, 12:02 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « includes/sidebar.tmpl ('k') | static/scss/layout/_navbar.scss » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: static/css/main.css
===================================================================
--- a/static/css/main.css
+++ b/static/css/main.css
@@ -1224,20 +1224,38 @@
margin: 0; }
.js #navbar {
position: fixed;
z-index: 1;
top: 0;
left: 0; }
+.site-title {
+ margin: 0;
+ font-size: 17px;
+ line-height: 64px;
+ word-spacing: 1px; }
saroyanm 2017/06/14 13:20:52 I assume you forgot to generate latest version, as
+
+.site-title a {
+ color: #161616;
+ text-decoration: none;
+ font-weight: 300; }
+
+.site-title .non-space {
+ display: inline-block;
+ width: 1px; }
+
+.site-title a strong {
+ font-weight: 400; }
+
@media (min-width: 1200px) {
- html[dir="rtl"] #navbar-logo {
+ html[dir="rtl"] .site-title {
margin-right: 208px; }
- html[dir="ltr"] #navbar-logo {
+ html[dir="ltr"] .site-title {
margin-left: 208px; } }
@media (max-width: 1199px) {
.js #navbar {
display: none; } }
#sidebar {
text-transform: uppercase;
@@ -1246,45 +1264,46 @@
font-size: 16px;
font-weight: 400;
line-height: 100%; }
#sidebar ul,
#sidebar li {
margin: 0;
padding: 0;
text-align: inherit; }
- #sidebar a,
- #sidebar a:hover,
- #sidebar a:active {
- display: block;
- text-decoration: none;
- color: inherit; }
#sidebar #sidebar-brand {
height: 64px;
line-height: 64px; }
#sidebar #sidebar-logo {
display: block;
padding: 0 32px;
text-align: inherit;
background-color: #8CC63E; }
#sidebar #sidebar-logo img {
width: 32px;
height: 32px;
vertical-align: middle; }
#sidebar #sidebar-open,
#sidebar #sidebar-close,
- #sidebar #sidebar-title {
+ #sidebar .site-title {
display: none; }
#sidebar #sidebar-close img {
width: 20px;
height: 20px; }
#sidebar #sidebar-open img {
width: 36px;
height: 18px; }
+#sidebar-menus a,
+#sidebar-menus a:hover,
+#sidebar-menus a:active {
+ display: block;
+ text-decoration: none;
+ color: inherit; }
+
@media (min-width: 1200px) {
#sidebar {
position: absolute;
top: 0;
width: 208px; }
.js #sidebar {
position: fixed;
z-index: 2;
@@ -1306,57 +1325,53 @@
width: 100%;
height: 64px; }
.js #sidebar.open {
height: 100%; }
html[dir="ltr"].js #sidebar {
text-align: left; }
html[dir="rtl"].js #sidebar {
text-align: right; }
- #sidebar #sidebar-title,
+ #sidebar .site-title,
#sidebar #sidebar-open {
display: block; }
#sidebar.open #sidebar-open {
display: none; }
#sidebar.open #sidebar-close {
display: block; }
#sidebar #sidebar-brand {
background-color: #FFFFFF; }
#sidebar #sidebar-open,
#sidebar #sidebar-close,
#sidebar #sidebar-logo,
- #sidebar #sidebar-title {
+ #sidebar .site-title {
padding: 0 16px; }
#sidebar #sidebar-open,
#sidebar #sidebar-close,
- #sidebar #sidebar-title {
+ #sidebar .site-title {
height: 64px;
line-height: 64px; }
#sidebar #sidebar-open,
#sidebar #sidebar-close {
width: 64px;
min-width: 64px;
margin: 0;
color: #FFFFFF;
border: none;
background-color: #161616;
font-size: 32px; }
- #sidebar #sidebar-title img {
- display: inline-block;
- height: 16px;
- vertical-align: middle; }
html[dir="ltr"] #sidebar-open,
html[dir="ltr"] #sidebar-close,
html[dir="ltr"] #sidebar-logo,
- html[dir="ltr"] #sidebar-title {
+ html[dir="ltr"] #sidebar .site-title {
float: left; }
html[dir="rtl"] #sidebar-open,
html[dir="rtl"] #sidebar-close,
html[dir="rtl"] #sidebar-logo,
- html[dir="rtl"] #sidebar-title {
+ html[dir="rtl"] #sidebar .site-title {
float: right; } }
#primary-navigation li {
position: relative; }
#primary-navigation ul ul li {
color: #8CC63E;
font-size: 14px; }
« no previous file with comments | « includes/sidebar.tmpl ('k') | static/scss/layout/_navbar.scss » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld