From a2250e1854f23351b2a39e304e2bd5977f3fd867 Mon Sep 17 00:00:00 2001
From: cash <cash@36083f99-b078-4883-b0ff-0f9b5a30f544>
Date: Sat, 26 Feb 2011 20:57:09 +0000
Subject: Fixes #2982 friends collections work even if the code is convoluted
 and poorly written.

git-svn-id: http://code.elgg.org/elgg/trunk@8495 36083f99-b078-4883-b0ff-0f9b5a30f544
---
 views/default/core/friends/collections.php       |  2 +-
 views/default/css/elements/forms.php             | 15 ++++---
 views/default/css/elements/misc.php              |  5 ++-
 views/default/forms/friends/collections/add.php  | 53 ++++++++++++++++++++++++
 views/default/forms/friends/collections/edit.php | 53 ------------------------
 views/default/input/friendspicker.php            | 13 ++++--
 6 files changed, 77 insertions(+), 64 deletions(-)
 create mode 100644 views/default/forms/friends/collections/add.php
 delete mode 100644 views/default/forms/friends/collections/edit.php

(limited to 'views/default')

diff --git a/views/default/core/friends/collections.php b/views/default/core/friends/collections.php
index 5c1c824e7..d88d591ad 100644
--- a/views/default/core/friends/collections.php
+++ b/views/default/core/friends/collections.php
@@ -32,7 +32,7 @@ if (is_array($vars['collections']) && sizeof($vars['collections'])) {
 <script>
 $(document).ready(function(){
 	$('#friends_collections_accordian h2').click(function () {
-		$(this.parentNode).children("[class=friends-picker]").slideToggle("fast");
+		$(this.parentNode).children("[class=friends-picker-main-wrapper]").slideToggle("fast");
 		//return false;
 	});
 });
diff --git a/views/default/css/elements/forms.php b/views/default/css/elements/forms.php
index c237c79a5..7c2d273f2 100644
--- a/views/default/css/elements/forms.php
+++ b/views/default/css/elements/forms.php
@@ -104,6 +104,9 @@ input[type="radio"] {
 	padding:0;
 	width: 730px;
 	height: auto;
+	background-color: #dedede;
+	-moz-border-radius: 8px;
+	-webkit-border-radius: 8px;
 }
 .friendspicker-savebuttons {
 	background: white;
@@ -176,8 +179,8 @@ input[type="radio"] {
 }
 .friends-picker-navigation-l a, .friends-picker-navigation-r a {
 	display: block;
-	height: 43px;
-	width: 43px;
+	height: 40px;
+	width: 40px;
 }
 .friends-picker-navigation-l {
 	right: 48px;
@@ -188,16 +191,16 @@ input[type="radio"] {
 	z-index:1;
 }
 .friends-picker-navigation-l {
-	background: url("<?php echo elgg_get_site_url(); ?>_graphics/elgg_sprites.png") no-repeat left top;
+	background: url("<?php echo elgg_get_site_url(); ?>_graphics/friendspicker.png") no-repeat left top;
 }
 .friends-picker-navigation-r {
-	background: url("<?php echo elgg_get_site_url(); ?>_graphics/elgg_sprites.png") no-repeat -60px top;
+	background: url("<?php echo elgg_get_site_url(); ?>_graphics/friendspicker.png") no-repeat -60px top;
 }
 .friends-picker-navigation-l:hover {
-	background: url("<?php echo elgg_get_site_url(); ?>_graphics/elgg_sprites.png") no-repeat left -44px;
+	background: url("<?php echo elgg_get_site_url(); ?>_graphics/friendspicker.png") no-repeat left -44px;
 }
 .friends-picker-navigation-r:hover {
-	background: url("<?php echo elgg_get_site_url(); ?>_graphics/elgg_sprites.png") no-repeat -60px -44px;
+	background: url("<?php echo elgg_get_site_url(); ?>_graphics/friendspicker.png") no-repeat -60px -44px;
 }
 .friendspicker-savebuttons .elgg-button-submit,
 .friendspicker-savebuttons .elgg-button-cancel {
diff --git a/views/default/css/elements/misc.php b/views/default/css/elements/misc.php
index abcbb894a..d159761a1 100644
--- a/views/default/css/elements/misc.php
+++ b/views/default/css/elements/misc.php
@@ -50,11 +50,14 @@
 	margin: 10px 0;
 	padding: 4px 2px 4px 6px;
 }
+#friends_collections_accordian li h2:hover {
+	background-color: #333333;
+}
 #friends_collections_accordian .friends_collections_controls {
 	float: right;
 	font-size: 70%;
 }
-#friends_collections_accordian .friends-picker {
+#friends_collections_accordian .friends-picker-main-wrapper {
 	background: none repeat scroll 0 0 white;
 	display: none;
 	padding: 0;
diff --git a/views/default/forms/friends/collections/add.php b/views/default/forms/friends/collections/add.php
new file mode 100644
index 000000000..644c7adb9
--- /dev/null
+++ b/views/default/forms/friends/collections/add.php
@@ -0,0 +1,53 @@
+<?php
+/**
+ * Form body for editing or adding a friend collection
+ *
+ * @package Elgg
+ * @subpackage Core
+ *
+ * @uses $vars['collection'] Optionally, the collection to edit
+ */
+
+// Set title, form destination
+if (isset($vars['collection'])) {
+	$title = $vars['collection']->name;
+	$highlight = 'default';
+} else  {
+	$title = "";
+	$highlight = 'all';
+}
+
+echo "<div class=\"mtm\"><label>" . elgg_echo("friends:collectionname") . "<br/>";
+echo elgg_view("input/text", array(
+		"name" => "collection_name",
+		"value" => $title,
+	));
+echo "</label></div>";
+
+echo "<div>";
+if ($vars['collection_members']) {
+	echo elgg_echo("friends:collectionfriends") . "<br />";
+	foreach ($vars['collection_members'] as $mem) {
+		echo elgg_view_entity_icon($mem, 'tiny');
+		echo $mem->name;
+	}
+}
+echo "</div>";
+
+echo "<div><label>" . elgg_echo("friends:addfriends") . "</label>";
+echo elgg_view('input/friendspicker', array(
+	'entities' => $vars['friends'],
+	'name' => 'friends_collection',
+	'highlight' => $highlight,
+));
+echo "</div>";
+
+echo "<div>";
+if (isset($vars['collection'])) {
+	echo elgg_view('input/hidden', array(
+		'name' => 'collection_id',
+		'value' => $vars['collection']->id,
+	));
+}
+echo elgg_view('input/submit', array('name' => 'submit', 'value' => elgg_echo('save')));
+echo "</div>";
diff --git a/views/default/forms/friends/collections/edit.php b/views/default/forms/friends/collections/edit.php
deleted file mode 100644
index 975307085..000000000
--- a/views/default/forms/friends/collections/edit.php
+++ /dev/null
@@ -1,53 +0,0 @@
-<?php
-/**
- * Form body for editing or adding a friend collection
- *
- * @package Elgg
- * @subpackage Core
- *
- * @uses $vars['collection'] Optionally, the collection to edit
- */
-
-// Set title, form destination
-if (isset($vars['collection'])) {
-	$title = $vars['collection']->name;
-	$highlight = 'default';
-} else  {
-	$title = "";
-	$highlight = 'all';
-}
-
-echo "<div><label>" . elgg_echo("friends:collectionname") . "<br/>";
-echo elgg_view("input/text", array(
-		"name" => "collection_name",
-		"value" => $title,
-	));
-echo "</label></div>";
-
-echo "<div>";
-if ($vars['collection_members']) {
-	echo elgg_echo("friends:collectionfriends") . "<br />";
-	foreach ($vars['collection_members'] as $mem) {
-		echo elgg_view_entity_icon($mem, 'tiny');
-		echo $mem->name;
-	}
-}
-echo "</div>";
-
-echo "<div><label>" . elgg_echo("friends:addfriends") . "</label>";
-echo elgg_view('input/friendspicker', array(
-	'entities' => $vars['friends'],
-	'name' => 'friends_collection',
-	'highlight' => $highlight,
-));
-echo "</div>";
-
-echo "<div>";
-if (isset($vars['collection'])) {
-	echo elgg_view('input/hidden', array(
-		'name' => 'collection_id',
-		'value' => $vars['collection']->id,
-	));
-}
-echo elgg_view('input/submit', array('name' => 'submit', 'value' => elgg_echo('save')));
-echo "</div>";
diff --git a/views/default/input/friendspicker.php b/views/default/input/friendspicker.php
index b95640b0b..82ef3e3d7 100644
--- a/views/default/input/friendspicker.php
+++ b/views/default/input/friendspicker.php
@@ -3,6 +3,8 @@
  * Elgg friends picker
  * Lists the friends picker
  *
+ * @warning Below is the ugliest code in Elgg. It needs to be rewritten or removed
+ *
  * @package Elgg
  * @subpackage Core
  *
@@ -99,7 +101,7 @@ foreach ($users as $letter => $letter_users) {
 if (!$callback) {
 	?>
 
-	<div class="friends-picker">
+	<div class="friends-picker-main-wrapper">
 
 	<?php
 
@@ -118,7 +120,7 @@ if (!isset($vars['replacement'])) {
 ?>
 <?php //@todo JS 1.8: no ?>
 <script language="text/javascript">
-		$(function() { // onload...do
+	$(function() { // onload...do
 		$('#collectionMembersForm<?php echo $friendspicker; ?>').submit(function() {
 			var inputs = [];
 			$(':input', this).each(function() {
@@ -145,6 +147,11 @@ if (!isset($vars['replacement'])) {
 <form id="collectionMembersForm<?php echo $friendspicker; ?>" action="<?php echo $formtarget; ?>" method="post"> <!-- action="" method=""> -->
 
 <?php
+		echo elgg_view('input/securitytoken');
+		echo elgg_view('input/hidden', array(
+			'name' => 'collection_id',
+			'value' => $vars['collection_id'],
+		));
 	}
 ?>
 
@@ -305,7 +312,7 @@ if (!isset($vars['replacement'])) {
 	// initialise picker
 	$("div#friends-picker<?php echo $friendspicker; ?>").friendsPicker(<?php echo $friendspicker; ?>);
 </script>
-<script>
+<script type="text/javascript">
 $(document).ready(function () {
 // manually add class to corresponding tab for panels that have content
 <?php
-- 
cgit v1.2.3