From 1b460feda2263fc36dc0edafca3549c722738a4a Mon Sep 17 00:00:00 2001
From: James Taylor <james@jamestaylor.org>
Date: Fri, 14 Aug 2009 12:17:45 -0400
Subject: [PATCH] Fix for library expand buttons in IE (even though the old way
 used to work, should work, and was tested).

---
 templates/library/browse_library.mako | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/templates/library/browse_library.mako b/templates/library/browse_library.mako
index 0ef2fae1003..919335aa064 100644
--- a/templates/library/browse_library.mako
+++ b/templates/library/browse_library.mako
@@ -38,14 +38,21 @@ class RowCounter( object ):
                     descendents = descendents.add( child_descendents );
                 });
                 // Set up expand / hide link
+                // HACK: assume descendents are invisible. The caller actually
+                //       ensures this for the root node. However, if we start
+                //       remembering folder states, we'll need something
+                //       more sophisticated here.
+                var visible = false;
                 $(q).find( "span.expandLink").click( function() {
-                    if ( children.is( ":visible" ) ) {
+                    if ( visible ) {
                         descendents.hide();
                         descendents.removeClass( "expanded" );
                         q.removeClass( "expanded" );
+                        visible = false;
                     } else {
                         children.show();
                         q.addClass( "expanded" );
+                        visible = true;
                     }
                 });
                 // Check/uncheck boxes in subfolders.