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

Unified Diff: static/scss/layout/_sidebar.scss

Issue 29453600: Issue 4961 - Fix card group field alignment on acceptableads.com (Closed) Base URL: https://hg.adblockplus.org/web.acceptableads.com
Patch Set: Rebase Created July 5, 2017, 9:19 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 | « static/scss/layout/_navbar.scss ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: static/scss/layout/_sidebar.scss
===================================================================
--- a/static/scss/layout/_sidebar.scss
+++ b/static/scss/layout/_sidebar.scss
@@ -32,16 +32,24 @@
text-align: inherit;
}
li::before
{
@extend .unstyle-list;
}
+ a,
+ a%active
+ {
+ display: block;
+ text-decoration: none;
+ color: inherit;
+ }
+
// #sidebar-brank blocks
// #sidebar-logo is visible on desktop
// #sidebar-open, #sidebar-close, and #sidebar-tittle are visible on mobile
#sidebar-brand
{
height: $lg;
line-height: $lg;
@@ -59,17 +67,17 @@
width: $md;
height: $md;
vertical-align: middle;
}
}
#sidebar-open,
#sidebar-close,
- .site-title
+ #sidebar-title
{
display: none;
}
#sidebar-close img
{
width: 20px;
height: 20px;
@@ -77,27 +85,16 @@
#sidebar-open img
{
width: 36px;
height: 18px;
}
}
-#sidebar-menus
-{
- a,
- a%active
- {
- display: block;
- text-decoration: none;
- color: inherit;
- }
-}
-
// Positioning the sidebar on desktop
@media(min-width: $sidebar-breakpoint-x)
{
// since the primary navigation collapses only when javascript is enabled
// we must position this absolute by default so that it can scroll
#sidebar
{
position: absolute;
@@ -161,17 +158,17 @@
html[dir="rtl"].js #sidebar
{
text-align: right;
}
#sidebar
{
// show sidebar navbar title and open button by default
- .site-title,
+ #sidebar-title,
#sidebar-open
{
display: block;
}
// Toggle open/close buttons
&.open
{
@@ -190,56 +187,63 @@
#sidebar-brand
{
background-color: $inverted-fg;
}
#sidebar-open,
#sidebar-close,
#sidebar-logo,
- .site-title
+ #sidebar-title
{
padding: 0 $sm;
}
#sidebar-open,
#sidebar-close,
- .site-title
+ #sidebar-title
{
height: $lg;
line-height: $lg;
}
#sidebar-open,
#sidebar-close
{
width: $lg;
min-width: $lg;
margin: 0;
color: $inverted-fg;
border: none;
background-color: $inverted-bg;
font-size: $md;
}
+
+ #sidebar-title img
+ {
+ display: inline-block;
+ height: $sm;
+ vertical-align: middle;
+ }
}
// align blocks left/right ltr/rtl in navbar
html[dir="ltr"] #sidebar-open,
html[dir="ltr"] #sidebar-close,
html[dir="ltr"] #sidebar-logo,
- html[dir="ltr"] #sidebar .site-title
+ html[dir="ltr"] #sidebar-title
{
float: left;
}
html[dir="rtl"] #sidebar-open,
html[dir="rtl"] #sidebar-close,
html[dir="rtl"] #sidebar-logo,
- html[dir="rtl"] #sidebar .site-title
+ html[dir="rtl"] #sidebar-title
{
float: right;
}
}
// Primary navigation //////////////////////////////////////////////////////////
#primary-navigation
« no previous file with comments | « static/scss/layout/_navbar.scss ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld