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

Unified 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.
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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: static/scss/components/_article.scss
===================================================================
--- a/static/scss/components/_article.scss
+++ b/static/scss/components/_article.scss
@@ -31,16 +31,36 @@
.article-heading,
.article-browser-selector
{
padding-bottom: $md;
margin-bottom: $md;
border-bottom: 1px dotted $gray;
}
+.article-heading .has-pre-icon
+{
+ @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
+ {
+ padding: 0;
+ }
+}
+
+.article-heading .pre-icon
+{
+ display: none;
+
+ @media (min-width: $phablet-breakpoint)
+ {
+ display: block;
+ top: 0.25em; // Align icon to first line of heading
+ margin-top: 0;
+ }
+}
+
.article-browser-selector
{
@extend .clearfix;
display: none;
}
.js .article-browser-selector
{
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld