[Koha-patches] [PATCH] Adding show/hide function to add tag form; fixing bug in series search link (Thanks Tim!)

[Owen Leonard]

---
 koha-tmpl/opac-tmpl/prog/en/css/opac.css           |   22 ++++++++++++--
 .../opac-tmpl/prog/en/modules/opac-detail.tmpl     |   31 ++++++++++++++-----
 2 files changed, 42 insertions(+), 11 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/prog/en/css/opac.css b/koha-tmpl/opac-tmpl/prog/en/css/opac.css
index cf65e43..6350e97 100644
--- a/koha-tmpl/opac-tmpl/prog/en/css/opac.css
+++ b/koha-tmpl/opac-tmpl/prog/en/css/opac.css
@@ -1024,18 +1024,31 @@ width: 100%;
 /* End hide from IE-mac */
 
 .results_summary {
-    display: block;
-    font-size : 85%;
-    color: #707070;
+	display: block;
+	font-size : 85%;
+	color: #707070;
 	padding : 0 0 .5em 0;
 }
 
 .results_summary .label {
 	color: #202020;
 }
+
 .results_summary a {
 	font-weight: normal;
 }
+
+.results_summary ul {
+	display : inline;
+	margin: 0;
+	padding: 0;
+	list-style : none;
+}
+
+.results_summary ul li {
+	display : inline;
+}
+
 .dialog {
   border: 1px solid #bcbcbc;
 	padding : .5em;
@@ -1647,3 +1660,6 @@ input.editshelf:active,input.deleteshelf:active {
 	margin : 0 -1em -1em -1em;
 	padding : .3em 1em;
 }
+input.tagbutton {
+	padding : 1px !important;
+}
\ No newline at end of file
diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tmpl b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tmpl
index c392590..0c583d9 100755
--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tmpl
+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tmpl
@@ -3,10 +3,23 @@
 <script type="text/JavaScript" language="JavaScript">
 //<![CDATA[
 	 $(document).ready(function() {
+	 	$("#tagform").hide();
+		$("#addtagl").show();
 		$('#bibliodescriptions > ul').tabs();
+		$("#addtagl a").click(function(){
+			$("#addtagl").hide();
+			$("#tagform, #tagform a").show();
+		});
+		$("#tagform a").click(function(){
+			$("#addtagl").show();
+			$("#tagform").hide();
+		});
 	}); 
 //]]>
 </script>
+<style type="text/css">
+	#addtagl { display: none; }
+</style>
 <script type="text/javascript">                            
     var GB_ROOT_DIR = "/intranet-tmpl/prog/en/lib/greybox/";                               
 </script>
@@ -68,7 +81,7 @@
     <span class="results_summary"><span class="label">Series:</span>
             <!-- TMPL_LOOP NAME="MARCSERIES" -->
             <!-- TMPL_LOOP NAME="MARCSERIES_SUBFIELDS_LOOP" -->
-            <a href="/cgi-bin/koha/opac-search.pl?q=series:<!-- TMPL_VAR NAME="value"-->"><!-- TMPL_VAR NAME="value" --></a>
+            <a href="/cgi-bin/koha/opac-search.pl?q=se:<!-- TMPL_VAR NAME="value"-->"><!-- TMPL_VAR NAME="value" --></a>
             <!-- /TMPL_LOOP -->
             <!-- /TMPL_LOOP -->
     </span>
@@ -152,20 +165,22 @@
     </span>
     <!-- /TMPL_IF -->
 <!-- /TMPL_IF -->
+
 	<!-- TMPL_IF NAME="TagsShowOnDetail" -->
         <div class="results_summary">
         <!-- TMPL_IF NAME="TagLoop" -->
         <span class="label">Tags:</span>
-            <ul style="display: inline; list-style: none;"><!-- TMPL_LOOP NAME="TagLoop" -->
-            <li style="display: inline; list-style: none;"><a href="/cgi-bin/koha/opac-search.pl?tag=<!-- TMPL_VAR NAME="term" ESCAPE="URL" -->&amp;q=<!-- TMPL_VAR NAME="term" ESCAPE="URL" -->"><!-- TMPL_VAR NAME="term" ESCAPE="HTML" --></a> <span class="weight">(<!-- TMPL_VAR NAME="weight_total" -->)</span></li>
+            <ul><!-- TMPL_LOOP NAME="TagLoop" -->
+            <li><a href="/cgi-bin/koha/opac-search.pl?tag=<!-- TMPL_VAR NAME="term" ESCAPE="URL" -->&amp;q=<!-- TMPL_VAR NAME="term" ESCAPE="URL" -->"><!-- TMPL_VAR NAME="term" ESCAPE="HTML" --></a> <span class="weight">(<!-- TMPL_VAR NAME="weight_total" -->)</span></li>
             <!-- /TMPL_LOOP -->
-            </ul>
+            </ul><!-- TMPL_ELSE -->No tags for this title. 
         <!-- /TMPL_IF -->
         <!-- TMPL_IF NAME="TagsInputOnDetail" -->
-            <span class="label">New:</span>
-            <form name="tagform<!-- TMPL_VAR NAME="biblionumber" -->" method="post" action="/cgi-bin/koha/opac-tags.pl">
-                <input name="newtag<!-- TMPL_VAR NAME="biblionumber" -->" id="newtag<!-- TMPL_VAR NAME="biblionumber" -->" maxlength="100" />
-                <input name="tagbutton" class="tagbutton" type="submit" value="Add" />
+		<span class="label" id="addtagl"><a href="#">Add</a></span>
+         <form name="tagform<!-- TMPL_VAR NAME="biblionumber" -->" method="post" action="/cgi-bin/koha/opac-tags.pl" id="tagform">
+                <label for="newtag<!-- TMPL_VAR NAME="biblionumber" -->">New:</span><input name="newtag<!-- TMPL_VAR NAME="biblionumber" -->" id="newtag<!-- TMPL_VAR NAME="biblionumber" -->" maxlength="100" />
+                <input name="tagbutton" class="input tagbutton" type="submit" value="Add" />
+				<a href="#">Cancel</a>
             </form>
             <br />
             <span id="tagstatus<!-- TMPL_VAR NAME="biblionumber" -->" class="tagstatus" style="display:none;">
-- 
1.5.2.1