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

Unified Diff: static/scss/components/_article.scss

Issue 29633564: Issue 5765 - Remove custom spacing units in help.eyeo.com (Closed) Base URL: https://hg.adblockplus.org/help.eyeo.com
Patch Set: Created Dec. 8, 2017, 9:55 a.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
Index: static/scss/components/_article.scss
===================================================================
--- a/static/scss/components/_article.scss
+++ b/static/scss/components/_article.scss
@@ -15,29 +15,29 @@
// along with help.eyeo.com. If not, see <http://www.gnu.org/licenses/>.
/*******************************************************************************
* Article
******************************************************************************/
.article.card
{
- padding: $md;
+ padding: $small-space;
@media (min-width: $phablet-breakpoint)
{
- padding: $lg;
+ padding: $medium-space;
}
}
.article-heading,
.article-browser-selector
{
- padding-bottom: $md;
- margin-bottom: $md;
+ padding-bottom: 15px;
+ margin-bottom: 15px;
ire 2017/12/08 09:59:03 These two needed to be static values since the fon
juliandoucette 2017/12/11 15:21:40 Acknowledged.
border-bottom: 1px dotted $gray;
}
.article-heading .has-pre-icon
{
@media (max-width: $phablet-breakpoint - 1)
{
padding: 0;
@@ -46,17 +46,17 @@
.article-heading .pre-icon
{
display: none;
@media (min-width: $phablet-breakpoint)
{
display: block;
- top: 0.25em; // Align icon to first line of heading
+ top: $small-space / 4; // Align icon to first line of heading
margin-top: 0;
}
}
.article-browser-selector
{
@extend .clearfix;
display: none;

Powered by Google App Engine
This is Rietveld