From f4b928e969879c2ec50e98a2a0029e031e18b287 Mon Sep 17 00:00:00 2001
From: lars <lars@systemausfall.org>
Date: Sun, 21 Jan 2007 18:08:38 +0000
Subject: [PATCH] unified up/down link anchors: always adjust to the
 appropriate fieldset (instead of the moved plugin)

---
 plugins/plugin_manager/plugin_list.cs | 26 +++++++++++++-------------
 1 file changed, 13 insertions(+), 13 deletions(-)

diff --git a/plugins/plugin_manager/plugin_list.cs b/plugins/plugin_manager/plugin_list.cs
index 3a22c65..971b302 100644
--- a/plugins/plugin_manager/plugin_list.cs
+++ b/plugins/plugin_manager/plugin_list.cs
@@ -13,7 +13,7 @@
 <?cs # <form> starts here ?>
 <?cs call:print_form_header("manage_plugins", "plugin_manager") ?>
 
-<fieldset>
+<fieldset id="volume_plugins">
 	<legend><?cs var:html_escape(Lang.Plugins.plugin_manager.Title.VolumePlugins) ?></legend>
 	<?cs call:show_help(Lang.Plugins.plugin_manager.Help.VolumePlugins) ?>
 
@@ -40,23 +40,23 @@
 		each:x = Settings.PluginList ?><?cs if:(x.Rank == index) && x.Types.volume
 		?><?cs set: run_counter = run_counter + 1 ?>
 		<tr>
-			<td style="text-align:left"><a name="<?cs var:html_escape(name(x)) ?>"><?cs var:html_escape(Lang.Plugins[name(x)].Name) ?></a></td>
+			<td style="text-align:left" id="plugin_anchor_<?cs var:html_escape(name(x)) ?>"><?cs var:html_escape(Lang.Plugins[name(x)].Name) ?></td>
 			<td><input type="checkbox" name="<?cs var:name(x) ?>_visible_volume" <?cs if:x.Visible.volume ?>checked="checked"<?cs /if ?> /></td>
 			<td><input type="checkbox" name="<?cs var:name(x) ?>_visible_properties" <?cs if:x.Visible.properties ?>checked="checked"<?cs /if ?> /></td>
 			<td><input type="checkbox" name="<?cs var:name(x) ?>_auth" <?cs if:x.RequestAuth ?>checked="checked"<?cs /if ?> /></td>
 			<td class="down">
-				<?cs if:run_counter != all_count ?><a href="<?cs call:link("plugin_manager", 
-					"plugin_name", name(x), "action", "down") ?>&amp;x=<?cs 
-					var:run_counter ?>#volume_plugins"><img 
+				<?cs if:run_counter != all_count ?><a href="<?cs
+					call:link("plugin_manager", "plugin_name", name(x), "action", "down")
+					?>&amp;x=<?cs var:run_counter ?>#volume_plugins"><img 
 					src="icons/plugin_manager?image=tango-go-down.gif" alt="<?cs
 					var:html_escape(Lang.Plugins.plugin_manager.Button.Down) ?>" 
 					title="<?cs var:html_escape(Lang.Plugins.plugin_manager.Button.Down) 
 					?>" /></a><?cs /if ?>
 			</td>
 			<td class="up">
-				<?cs if:run_counter != 1 ?><a href="<?cs call:link("plugin_manager",
-					"plugin_name", name(x), "action", "up") ?>&amp;x=<?cs
-					var:run_counter ?>#volume_plugins" ><img
+				<?cs if:run_counter != 1 ?><a href="<?cs
+					call:link("plugin_manager", "plugin_name", name(x), "action", "up")
+					?>&amp;x=<?cs var:run_counter ?>#volume_plugins" ><img
 					src="icons/plugin_manager?image=tango-go-up.gif" alt="<?cs
 					var:html_escape(Lang.Plugins.plugin_manager.Button.Up) ?>"
 					title="<?cs var:html_escape(Lang.Plugins.plugin_manager.Button.Up)
@@ -68,7 +68,7 @@
 	</table>
 </fieldset>
 
-<fieldset>
+<fieldset id="system_plugins">
 	<legend><?cs var:html_escape(Lang.Plugins.plugin_manager.Title.SystemPlugins) ?></legend>
 	<?cs call:show_help(Lang.Plugins.plugin_manager.Help.SystemPlugins) ?>
 	<table class="plugin_list">
@@ -93,7 +93,7 @@
 	loop:index = #0, #100, #1 ?><?cs
 	each:x = Settings.PluginList ?><?cs if:(x.Rank == index) && x.Types.system
 		?><?cs set: run_counter = run_counter + 1 ?><tr>
-			<td style="text-align:left"><a name="<?cs var:html_escape(name(x)) ?>"><?cs var:html_escape(Lang.Plugins[name(x)].Name) ?></a></td>
+			<td style="text-align:left" id="plugin_anchor_<?cs var:html_escape(name(x)) ?>"><?cs var:html_escape(Lang.Plugins[name(x)].Name) ?></td>
 			<td><input type="checkbox" name="<?cs var:name(x) ?>_visible_menu" <?cs if:x.Visible.menu ?>checked="checked"<?cs /if ?> /></td>
 			<td><input type="checkbox" name="<?cs var:name(x) ?>_visible_preferences" <?cs if:x.Visible.preferences ?>checked="checked"<?cs /if ?> /></td>
 			<td><input type="checkbox" name="<?cs var:name(x) ?>_auth" <?cs if:x.RequestAuth ?>checked="checked"<?cs /if ?> /></td>
@@ -106,9 +106,9 @@
 					title="<?cs var:html_escape(Lang.Plugins.plugin_manager.Button.Down)
 					?>" /></a><?cs /if ?>
 			<td class="up">
-				<?cs if:run_counter != 1 ?><a href="<?cs call:link("plugin_manager",
-					"plugin_name", name(x), "action", "up") ?>&amp;x=<?cs var:run_counter
-					?>#<?cs var:html_escape(name(x)) ?>" ><img
+				<?cs if:run_counter != 1 ?><a href="<?cs
+					call:link("plugin_manager", "plugin_name", name(x), "action", "up")
+					?>&amp;x=<?cs var:run_counter ?>#system_plugins" ><img
 					src="icons/plugin_manager?image=tango-go-up.gif" alt="<?cs
 					var:html_escape(Lang.Plugins.plugin_manager.Button.Up) ?>"
 					title="<?cs var:html_escape(Lang.Plugins.plugin_manager.Button.Up)