1 files changed, 28 insertions, 38 deletions

diff --git a/views/default/css/elements/page_layout.php b/views/default/css/elements/page_layout.php
index ebb201225..13c6ce1d1 100644
--- a/views/default/css/elements/page_layout.php
+++ b/views/default/css/elements/page_layout.php
@@ -1,15 +1,31 @@
 <?php
 /**
+ * Page Layout
  *
+ * Contains CSS for the page shell and page layout
+ *
+ * Classic layout: 990 wide centered. Used in default page shell
  */
 ?>
 
 /* ***************************************
 	PAGE LAYOUT
 *************************************** */
-<?php //@todo should be in base/reset ?>
-body {
-	background-color: white;
+/***** CLASSIC LAYOUT ******/
+.elgg-classic .elgg-page-header > .elgg-inner {
+	width: 990px;
+	margin: 0 auto;
+	height: 90px;
+}
+.elgg-classic .elgg-page-body > .elgg-inner {
+	width: 990px;
+	margin: 0 auto;
+}
+.elgg-classic .elgg-page-footer > .elgg-inner {
+	width: 990px;
+	margin: 0 auto;
+	padding: 5px 0;
+	border-top: 1px solid #DEDEDE;
 }
 
 /***** TOPBAR ******/
@@ -25,7 +41,6 @@ body {
 .elgg-page-topbar > .elgg-inner {
 	padding: 2px 10px 2px 8px;
 }
-
 <?php //@todo location-dependent styles ?>
 .elgg-page-topbar a {
 	color: #eeeeee;
@@ -41,7 +56,7 @@ body {
 	color: #71cbff;
 	text-decoration: none;
 }
-/* elgg logo and user avatar need to be adjusted slightly */
+<?php // elgg logo and user avatar need to be adjusted slightly ?>
 .elgg-page-topbar img {
 	margin-top: -1px;
 }
@@ -56,7 +71,6 @@ body {
 }
 
 /***** PAGE HEADER ******/
-
 .elgg-page-header {
 	x-overflow: hidden;
 	position: relative;
@@ -65,15 +79,9 @@ body {
 	background-repeat: repeat-x;
 	background-position: bottom left;
 }
-
-<?php //@todo Put all elgg-classic styles together ?>
-.elgg-classic .elgg-page-header > .elgg-inner {
-	width: 990px;
-	margin: 0 auto;
-	height: 90px;
+.elgg-page-header > .elgg-inner {
 	position: relative;
 }
-
 .elgg-site-title, .elgg-site-title:hover {
 	font-size: 2em;
 	line-height: 1.4em;
@@ -85,14 +93,9 @@ body {
 }
 
 /***** PAGE BODY ******/
-
 .elgg-page-body > .elgg-inner {
 	min-height: 360px;
 }
-.elgg-classic .elgg-page-body > .elgg-inner {
-	width: 990px;
-	margin: 0 auto;
-}
 #elgg-layout-one-column {
 	padding: 10px 0;
 }
@@ -106,11 +109,6 @@ body {
 	background-repeat: repeat-y;
 	background-position: right top;
 }
-.elgg-main {
-	position: relative;
-	min-height: 360px;
-	padding: 10px;
-}
 .elgg-aside {
 	padding: 20px 10px;
 	position: relative;
@@ -126,37 +124,31 @@ body {
 	width: 160px;
 	margin: 0 10px 0 0;
 }
-
+.elgg-main {
+	position: relative;
+	min-height: 360px;
+	padding: 10px;
+}
 .elgg-main .elgg-header {
 	padding-bottom: 3px;
 	border-bottom: 1px solid #CCCCCC;
 	margin-bottom: 10px;
 }
-
 <?php //@todo location-dependent styles ?>
 .elgg-main .elgg-header h2 {
 	float: left;
 	max-width: 530px;
 	margin-right: 10px;
 }
-
 .elgg-main > .elgg-header a {
 	float: right;
 }
 
 /***** PAGE FOOTER ******/
-
 .elgg-page-footer {
 	position: relative;
 	z-index: 999;
 }
-.elgg-classic .elgg-page-footer > .elgg-inner {
-	width: 990px;
-	margin: 0 auto;
-	padding: 5px 0;
-	border-top: 1px solid #DEDEDE;
-}
-
 <?php //@todo location-dependent styles ?>
 .elgg-page-footer a {
 	float: left;
@@ -168,11 +160,9 @@ body {
 	float: left;
 	width: 100%;
 }
-.elgg-page-footer .elgg-inner,
-.elgg-page-footer .elgg-inner a,
-.elgg-page-footer .elgg-inner p {
+.elgg-page-footer, .elgg-page-footer a {
 	color: #999999;
 }
-.elgg-page-footer .elgg-inner a:hover {
+.elgg-page-footer a:hover {
 	color: #666666;
 }