From 21c94bd2185b20d4f551e27967134db357f61f3b Mon Sep 17 00:00:00 2001
From: Diana Whitten <hurgleburgler@gmail.com>
Date: Mon, 21 Dec 2015 16:21:26 -0700
Subject: [PATCH] Removing container ID element

Removing the element, as it was not really needed at all.
Also, removed a debug statement that was discovered in
material.hamburger.js while refactoring for this change.

Added a 'container-fluid' class to the content body
as well, because it is not possible to configure Horizon
as a non-fluid layout without a container element around
the first 'row' element.

Change-Id: I097ea31f991e8d15987b7cf54b77a309d6155771
Closes-bug: #1368924
---
 .../static/dashboard/scss/_layout.scss        |  3 ++-
 openstack_dashboard/templates/base.html       | 20 +++++++++----------
 .../material/static/js/material.hamburger.js  |  3 +--
 3 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/openstack_dashboard/static/dashboard/scss/_layout.scss b/openstack_dashboard/static/dashboard/scss/_layout.scss
index 18472a43e8..0561f3dc45 100644
--- a/openstack_dashboard/static/dashboard/scss/_layout.scss
+++ b/openstack_dashboard/static/dashboard/scss/_layout.scss
@@ -1,7 +1,8 @@
 // Make the sidebar span the entire page, instead of just the Viewport
 // Display Table is not ideal, but it will allow dynamic resizing of
 // the sidebar while growing its container to fill the entire window
-html, body:not(#splash), #container {
+html,
+body:not(#splash) {
   height: 100%;
 }
 
diff --git a/openstack_dashboard/templates/base.html b/openstack_dashboard/templates/base.html
index 5eb6f653a7..61a35160db 100644
--- a/openstack_dashboard/templates/base.html
+++ b/openstack_dashboard/templates/base.html
@@ -24,16 +24,16 @@
       </div>
     </noscript>
     {% block content %}
-      <div id="container">
-        <div class='topbar'>
-          {% include "header/_header.html" %}
-        </div>
-        <div id='main_content'>
-          {% include "horizon/_messages.html" %}
-          {% block sidebar %}
-            {% include 'horizon/common/_sidebar.html' %}
-          {% endblock %}
-          <div id='content_body'>
+      <div class='topbar'>
+        {% include "header/_header.html" %}
+      </div>
+      <div id='main_content'>
+        {% include "horizon/_messages.html" %}
+        {% block sidebar %}
+          {% include 'horizon/common/_sidebar.html' %}
+        {% endblock %}
+        <div id='content_body'>
+          <div class='container-fluid'>
             <div class="row">
               <div class="col-xs-12">
                 {% block page_header %}
diff --git a/openstack_dashboard/themes/material/static/js/material.hamburger.js b/openstack_dashboard/themes/material/static/js/material.hamburger.js
index 0874ae54e7..a61304a0c0 100644
--- a/openstack_dashboard/themes/material/static/js/material.hamburger.js
+++ b/openstack_dashboard/themes/material/static/js/material.hamburger.js
@@ -19,11 +19,10 @@ $(document).ready(function () {
   var $sidenav = $('#sidebar');
   var $mask = $(document.createElement('div'))
     .prop('id', 'md-hamburger-mask')
-    .appendTo($('#container'));
+    .appendTo($('#content_body'));
 
   // Hamburger Happiness !!!
   $(document).on('click', '.md-hamburger', function () {
-    var $foo = $(this).find('.md-hamburger-layer');
     $(this).find('.md-hamburger-layer')
       .toggleClass('md-hamburger-arrow')
       .toggleClass('md-hamburger-menu');