From 711cdde5b9755140981000adc6dd7897d836ccc5 Mon Sep 17 00:00:00 2001
From: Bharat Mediratta <bharat@menalto.com>
Date: Sat, 12 Sep 2009 20:46:54 -0700
Subject: Only tack ?show=<current_id> on for the last parent, because that's
 the only parent for which the id is relevant.  In a perfect world each
 parent's link would have a ?show=<id> for the next child's id.  But that
 would require some confusing code and I don't want to put that into the
 default/example theme yet.

---
 themes/default/views/page.html.php | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

(limited to 'themes/default')

diff --git a/themes/default/views/page.html.php b/themes/default/views/page.html.php
index 733534ea..19d8cc00 100644
--- a/themes/default/views/page.html.php
+++ b/themes/default/views/page.html.php
@@ -95,7 +95,12 @@
         <ul class="gBreadcrumbs">
           <? foreach ($parents as $parent): ?>
           <li>
-            <a href="<?= $parent->url("show={$theme->item()->id}") ?>">
+            <!-- Adding ?show=<id> causes Gallery3 to display the page
+                 containing that photo.  For now, we just do it for
+                 the immediate parent so that when you go back up a
+                 level you're on the right page. -->
+            <a href="<?= $parent->url($parent == $theme->item()->parent() ?
+                     "show={$theme->item()->id}" : null) ?>">
               <?= html::purify($parent->title) ?>
             </a>
           </li>
-- 
cgit v1.2.3


From 75c68084136321b108e379643065fd21167990f8 Mon Sep 17 00:00:00 2001
From: Bharat Mediratta <bharat@menalto.com>
Date: Sun, 13 Sep 2009 14:25:56 -0700
Subject: Only show the block anchor if one exists.  Show block anchors for
 admin themes too.

---
 themes/admin_default/views/block.html.php | 3 +++
 themes/default/views/block.html.php       | 2 ++
 2 files changed, 5 insertions(+)

(limited to 'themes/default')

diff --git a/themes/admin_default/views/block.html.php b/themes/admin_default/views/block.html.php
index 21512d02..6cbea76e 100644
--- a/themes/admin_default/views/block.html.php
+++ b/themes/admin_default/views/block.html.php
@@ -1,4 +1,7 @@
 <?php defined("SYSPATH") or die("No direct script access.") ?>
+<? if ($anchor): ?>
+<a name="<?= $anchor ?>"></a>
+<? endif ?>
 <div block_id="<?= $id ?>" id="<?= $css_id ?>" class="gBlock ui-widget">
   <div class="ui-dialog-titlebar ui-widget-header ui-helper-clearfix ui-icon-right">
     <? if ($css_id != "gBlockAdder"): ?>
diff --git a/themes/default/views/block.html.php b/themes/default/views/block.html.php
index 37504861..e8cff833 100644
--- a/themes/default/views/block.html.php
+++ b/themes/default/views/block.html.php
@@ -1,5 +1,7 @@
 <?php defined("SYSPATH") or die("No direct script access.") ?>
+<? if ($anchor): ?>
 <a name="<?= $anchor ?>"></a>
+<? endif ?>
 <div id="<?= $css_id ?>" class="gBlock">
   <h2><?= $title ?></h2>
   <div class="gBlockContent">
-- 
cgit v1.2.3


From ffccfb9e634de527261dca8e46ac4053806ec87b Mon Sep 17 00:00:00 2001
From: Bharat Mediratta <bharat@menalto.com>
Date: Tue, 15 Sep 2009 20:23:32 -0700
Subject: Move the code that adds the .gShortForm CSS class out of
 gallery.short_forms.js and into the theme, since this is really a theme
 decision.

---
 lib/gallery.form.js          | 13 -------------
 themes/default/js/ui.init.js |  7 +++++--
 2 files changed, 5 insertions(+), 15 deletions(-)

(limited to 'themes/default')

diff --git a/lib/gallery.form.js b/lib/gallery.form.js
index 67245e60..77ce3b7d 100644
--- a/lib/gallery.form.js
+++ b/lib/gallery.form.js
@@ -1,14 +1,3 @@
-/**
- * Handle initialization of all short forms
- *
- * @param shortForms array Array of short form IDs
- */
-function handle_short_form_event(short_forms) {
-  for (var i in short_forms) {
-    short_form_init(short_forms[i]);
-  }
-}
-
 /**
  * Initialize a short form. Short forms may contain only one text input.
  *
@@ -16,8 +5,6 @@ function handle_short_form_event(short_forms) {
  */
 function short_form_init(form_id) {
   var form = $(form_id);
-  form.addClass("gShortForm");
-
   var label = form.find("label:first");
   var input = form.find("input[type=text]:first");
   var button = form.find("input[type=submit]");
diff --git a/themes/default/js/ui.init.js b/themes/default/js/ui.init.js
index 949933e9..93dfb275 100644
--- a/themes/default/js/ui.init.js
+++ b/themes/default/js/ui.init.js
@@ -2,7 +2,7 @@
  * Initialize jQuery UI and Gallery Plugin elements
  */
 
-var shortForms = new Array(
+var short_forms = new Array(
   "#gQuickSearchForm",
   "#gAddTagForm",
   "#gSearchForm"
@@ -36,7 +36,10 @@ $(document).ready(function() {
   }
 
   // Initialize short forms
-  handleShortFormEvent(shortForms);
+  for (var i in short_forms) {
+    short_form_init(short_forms[i]);
+    $(short_forms[i]).addClass("gShortForm");
+  }
   $(".gShortForm input[type=text]").addClass("ui-corner-left");
   $(".gShortForm input[type=submit]").addClass("ui-state-default ui-corner-right");
 
-- 
cgit v1.2.3


From 4e1e24ba1a976cd0bb7ca7dd03c6001f906a25dd Mon Sep 17 00:00:00 2001
From: Bharat Mediratta <bharat@menalto.com>
Date: Wed, 16 Sep 2009 20:34:42 -0700
Subject: Add a movie_menu() theme callback, and have the default theme call it
 in the sidebar on movie page types.

---
 modules/gallery/libraries/Theme_View.php | 6 ++++++
 themes/default/views/sidebar.html.php    | 2 ++
 2 files changed, 8 insertions(+)

(limited to 'themes/default')

diff --git a/modules/gallery/libraries/Theme_View.php b/modules/gallery/libraries/Theme_View.php
index 541bce88..130e2dce 100644
--- a/modules/gallery/libraries/Theme_View.php
+++ b/modules/gallery/libraries/Theme_View.php
@@ -111,6 +111,12 @@ class Theme_View_Core extends Gallery_View {
     return $menu->compact();
   }
 
+  public function movie_menu() {
+    $menu = Menu::factory("root");
+    module::event("movie_menu", $menu, $this);
+    return $menu->compact();
+  }
+
   public function context_menu($item, $thumbnail_css_selector) {
     $menu = Menu::factory("root")
       ->append(Menu::factory("submenu")
diff --git a/themes/default/views/sidebar.html.php b/themes/default/views/sidebar.html.php
index 928ecb93..04379eb6 100644
--- a/themes/default/views/sidebar.html.php
+++ b/themes/default/views/sidebar.html.php
@@ -6,6 +6,8 @@
       <?= $theme->album_menu() ?>
     <? elseif ($page_type == "photo") : ?>
       <?= $theme->photo_menu() ?>
+    <? elseif ($page_type == "movie") : ?>
+      <?= $theme->movie_menu() ?>
     <? elseif ($page_type == "tag") : ?>
       <?= $theme->tag_menu() ?>
     <? endif ?>
-- 
cgit v1.2.3


From 62863f5fead46480cf1a3f5a96eb52ce439d0f0e Mon Sep 17 00:00:00 2001
From: Chad Kieffer <ckieffer@gmail.com>
Date: Sat, 19 Sep 2009 18:55:49 -0600
Subject: Address IE8 hover effect issues. Closes #666.

---
 themes/default/js/ui.init.js | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

(limited to 'themes/default')

diff --git a/themes/default/js/ui.init.js b/themes/default/js/ui.init.js
index 93dfb275..e645eb71 100644
--- a/themes/default/js/ui.init.js
+++ b/themes/default/js/ui.init.js
@@ -91,12 +91,19 @@ $(document).ready(function() {
       },
       function() {
         // Reset item height, position, and z-index
-        var sib_height = $(this).next().height();
+        if ($(this).next().height()) {
+          var sib_height = $(this).next().height();
+        } else {
+          var sib_height = $(this).prev().height();
+        }
+        if ($.browser.msie && $.browser.version >= 8) {
+          sib_height = sib_height + 1;
+        }
         $(this).css("height", sib_height);
         $(this).css("position", "relative");
         $(this).css("top", null);
         $(this).css("left", null);
-        $(this).css("z-index", null);
+        $(this).css("z-index", 1);
         // Remove the placeholder and hover class from the item
         $("#gPlaceHolder").remove();
         $(this).removeClass("gHoverItem");
-- 
cgit v1.2.3