cleaning up the page footer - still need to add css for footer menu

git-svn-id: http://code.elgg.org/elgg/trunk@7685 36083f99-b078-4883-b0ff-0f9b5a30f544

4 files changed, 21 insertions, 23 deletions

diff --git a/mod/reportedcontent/views/default/reportedcontent/css.php b/mod/reportedcontent/views/default/reportedcontent/css.php
index c66cd2004..ed498f03a 100644
--- a/mod/reportedcontent/views/default/reportedcontent/css.php
+++ b/mod/reportedcontent/views/default/reportedcontent/css.php
@@ -9,12 +9,9 @@
 
 ?>
 /* Reported Content */
-#report_this {
-	text-align: left;
-	float:left;
-}
-#report_this a {
+.report-this {
 	font-size: 90%;
-	padding:0 0 4px 20px;
+}
+.report-this-icon {
 	background: url(<?php echo elgg_get_site_url(); ?>mod/reportedcontent/graphics/icon_reportthis.gif) no-repeat left top;
 }
diff --git a/mod/reportedcontent/views/default/reportedcontent/footer_link.php b/mod/reportedcontent/views/default/reportedcontent/footer_link.php
index 69e6f0967..4e893522c 100644
--- a/mod/reportedcontent/views/default/reportedcontent/footer_link.php
+++ b/mod/reportedcontent/views/default/reportedcontent/footer_link.php
@@ -9,6 +9,8 @@ $title = elgg_echo('reportedcontent:this:tooltip');
 $text  = elgg_echo('reportedcontent:this');
 ?>
 
-<div id="report_this">
-	<a href="javascript:location.href='<?php echo elgg_get_site_url(); ?>pg/reportedcontent/add/?address='+encodeURIComponent(location.href)+'&title='+encodeURIComponent(document.title)" title="<?php echo $title; ?>"><?php echo $text; ?></a>
-</div>
+
+<a class="report-this" href="javascript:location.href='<?php echo elgg_get_site_url(); ?>pg/reportedcontent/add/?address='+encodeURIComponent(location.href)+'&title='+encodeURIComponent(document.title)" title="<?php echo $title; ?>">
+	<span class="elgg-icon report-this-icon"></span>
+	<?php echo $text; ?>
+</a>
diff --git a/mod/sitepages/start.php b/mod/sitepages/start.php
index 844187503..08c01d6c7 100644
--- a/mod/sitepages/start.php
+++ b/mod/sitepages/start.php
@@ -24,6 +24,8 @@ function sitepages_init() {
 	// Extend CSS
 	elgg_extend_view('css/screen', 'sitepages/css');
 
+	sitepages_setup_footer_menu();
+
 	// register our subtype
 	run_function_once('sitepages_runonce');
 
@@ -33,7 +35,6 @@ function sitepages_init() {
 	// Register a URL handler for external pages
 	register_entity_url_handler('sitepages_url', 'object', 'sitepages');
 
-	elgg_extend_view('footer/links', 'sitepages/footer_menu');
 	elgg_extend_view('html_head/extend', 'sitepages/metatags');
 
 	// Replace the default index page if user has requested and the site is not running walled garden
@@ -137,6 +138,17 @@ function sitepages_page_handler($page) {
 	echo elgg_view_page($title, $content);
 }
 
+/**
+ * Setup the links to site pages
+ */
+function sitepages_setup_footer_menu() {
+	$pages = array('about', 'terms', 'privacy');
+	foreach ($pages as $page) {
+		$url = "pg/sitepages/read/$page";
+		$item = new ElggMenuItem($page, elgg_echo("sitepages:$page"), $url);
+		elgg_register_menu_item('footer', $item);
+	}
+}
 
 /**
  * Register some default keywords.
diff --git a/mod/sitepages/views/default/sitepages/footer_menu.php b/mod/sitepages/views/default/sitepages/footer_menu.php
deleted file mode 100644
index d3858cb20..000000000
--- a/mod/sitepages/views/default/sitepages/footer_menu.php
+++ /dev/null
@@ -1,13 +0,0 @@
-<?php
-/**
- * Footer view to add links to the semi-static pages.
- *
- * @package SitePages
- */
-?>
-
-<div class="footer_toolbar_links">|
-<a href="<?php echo elgg_get_site_url(); ?>pg/sitepages/read/About/"><?php echo elgg_echo('sitepages:about'); ?></a> |
-<a href="<?php echo elgg_get_site_url(); ?>pg/sitepages/read/Terms/"><?php echo elgg_echo('sitepages:terms'); ?></a> |
-<a href="<?php echo elgg_get_site_url(); ?>pg/sitepages/read/Privacy/"><?php echo elgg_echo('sitepages:privacy'); ?></a> |
-</div>
\ No newline at end of file