The "hide window" button in the cart called a quit() function
which deletes any checked items. My understanding of the "hide
window" button is that it should *only* hide the window, not
perform any other operations. I'm removing the calls to the
quit() function in both the OPAC and staff client and adding
a CSS class, "close," which is already tied via jQuery to
the window.close() function.

To test in both OPAC and staff client: Add items to the cart.
Try clicking the "hide window" button with items selected and
without. In both cases the window should close without affecting
the contents of the cart.
---
 .../intranet-tmpl/prog/en/modules/basket/basket.tt |    2 +-
 koha-tmpl/opac-tmpl/prog/en/modules/opac-basket.tt |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/basket/basket.tt 
b/koha-tmpl/intranet-tmpl/prog/en/modules/basket/basket.tt
index 0896075..4239b4f 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/basket/basket.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/basket/basket.tt
@@ -108,7 +108,7 @@ function placeHold () {
        </li>
        <li>
            <span id="hide" class="yui-button yui-link-button"><span 
class="first-child">
-               <a class="hide" href="basket.pl" onclick="quit(); return 
false;">Hide Window</a>
+               <a class="hide close" href="basket.pl">Hide Window</a>
            </span></span>
        </li>
 </ul>
diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-basket.tt 
b/koha-tmpl/opac-tmpl/prog/en/modules/opac-basket.tt
index 62be158..a229521 100644
--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-basket.tt
+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-basket.tt
@@ -128,7 +128,7 @@ function tagAdded() {
        <li><a class="download" href="opac-basket.pl" 
onclick="downloadBasket(); return false;">Download</a></li>
        <li><a class="print" href="opac-basket.pl" onclick="printBasket(); 
return false;">Print</a></li>
        <li><a class="empty" href="opac-basket.pl" onclick="delBasket(); return 
false;">Empty and Close</a></li>
-       <li><a class="hide" href="opac-basket.pl" onclick="quit(); return 
false;">Hide Window</a></li>
+       <li><a class="hide close" href="opac-basket.pl">Hide Window</a></li>
 </ul>
 </div>
         [% END %]
-- 
1.7.3

_______________________________________________
Koha-patches mailing list
Koha-patches@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-patches
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to