From 2062a91d9ca2f31ccf4cd1eaa338c5190bffe56d Mon Sep 17 00:00:00 2001
From: brettp <brettp@36083f99-b078-4883-b0ff-0f9b5a30f544>
Date: Fri, 4 Feb 2011 23:03:32 +0000
Subject: Added support for screenshots in the advanced plugin admin.

git-svn-id: http://code.elgg.org/elgg/trunk@8021 36083f99-b078-4883-b0ff-0f9b5a30f544
---
 views/default/admin/components/plugin.php | 19 +++++++------------
 views/default/css/admin.php               |  5 ++++-
 2 files changed, 11 insertions(+), 13 deletions(-)

(limited to 'views/default')

diff --git a/views/default/admin/components/plugin.php b/views/default/admin/components/plugin.php
index 4c6a8b756..0185ba0ff 100644
--- a/views/default/admin/components/plugin.php
+++ b/views/default/admin/components/plugin.php
@@ -124,20 +124,15 @@ if ($categories) {
 
 // @todo We need to make a page handler to read these files in.
 // this is broken.
-$screenshot_html = '';
+$screenshots_html = '';
 $screenshots = $plugin->manifest->getScreenshots();
 if ($screenshots) {
 	$base_url = elgg_get_plugins_path() . $plugin->getID() . '/';
-	$limit = 4;
 	foreach ($screenshots as $screenshot) {
-		if ($limit <= 0) {
-			break;
-		}
-
-		$screenshot_src = $plugin->getPath() . $screenshot['path'];
-		$screenshots .= "<li class=\"plugin-screenshot\"><a href=\"$screenshot_src\"><img src=\"$screenshot_src\"></a></li>";
+		$screenshot_full = "{$vars['url']}pg/admin_plugin_screenshot/{$plugin->getID()}/full/{$screenshot['path']}";
+		$screenshot_src = "{$vars['url']}pg/admin_plugin_screenshot/{$plugin->getID()}/thumbnail/{$screenshot['path']}";
 
-		$limit--;
+		$screenshots_html .= "<li class=\"elgg-plugin-screenshot prm ptm\"><a href=\"$screenshot_full\"><img src=\"$screenshot_src\"></a></li>";
 	}
 }
 
@@ -189,7 +184,7 @@ if (elgg_view_exists($settings_view)) {
 		}
 		?>
 
-		<p><a class="manifest_details small link"><?php echo elgg_echo("admin:plugins:label:moreinfo"); ?></a></p>
+		<div class="pts"><a class="manifest_details small link"><?php echo elgg_echo("admin:plugins:label:moreinfo"); ?></a></div>
 
 		<div class="manifest_file hidden">
 
@@ -201,9 +196,9 @@ if (elgg_view_exists($settings_view)) {
 			<?php
 		}
 
-		if ($screenshots) {
+		if ($screenshots_html) {
 			?>
-			<div><ul><?php echo $screenshots; ?></ul></div>
+			<div><ul><?php echo $screenshots_html; ?></ul></div>
 			<?php
 		}
 		?>
diff --git a/views/default/css/admin.php b/views/default/css/admin.php
index 4376ca41a..99ac0f455 100644
--- a/views/default/css/admin.php
+++ b/views/default/css/admin.php
@@ -128,7 +128,7 @@ table.mceLayout {
 	width:100% !important;
 }
 
-.clearfloat { 
+.clearfloat {
 	clear:both;
 }
 
@@ -960,6 +960,9 @@ ul.admin_plugins {
 .elgg-unsatisfied-dependency {
 	font-weight: bold;
 }
+.elgg-plugin-screenshot {
+	display: inline;
+}
 .active {
 	border:1px solid #999999;
 	background:white;
-- 
cgit v1.2.3