Issue #2226213 by svenryen, Manuel Garcia: Split the right sidebar region into...

Issue #2226213 by svenryen, Manuel Garcia: Split the right sidebar region into stacked 'Major' and 'Minor' region

bluecheese.info

@@ -6,6 +6,7 @@ core = 7.x
 
 ; -------- regions ---------------
 regions[page_tools] = Page Tools
+regions[sidebar_first] = Sidebar first
 regions[sidebar_second] = Sidebar second
 regions[highlighted] = Highlighted
 regions[navigation] = Navigation

sass/partials/common/_block.scss

-#aside {
+#aside,
+#sidebar-first {
   .block {
     background: $pale-yellow;
   }

sass/partials/common/_book.scss

 // Book pages
 .book-navigation {
   clear: both;
-  
+
   ul.menu,
   .block & ul.menu {
     border-top: 1px solid nth($greys, 4);
@@ -16,7 +16,10 @@
 }
 
 // Sidebar blocks on book pages
-#aside #block-drupalorg-handbook-meta-sidebar,
-#aside #block-drupalorg-handbook-about-page {
-  background: $white;
+#aside,
+#sidebar-first {
+  #block-drupalorg-handbook-meta-sidebar,
+  #block-drupalorg-handbook-about-page {
+    background: $white;
+  }
 }

sass/partials/common/_content.scss

@@ -17,7 +17,8 @@
 #content-top-region,
 #content,
 #content-bottom-region,
-#aside-region {
+#aside-region,
+#sidebar-first-region {
   margin-bottom: 13px;
 }
 
@@ -33,7 +34,7 @@
 
 /* Submitted time on nodes and comments */
 .submitted time {
-  font-style: italic; 
+  font-style: italic;
 }
 
 /* todo: check what is this */

sass/partials/common/_lists.scss

@@ -72,14 +72,18 @@ ol {
     list-style-type: upper-alpha;
   }
 }
-#aside .block ol {
-  list-style: none;
-  padding-left: 0;
+#aside,
+#sidebar-first {
+  .block ol {
+    list-style: none;
+    padding-left: 0;
+  }
 }
-
 .profile .item-list li,
 #aside .block .item-list li,
 #aside .block .links li,
+#sidebar-first .block .item-list li,
+#sidebar-first .block .links li,
 #homebox .block .menu li,
 #homebox .block .item-list li {
   line-height: 1.2em;

sass/partials/common/_page.scss

@@ -8,11 +8,15 @@
     @include span-columns(12 omega);
     z-index: 1;
   }
-  .one-sidebar #main {
-    @include span-columns(8);
+  .one-sidebar,
+  .two-sidebars {
+    #main {
+      @include span-columns(8);
+    }
   }
 
-  #aside {
+  #aside,
+  #sidebar-first {
     @include span-columns(4 omega);
   }
 
@@ -26,11 +30,17 @@
 
 //This is how we fix nested classes that are in content for now
 /* todo: remove grid classes post upgrade */
-.one-sidebar #main .grid-4 {
-  @include span-columns(4,8);
+.one-sidebar,
+.two-sidebars {
+  #main .grid-4 {
+    @include span-columns(4,8);
+  }
 }
-.one-sidebar #main .grid-4.omega {
-  @include omega();
+.one-sidebar,
+.two-sidebars {
+  #main .grid-4.omega {
+    @include omega();
+  }
 }
 
 /* todo: check if needed, page-titles are h1 everywhere */

sass/partials/drupalorg/_marketplace.scss

@@ -143,7 +143,8 @@
   @include marketplace-big-arrow-list;
 }
 
-#aside {
+#aside,
+#sidebar-first {
   #block-block-40,
   #block-block-42,
   #block-block-46,
@@ -159,7 +160,8 @@
   }
 }
 
-#aside #block-views-drupalorg-training-block-1 {
+#aside #block-views-drupalorg-training-block-1,
+#sidebar-first #block-views-drupalorg-training-block-1 {
   background-color: $white;
 
   ul.views-summary li {

tpl/page.tpl.php

@@ -73,13 +73,21 @@
 
       </div> <!-- /#page-heading -->
 
-      <div id="main" role="main">
         <?php if ($tabs['#primary']): ?>
           <div id="tabs" class="clearfix">
             <?php print render($tabs); ?>
           </div> <!-- /#tabs -->
         <?php endif; // end if $tabs ?>
 
+      <?php if ($page['sidebar_first']): ?>
+        <div id="sidebar-first" role="complementary" >
+          <div id="sidebar-first-region">
+            <?php print render($page['sidebar_first']); ?>
+          </div> <!-- /#column-right-region-first -->
+        </div> <!-- /#column-right-first -->
+      <?php endif; // end if $right-first ?>
+
+      <div id="main" role="main">
         <?php if ($page['content_top']): ?>
           <div id="content-top-region" class="clearfix">
             <?php print render($page['content_top']); ?>