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

Side by Side Diff: static/scss/components/_article.scss

Issue 29601607: Issue 5963 - Fix alignment of product icon next to article title on help.eyeo.com (Closed) Base URL: https://hg.adblockplus.org/help.eyeo.com
Patch Set: Created Nov. 8, 2017, 3:37 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
« no previous file with comments | « no previous file | 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 help.eyeo.com. 1 // This file is part of help.eyeo.com.
2 // Copyright (C) 2017 Eyeo GmbH 2 // Copyright (C) 2017 Eyeo GmbH
3 // 3 //
4 // help.eyeo.com is free software: you can redistribute it and/or modify 4 // help.eyeo.com 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 // help.eyeo.com is distributed in the hope that it will be useful, 9 // help.eyeo.com 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 18 matching lines...) Expand all
29 } 29 }
30 30
31 .article-heading, 31 .article-heading,
32 .article-browser-selector 32 .article-browser-selector
33 { 33 {
34 padding-bottom: $md; 34 padding-bottom: $md;
35 margin-bottom: $md; 35 margin-bottom: $md;
36 border-bottom: 1px dotted $gray; 36 border-bottom: 1px dotted $gray;
37 } 37 }
38 38
39 .article-heading .has-pre-icon
40 {
41 @media (max-width: $phablet-breakpoint + 1)
juliandoucette 2017/11/09 14:22:30 +1 should be -1. (See 577px width)
ire 2017/11/10 09:44:25 Oops you're right. I always mix up whether it's +1
42 {
43 padding: 0;
44 }
45 }
46
47 .article-heading .pre-icon
48 {
49 display: none;
50
51 @media (min-width: $phablet-breakpoint)
52 {
53 display: block;
54 top: 0.25em; // Align icon to first line of heading
55 margin-top: 0;
56 }
57 }
58
39 .article-browser-selector 59 .article-browser-selector
40 { 60 {
41 @extend .clearfix; 61 @extend .clearfix;
42 display: none; 62 display: none;
43 } 63 }
44 64
45 .js .article-browser-selector 65 .js .article-browser-selector
46 { 66 {
47 display: block; 67 display: block;
48 } 68 }
(...skipping 11 matching lines...) Expand all
60 display: none; 80 display: none;
61 } 81 }
62 82
63 @each $browser in $supported-browsers 83 @each $browser in $supported-browsers
64 { 84 {
65 .js .ua-#{$browser} .platform-#{$browser} 85 .js .ua-#{$browser} .platform-#{$browser}
66 { 86 {
67 display: block; 87 display: block;
68 } 88 }
69 } 89 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld