From heath at outerspaceconsultants.com  Wed Feb 18 19:10:00 2004
From: heath at outerspaceconsultants.com (Heath S. Hendrickson)
Date: Wed Feb 18 19:09:50 2004
Subject: [ansel] questions about gallery object
Message-ID: <40342908.7040500@outerspaceconsultants.com>

How do I get the gallery name given a DataTreeObject_Gallery?  I thought 
it would be:

$gallery->get('name');

but this just returns the display name of the gallery, not the actual 
galleryId name.

Here's an example dump from print_r($gallery):

datatreeobject_gallery Object
(
    [data] => Array
        (
            [thumbnail_width] => 150
            [thumbnail_height] => 150
            [image_width] => 800
            [image_height] => 600
            [default] => 427
            [default-type] => auto
            [num-images] => 38
            [perm] => Array
                (
                    [guest] => 6
                    [default] => 6
                    [users] => Array
                        (
                            [heath] => 30
                        )

                )

            [type] => 1
            [num_images] => 0
            [desc] => Thanksgiving spent at the residence of John Kendrick in Mineral Wells, Texas
            [name] => Thanksgiving 2003
            [owner] => heath
        )

    [name] => Anna:AJH_Thanksgiving2003
    [order] => 

running $gallery->get('name') returns "Thanksgiving 2003" and not 
"Anna:AJH_Thanksgiving2003".  I had to resort to calling 
$ansel_shares->_datatree->getName($cid) to get the actual $galleryId value.

Am I missing something here?

heath


From chuck at horde.org  Wed Feb 18 19:14:01 2004
From: chuck at horde.org (Chuck Hagenbuch)
Date: Wed Feb 18 19:16:46 2004
Subject: [ansel] questions about gallery object
In-Reply-To: <40342908.7040500@outerspaceconsultants.com>
References: <40342908.7040500@outerspaceconsultants.com>
Message-ID: <20040218221401.qij3yocw4sw84gcw@marina.horde.org>

Quoting "Heath S. Hendrickson" <heath@outerspaceconsultants.com>:

> How do I get the gallery name given a DataTreeObject_Gallery?  I thought
> it would be:
>
> $gallery->get('name');
>
> but this just returns the display name of the gallery, not the actual
> galleryId name.

$gallery->getName().

-chuck

--
"Here, I brought some cole slaw. It's made from peeeooople! Just kidding."
From heath at outerspaceconsultants.com  Wed Feb 18 19:22:49 2004
From: heath at outerspaceconsultants.com (Heath S. Hendrickson)
Date: Wed Feb 18 19:22:40 2004
Subject: [ansel] Tree view almost done
Message-ID: <40342C09.406@outerspaceconsultants.com>

 I'm 99% done with the new tree view.  I didn't use Horde_Tree simply 
because I couldn't get my hands wrapped around how to use it.  I'm just 
having a problem with dynamically changing the link that the 
gallery_preview image goes to.  I've tried using the 
document.$owner_link.href = $owner_view_url[]; approach, but all I get 
are javascript errors complaining that document.$owner_link doesn't have 
any properties.

Here's the JS snippet:

link = document.getElementById('<?php echo $owner ?>_link');
link.href = <?php echo $owner ?>_view_url[$id];   // $id is passed into 
the JS


And here's the definition of the link:

<td class="preview"><?php echo Horde::link('" name="' . $owner . 
'_link"') . Horde::img('', _("Gallery Preview"), 'vpace="5" 
class="solidbox" name="' . $owner . "_preview"') ?></a></td>


The resulting output in the browser looks like:

The JS:

    link = document.getElementById('heath_link');
    link.href = heath_view_url[$id];


The link:

<td class="preview"><a href="" name="heath_link"><img src="/horde/ansel/graphics/" alt="Gallery Preview" title="Gallery Preview" vspace="5" class="solidbox" name="heath_preview" /></a></td>


Anyone have any suggestions on what I'm doing wrong here?

heath
From heath at outerspaceconsultants.com  Wed Feb 18 19:29:42 2004
From: heath at outerspaceconsultants.com (Heath S. Hendrickson)
Date: Wed Feb 18 19:29:30 2004
Subject: [ansel] questions about gallery object
In-Reply-To: <20040218221401.qij3yocw4sw84gcw@marina.horde.org>
References: <40342908.7040500@outerspaceconsultants.com>
	<20040218221401.qij3yocw4sw84gcw@marina.horde.org>
Message-ID: <40342DA6.8000507@outerspaceconsultants.com>

Gotcha.  I looked at Gallery.php and didn't see that fuction, so I 
searched up the class parents and found the one in DataTree, hence my 
kludge...

Thanks,
heath

Chuck Hagenbuch wrote:

> Quoting "Heath S. Hendrickson" <heath@outerspaceconsultants.com>:
>
>> How do I get the gallery name given a DataTreeObject_Gallery?  I thought
>> it would be:
>>
>> $gallery->get('name');
>>
>> but this just returns the display name of the gallery, not the actual
>> galleryId name.
>
>
> $gallery->getName().
>
> -chuck
>
> -- 
> "Here, I brought some cole slaw. It's made from peeeooople! Just 
> kidding."


From heath at outerspaceconsultants.com  Wed Feb 18 19:37:45 2004
From: heath at outerspaceconsultants.com (Heath S. Hendrickson)
Date: Wed Feb 18 19:37:38 2004
Subject: [ansel] Tree view almost done
In-Reply-To: <40342C09.406@outerspaceconsultants.com>
References: <40342C09.406@outerspaceconsultants.com>
Message-ID: <40342F89.9060205@outerspaceconsultants.com>

Wouldn't you know it... I've spent a lot of time researching this online 
and couldn't find an answer (I found all sorts of references to changing 
a Frame location href...)...  And then not 5 mintues after I type up 
this message and send it out do I find the answer.

I needed to make it <a href="" id=""> instead of <a href="" name="">.  
Doing that fixed it all up.

All I have left to do is create the template for the tree view that 
isn't groupByOwner and then I can submit the patches...

heath


Heath S. Hendrickson wrote:

> I'm 99% done with the new tree view.  I didn't use Horde_Tree simply 
> because I couldn't get my hands wrapped around how to use it.  I'm 
> just having a problem with dynamically changing the link that the 
> gallery_preview image goes to.  I've tried using the 
> document.$owner_link.href = $owner_view_url[]; approach, but all I get 
> are javascript errors complaining that document.$owner_link doesn't 
> have any properties.
>
> Here's the JS snippet:
>
> link = document.getElementById('<?php echo $owner ?>_link');
> link.href = <?php echo $owner ?>_view_url[$id];   // $id is passed 
> into the JS
>
>
> And here's the definition of the link:
>
> <td class="preview"><?php echo Horde::link('" name="' . $owner . 
> '_link"') . Horde::img('', _("Gallery Preview"), 'vpace="5" 
> class="solidbox" name="' . $owner . "_preview"') ?></a></td>
>
>
> The resulting output in the browser looks like:
>
> The JS:
>
>    link = document.getElementById('heath_link');
>    link.href = heath_view_url[$id];
>
>
> The link:
>
> <td class="preview"><a href="" name="heath_link"><img 
> src="/horde/ansel/graphics/" alt="Gallery Preview" title="Gallery 
> Preview" vspace="5" class="solidbox" name="heath_preview" /></a></td>
>
>
> Anyone have any suggestions on what I'm doing wrong here?
>
> heath


From heath at outerspaceconsultants.com  Wed Feb 18 22:14:08 2004
From: heath at outerspaceconsultants.com (Heath S. Hendrickson)
Date: Wed Feb 18 22:14:10 2004
Subject: [ansel] Tree view mods
Message-ID: <40345430.1040900@outerspaceconsultants.com>

Attached is a single unified diff (via 'cvs diff -u' on the ansel root
directory) and two new files(that both go into templates/list/).

   ansel_tree_view.patches
   tree_gallery.inc
   tree_owner.inc

It's based on the tile listing with some heavy modifications to provide 
the full <UL> styled listing of all galleries either by owner or as a 
single list.  The two views are slightly different for now, but I'm 
thinking of modifying the owner view to not provide multiple preview 
slots, and just look like the gallery view with owner headings.  That's 
for another day, though.

I've tested it out on my system and it works fine, but I only have one
user with galleries (albeit sub-galleries as well).

Anyway, suggestions for modifications or changes are welcome, especially
if someone has a cleaner way of doing things than what I've done.  I
originally wanted to use the onMouseOver to change the preview and links
and then use onClick to go to the gallery, but Horde::link is hardcoded
to only write the window.status() in the onMouseOver event and I didn't
feel like messing with core Horde functionality.

Note that the CSS isn't the best (red text with white background for the
hover event), but that's easily modified in the html.php file.  I just
wanted something so I could make sure it was working.

heath

-------------- next part --------------
<script language="Javascript" type="text/javascript">
<!--

<?php echo $owner ?>_previews = new Array();
<?php echo $owner ?>_gallery_size = new Array();
<?php echo $owner ?>_gallery_id = new Array();
<?php echo $owner ?>_gallery_desc = new Array();
<?php echo $owner ?>_view_url = new Array();
<?php echo $owner ?>_slide_url = new Array();
<?php echo $owner ?>_zip_url = new Array();

<?php

$i = 0;
function createJS(&$parent, $cid)
{
    global $ansel_shares, $i, $owner;
    $gallery = &$ansel_shares->getGalleryById($cid);
    //$galleryId = $ansel_shares->_datatree->getName($cid);
    $galleryId = $gallery->getName();

    echo $owner . '_previews[' . $i . '] = \'' . addslashes(str_replace('&amp;', '&', Ansel::getImageUrl($gallery->getDefaultImage(), 'thumb'))) . "';\n";
    echo $owner . '_gallery_size[' . $i . '] = ' . $gallery->get('num-images') . ";\n";
    echo $owner . '_gallery_id[' . $i . '] = \'' . addslashes($galleryId) . "';\n";
    echo $owner . '_gallery_desc[' . $i . '] = \'' . addslashes($gallery->get('desc')) . "';\n";
    echo $owner . '_view_url[' . $i . '] = \'' . Horde::applicationUrl(Util::addParameter('view.php', 'gallery', $galleryId)) . "';\n";
    echo $owner . '_slide_url[' . $i . '] = \'' . Horde::applicationUrl(Util::addParameter('slideshow.php', 'gallery', $galleryId)) . "';\n";
    $galleryurl = Util::addParameter('gallery.php', 'gallery', $galleryId);
    echo $owner . '_zip_url[' . $i . '] = \'' . Horde::applicationUrl(Util::addParameter($galleryurl, 'actionID', 'downloadzip')) . "';\n";
    $i++;

    if (is_array($parent)) {
        array_walk($parent, 'createJS');
        return;
    }
}

foreach ($tree as $parentId => $children) {
    array_walk($children, 'createJS');
}
?>

function <?php echo $owner ?>_updateTile($id)
{
    size = <?php echo $owner ?>_gallery_size[$id];
    desc = <?php echo $owner ?>_gallery_desc[$id];
    preview = <?php echo $owner ?>_previews[$id];
    link = document.getElementById('<?php echo $owner; ?>_link');
    link.href = <?php echo $owner ?>_view_url[$id];

    html = desc + ' (';
    if (size == 1) {
        html += '<?php echo _("1 image") ?>';
    } else {
        html += size + ' <?php echo _("images") ?>';
    }
    html += ')';

    if (size > 0) {
        html += '<br /><br /><?php echo Horde::link(sprintf("javascript: %s_downloadzip(' + \$id + ')",$owner), false, -1) . Horde::img('mime/compressed.gif', _("Download all images in a .zip file"), '', $registry->getParam('graphics', 'horde')) . ' ' . _("Download") ?></a>';
        html += '<br /><?php echo Horde::link(sprintf("javascript:%s_slideshow(' + \$id + ')", $owner), false, -1) . Horde::img('slideshow.gif', _("Slide show")) . ' ' . _("Slide show") ?></a>';
    }

    spanObj = document.getElementById('<?php echo $owner; ?>_gallery_information');
    spanObj.innerHTML = html;

    document['<?php echo $owner ?>_preview'].src = preview;
}

function <?php echo $owner ?>_downloadzip($id)
{
    document.location = <?php echo $owner ?>_zip_url[$id];
}

function <?php echo $owner ?>_slideshow($id)
{
    document.location = <?php echo $owner ?>_slide_url[$id];
}

//-->
</script>

<table border="0" cellpadding="0" cellspacing="0" class="greybox" width="600">
 <tr>
  <td class="header" colspan="2">
   <?php echo $title ?>
  </td>
 </tr>
 <tr>
  <td class="preview"><?php echo Horde::link('" id="' . $owner . '_link"') . Horde::img('', _("Gallery Preview"), 'vspace="5" class="solidbox" name="' . $owner . '_preview"') ?></a></td>
  <td valign="top">
   <table border="0" cellpadding="3" cellspacing="0" width="100%">
    <tr>
     <td>

      <div id="tree">
       <form name="<?php echo $owner ?>_galleries" action="view.php" method="get">

<?php

$j = 0;
function list_print(&$parent, $key, $subcount)
{
    global $ansel_shares, $j;
    $gallery = &$ansel_shares->getGalleryById($key);
    //$galleryId = $ansel_shares->_datatree->getName($key);
    $galleryId = $gallery->getName();
    $viewurl = Horde::applicationUrl(Util::addParameter('view.php', 'gallery', $galleryId));
    $slideurl = Horde::applicationUrl(Util::addParameter('slideshow.php', 'gallery', $galleryId));
    $galleryurl = Util::addParameter('gallery.php', 'gallery', $galleryId);
    $zipurl = Horde::applicationUrl(Util::addParameter($galleryurl, 'actionID', 'downloadzip'));
    //$link = Horde::link($viewurl, sprintf(_("View %s"), $gallery->get('name')), '', '' , sprintf("%s_updateTile(); return true;",$gallery->get('owner')));
    $link = Horde::link('#', sprintf(_("View %s"), $gallery->get('name')), '', '' , sprintf("%s_updateTile(%s); return true;",$gallery->get('owner'),$j));
    $j++;

    if (is_array($parent) && $ansel_shares->_datatree->getNumberOfChildren($gallery) > 0) {
        echo "  <LI>" . $link . htmlspecialchars($gallery->get('name')) . "</a>";
        $subcount++;
        echo "    <UL>\n";
        array_walk($parent, 'list_print', $subcount);
        for (;$subcount > 0; $subcount--) {
            echo "    </UL>\n";
        }
        echo "  </LI>\n";
        return;
    } else {
        echo "  <LI>" . $link . htmlspecialchars($gallery->get('name')) . "</a></LI>\n";
    }
}
echo "<UL>\n";
foreach ($tree as $parentId => $children) {
    array_walk($children, 'list_print', 0);
}
echo "</UL>\n";

echo "</div>\n";

?>

        <br /><br />
        <span id="<?php echo $owner ?>_gallery_information"></span>
      </form>
      <script language="Javascript" type="text/javascript">
        <?php echo $owner ?>_updateTile(0);
      </script>
     </td>
    </tr>
   </table>
  </td>
 </tr>
</table>
<br />
-------------- next part --------------
Index: list.php
===================================================================
RCS file: /usr/local/horde/cvs/ansel/list.php,v
retrieving revision 1.31
diff -u -r1.31 list.php
--- list.php	7 Feb 2004 17:43:32 -0000	1.31
+++ list.php	19 Feb 2004 04:28:55 -0000
@@ -33,11 +33,20 @@
             $identity = &Identity::singleton('none', $owner);
             $name = $identity->getValue('fullname') == '' ? $owner : $identity->getValue('fullname');
             $title = _(sprintf("%s's Galleries", $name));
-            $title .= ' (' . count($galleries) . ')';
-
-            if ($prefs->getValue('listtype') != 'list') {
+            if ($prefs->getValue('listtype') == 'tree') {
+                $galleryCount = 0;
+                foreach ($galleries as $galleryId => $gallery) {
+                    $tree[$galleryId] = Ansel::getFullTree($galleryId);
+                    $galleryCount += $ansel_shares->_datatree->getNumberOfChildren($gallery);
+                }
+                $galleryCount += count($galleries);
+                $title .= ' (' . $galleryCount . ')';
+                require ANSEL_TEMPLATES . '/list/tree_owner.inc';
+            } elseif ($prefs->getValue('listtype') == 'tile') {
+                $title .= ' (' . count($galleries) . ')';
                 require ANSEL_TEMPLATES . '/list/tile_owner.inc';
             } else {
+                $title .= ' (' . count($galleries) . ')';
                 require ANSEL_TEMPLATES . '/list/header.inc';
                 require ANSEL_TEMPLATES . '/list/gallery_headers.inc';
                 foreach ($galleries as $galleryId => $gallery) {
@@ -64,7 +73,12 @@
     require ANSEL_TEMPLATES . '/common-header.inc';
     Ansel::menu();
     if (count($galleries) > 0) {
-        if ($prefs->getValue('listtype') != 'list') {
+        if ($prefs->getValue('listtype') == 'tree') {
+            foreach ($galleries as $galleryId => $gallery) {
+                $tree[$galleryId] = Ansel::getFullTree($galleryId);
+            }
+            require ANSEL_TEMPLATES . '/list/tree_gallery.inc';
+        } elseif ($prefs->getValue('listtype') == 'tile') {
             $template = ANSEL_TEMPLATES . '/list/tile_gallery.inc';
         } else {
             $template = ANSEL_TEMPLATES . '/list/gallery_summaries.inc';
@@ -72,12 +86,14 @@
             require ANSEL_TEMPLATES . '/list/gallery_headers.inc';
         }
 
-        foreach ($galleries as $galleryId => $gallery) {
-            $viewurl = Horde::applicationUrl(Util::addParameter('view.php', 'gallery', $galleryId));
-            $slideurl = Horde::applicationUrl(Util::addParameter('slideshow.php', 'gallery', $galleryId));
-            $galleryurl = Util::addParameter('gallery.php', 'gallery', $galleryId);
-            $zipurl = Horde::applicationUrl(Util::addParameter($galleryurl, 'actionID', 'downloadzip'));
-            require $template;
+        if ($prefs->getValue('listtype') != 'tree') {
+            foreach ($galleries as $galleryId => $gallery) {
+                $viewurl = Horde::applicationUrl(Util::addParameter('view.php', 'gallery', $galleryId));
+                $slideurl = Horde::applicationUrl(Util::addParameter('slideshow.php', 'gallery', $galleryId));
+                $galleryurl = Util::addParameter('gallery.php', 'gallery', $galleryId);
+                $zipurl = Horde::applicationUrl(Util::addParameter($galleryurl, 'actionID', 'downloadzip'));
+                require $template;
+            }
         }
 
         if ($prefs->getValue('listtype') == 'list') {
Index: config/html.php.dist
===================================================================
RCS file: /usr/local/horde/cvs/ansel/config/html.php.dist,v
retrieving revision 1.7
diff -u -r1.7 html.php.dist
--- config/html.php.dist	14 Nov 2003 19:31:52 -0000	1.7
+++ config/html.php.dist	19 Feb 2004 04:33:49 -0000
@@ -42,3 +42,12 @@
 $css['.preview']['text-align'] = 'center';
 $css['.preview']['border-right'] = '1px solid #999999';
 $css['.preview']['background-color'] = $css['.item0']['background-color'];
+
+$css['#tree ul']['list-style'] = 'none';
+$css['#tree ul']['margin-left'] = '0';
+$css['#tree ul']['padding-left'] = '1em';
+$css['#tree ul']['text-indent'] = '-1em';
+$css['#tree ul li']['list-style'] = 'none';
+$css['#tree ul li']['list-style-image'] = 'none';
+$css['#tree li a:hover']['color'] = 'red';
+$css['#tree li a:hover']['background-color'] = 'white';
Index: lib/Ansel.php
===================================================================
RCS file: /usr/local/horde/cvs/ansel/lib/Ansel.php,v
retrieving revision 1.48
diff -u -r1.48 Ansel.php
--- lib/Ansel.php	11 Feb 2004 22:17:10 -0000	1.48
+++ lib/Ansel.php	18 Feb 2004 16:56:23 -0000
@@ -26,6 +26,8 @@
      * @param string  $parent      (optional) The parent share to start searching at.
      * @param boolean $allLevels   (optional) Return all levels, or just the direct children
      *                             of $parent? Defaults to all levels.
+     * @param string  $sortby      (optional) The sorting key
+     * @param string  $sortdir     (optional) The sorting direction.
      *
      * @return array  Returns a list of matching, sorted galleries.
      */
@@ -40,9 +42,9 @@
 
         /* Sorting criteria for the gallery list. */
         $sort_functions = array(
-            ANSEL_SORT_NAME => 'ByName',
-            ANSEL_SORT_DESC => 'ByDesc',
-            ANSEL_SORT_OWNER => 'ByOwner',
+            ANSEL_SORT_NAME => 'ByName',        // 0
+            ANSEL_SORT_DESC => 'ByDesc',        // 1
+            ANSEL_SORT_OWNER => 'ByOwner',      // 2
         );
 
         $galleries = $ansel_shares->listShares(Auth::getAuth(), $perm, false, $parent, $allLevels);
@@ -63,6 +65,21 @@
     }
 
     /**
+     * Returns the full tree of the galleryId passed in.
+     *
+     * @param integer $pid  (optional) the starting gallery for the tree
+     *
+     * @return array  Returns a tree array of galleries.
+     */
+    function getFullTree($pid = '-1')
+    {
+        global $ansel_shares;
+
+        return $ansel_shares->_datatree->get(DATATREE_FORMAT_TREE, $pid);
+    }
+
+
+    /**
      * Retrieves the current user's gallery list from storage arranged by
      * the galleries' owner.
      *
@@ -89,6 +106,10 @@
 
         /* Return empty lists now. */
         if (!count($galleries)) {
+        }
+
+        /* Return empty lists now. */
+        if (!count($galleries)) {
             return $galleries;
         }
 
@@ -279,10 +300,6 @@
 
     /**
      * Comparison function for sorting galleries by owner.
-     *
-     * @param array $a  Gallery one.
-     * @param array $b  Gallery two.
-     *
      * @return integer  1 if gallery one is greater, -1 if gallery two is greater; 0 if they are equal.
      */
     function _sortByOwner($a, $b)
-------------- next part --------------
<script language="Javascript" type="text/javascript">
<!--

all_previews = new Array();
all_gallery_size = new Array();
all_gallery_id = new Array();
all_gallery_desc = new Array();
all_view_url = new Array();
all_slide_url = new Array();
all_zip_url = new Array();

<?php

$i = 0;
function createJS(&$parent, $cid)
{
    global $ansel_shares, $i;
    $gallery = &$ansel_shares->getGalleryById($cid);
    $galleryId = $gallery->getName();

    echo 'all_previews[' . $i . '] = \'' . addslashes(str_replace('&amp;', '&', Ansel::getImageUrl($gallery->getDefaultImage(), 'thumb'))) . "';\n";
    echo 'all_gallery_size[' . $i . '] = ' . $gallery->get('num-images') . ";\n";
    echo 'all_gallery_id[' . $i . '] = \'' . addslashes($galleryId) . "';\n";
    echo 'all_gallery_desc[' . $i . '] = \'' . addslashes($gallery->get('desc')) . "';\n";
    echo 'all_view_url[' . $i . '] = \'' . Horde::applicationUrl(Util::addParameter('view.php', 'gallery', $galleryId)) . "';\n";
    echo 'all_slide_url[' . $i . '] = \'' . Horde::applicationUrl(Util::addParameter('slideshow.php', 'gallery', $galleryId)) . "';\n";
    $galleryurl = Util::addParameter('gallery.php', 'gallery', $galleryId);
    echo 'all_zip_url[' . $i . '] = \'' . Horde::applicationUrl(Util::addParameter($galleryurl, 'actionID', 'downloadzip')) . "';\n";
    $i++;

    if (is_array($parent)) {
        array_walk($parent, 'createJS');
        return;
    }
}

foreach ($tree as $parentId => $children) {
    array_walk($children, 'createJS');
}
?>

function all_updateTile($id)
{
    size = all_gallery_size[$id];
    desc = all_gallery_desc[$id];
    preview = all_previews[$id];
    link = document.getElementById('all_link');
    link.href = all_view_url[$id];

    html = desc + ' (';
    if (size == 1) {
        html += '<?php echo _("1 image") ?>';
    } else {
        html += size + ' <?php echo _("images") ?>';
    }
    html += ')';

    if (size > 0) {
        html += '<br /><br /><?php echo Horde::link("javascript: all_downloadzip(' + \$id + ')", false, -1) . Horde::img('mime/compressed.gif', _("Download all images in a .zip file"), '', $registry->getParam('graphics', 'horde')) . ' ' . _("Download") ?></a>';
        html += '<br /><?php echo Horde::link("javascript:all_slideshow(' + \$id + ')", false, -1) . Horde::img('slideshow.gif', _("Slide show")) . ' ' . _("Slide show") ?></a>';
    }

    spanObj = document.getElementById('all_gallery_information');
    spanObj.innerHTML = html;

    document['all_preview'].src = preview;
}

function all_downloadzip($id)
{
    document.location = all_zip_url[$id];
}

function all_slideshow($id)
{
    document.location = all_slide_url[$id];
}

//-->
</script>

<table border="0" cellpadding="0" cellspacing="0" class="greybox" width="600">
 <tr>
  <td class="header" colspan="2">
   <?php echo $title ?>
  </td>
 </tr>
 <tr>
  <td class="preview"><?php echo Horde::link('" id="all_link') . Horde::img('', _("Gallery Preview"), 'vspace="5" class="solidbox" name="all_preview"') ?></a></td>
  <td valign="top">
   <table border="0" cellpadding="3" cellspacing="0" width="100%">
    <tr>
     <td>

      <div id="tree">
       <form name="all_galleries" action="view.php" method="get">

<?php

$j = 0;
function list_print(&$parent, $key, $subcount)
{
    global $ansel_shares, $j;
    $gallery = &$ansel_shares->getGalleryById($key);
    $galleryId = $gallery->getName();
    $viewurl = Horde::applicationUrl(Util::addParameter('view.php', 'gallery', $galleryId));
    $slideurl = Horde::applicationUrl(Util::addParameter('slideshow.php', 'gallery', $galleryId));
    $galleryurl = Util::addParameter('gallery.php', 'gallery', $galleryId);
    $zipurl = Horde::applicationUrl(Util::addParameter($galleryurl, 'actionID', 'downloadzip'));
    //$link = Horde::link($viewurl, sprintf(_("View %s"), $gallery->get('name')), '', '' , sprintf("%s_updateTile(); return true;",$gallery->get('owner')));
    $link = Horde::link('#', sprintf(_("View %s"), $gallery->get('name')), '', '' , sprintf("all_updateTile(%s); return true;",$j));
    $j++;

    if (is_array($parent) && $ansel_shares->_datatree->getNumberOfChildren($gallery) > 0) {
        echo "  <LI>" . $link . htmlspecialchars($gallery->get('name')) . "</a>";
        $subcount++;
        echo "    <UL>\n";
        array_walk($parent, 'list_print', $subcount);
        for (;$subcount > 0; $subcount--) {
            echo "    </UL>\n";
        }
        echo "  </LI>\n";
        return;
    } else {
        echo "  <LI>" . $link . htmlspecialchars($gallery->get('name')) . "</a></LI>\n";
    }
}
echo "<UL>\n";
foreach ($tree as $parentId => $children) {
    array_walk($children, 'list_print', 0);
}
echo "</UL>\n";

echo "</div>\n";

?>

        <br /><br />
        <span id="all_gallery_information"></span>
      </form>
      <script language="Javascript" type="text/javascript">
        all_updateTile(0);
      </script>
     </td>
    </tr>
   </table>
  </td>
 </tr>
</table>
<br />
From chuck at horde.org  Thu Feb 19 12:57:42 2004
From: chuck at horde.org (Chuck Hagenbuch)
Date: Thu Feb 19 13:00:37 2004
Subject: [ansel] Tree view mods
In-Reply-To: <40345430.1040900@outerspaceconsultants.com>
References: <40345430.1040900@outerspaceconsultants.com>
Message-ID: <20040219155742.kxrd9a8w88kk48oo@marina.horde.org>

Quoting "Heath S. Hendrickson" <heath@outerspaceconsultants.com>:

> Attached is a single unified diff (via 'cvs diff -u' on the ansel root
> directory) and two new files(that both go into templates/list/).

Can you send a screenshot of this? It's great to have the patch written and in
the archives, but for now, I'm going to hold out for someone (maybe me) having
time to do a Horde_Tree based view.

Thanks!

-chuck

--
"Here, I brought some cole slaw. It's made from peeeooople! Just kidding."
From heath at outerspaceconsultants.com  Thu Feb 19 15:08:06 2004
From: heath at outerspaceconsultants.com (Heath S. Hendrickson)
Date: Thu Feb 19 15:07:57 2004
Subject: [ansel] Tree view mods
In-Reply-To: <20040219155742.kxrd9a8w88kk48oo@marina.horde.org>
References: <40345430.1040900@outerspaceconsultants.com>
	<20040219155742.kxrd9a8w88kk48oo@marina.horde.org>
Message-ID: <403541D6.7040909@outerspaceconsultants.com>

You can check it out at:

https://www.outerspaceconsultants.com/horde/ansel/

I had to make a small change to prefs.php to default to 'tree' instead 
of 'tile', but otherwise it's the same code as what I sent in.


heath


Chuck Hagenbuch wrote:

> Quoting "Heath S. Hendrickson" <heath@outerspaceconsultants.com>:
>
>> Attached is a single unified diff (via 'cvs diff -u' on the ansel root
>> directory) and two new files(that both go into templates/list/).
>
>
> Can you send a screenshot of this? It's great to have the patch 
> written and in
> the archives, but for now, I'm going to hold out for someone (maybe 
> me) having
> time to do a Horde_Tree based view.
>
> Thanks!
>
> -chuck
>
> -- 
> "Here, I brought some cole slaw. It's made from peeeooople! Just 
> kidding."

From ben at chavet.net  Sun Feb 22 19:08:51 2004
From: ben at chavet.net (Ben Chavet)
Date: Sun Feb 22 19:09:47 2004
Subject: [ansel] Tree view mods
In-Reply-To: <403541D6.7040909@outerspaceconsultants.com>
References: <40345430.1040900@outerspaceconsultants.com>
	<20040219155742.kxrd9a8w88kk48oo@marina.horde.org>
	<403541D6.7040909@outerspaceconsultants.com>
Message-ID: <20040222210851.kn628s0cc0kook4g@mail.chavet.net>

I think this has potential, but I have a suggestion:

We already use _screen and _thumb images, what if we added an _icon image that
is like 16x16 or 20x20 or whatever. We can use the _icon image beside a tree
node or in the current list view.  It would still give a preview, but in a
lower-profile manner to keep the list shorter (space-wise).  I like the
changing preview image, but I don't think it will work very well when 
you get a
lot of galleries in the tree.

Opinions, please.  Volunteer(s) would be good too, as I still don't have much
spare time :)

--Ben

Quoting "Heath S. Hendrickson" <heath@outerspaceconsultants.com>:

> You can check it out at:
>
> https://www.outerspaceconsultants.com/horde/ansel/
>
> I had to make a small change to prefs.php to default to 'tree' instead
> of 'tile', but otherwise it's the same code as what I sent in.
>
>
> heath
>
>
> Chuck Hagenbuch wrote:
>
>> Quoting "Heath S. Hendrickson" <heath@outerspaceconsultants.com>:
>>
>>> Attached is a single unified diff (via 'cvs diff -u' on the ansel root
>>> directory) and two new files(that both go into templates/list/).
>>
>>
>> Can you send a screenshot of this? It's great to have the patch 
>> written and in
>> the archives, but for now, I'm going to hold out for someone (maybe 
>> me) having
>> time to do a Horde_Tree based view.
>>
>> Thanks!
>>
>> -chuck
>>
>> -- "Here, I brought some cole slaw. It's made from peeeooople! Just 
>> kidding."
>
> --
> ansel mailing list - Join the hunt: http://horde.org/bounties/#ansel
> Frequently Asked Questions: http://horde.org/faq/
> To unsubscribe, mail: ansel-unsubscribe@lists.horde.org
From heath at outerspaceconsultants.com  Sun Feb 22 19:44:30 2004
From: heath at outerspaceconsultants.com (Heath S. Hendrickson)
Date: Sun Feb 22 19:44:26 2004
Subject: [ansel] Tree view mods
In-Reply-To: <20040222210851.kn628s0cc0kook4g@mail.chavet.net>
References: <40345430.1040900@outerspaceconsultants.com>	<20040219155742.kxrd9a8w88kk48oo@marina.horde.org>	<403541D6.7040909@outerspaceconsultants.com>
	<20040222210851.kn628s0cc0kook4g@mail.chavet.net>
Message-ID: <4039771E.6000809@outerspaceconsultants.com>

Perhaps, but is the concern strictly one of real-estate? I think one 
preview image should be sufficient for all the galleries.  I just didn't 
have the time to rewrite the entire "groupbyowner" part of the list.php 
code, so I kept it with a preview per owner (if you have it not 
groupbyowner, then there is only one preview image anyway).  It 
shouldn't be too difficult to rewrite that page to process the tree view 
different from the other views (list and tile) giving only a single 
preview image.

The idea of an icon image is noteworthy, I'm just not sure how much use 
a 20x20 version of a 3000x2000 image is going to be.

How about I see if I can make a quick re-write of tree code to ensure 
that there is only one preview page for all the galleries for both 
"groupbyowner" and otherwise.

h


Ben Chavet wrote:

> I think this has potential, but I have a suggestion:
>
> We already use _screen and _thumb images, what if we added an _icon 
> image that
> is like 16x16 or 20x20 or whatever. We can use the _icon image beside 
> a tree
> node or in the current list view.  It would still give a preview, but 
> in a
> lower-profile manner to keep the list shorter (space-wise).  I like the
> changing preview image, but I don't think it will work very well when 
> you get a
> lot of galleries in the tree.
>
> Opinions, please.  Volunteer(s) would be good too, as I still don't 
> have much
> spare time :)
>
> --Ben
>

From ben at chavet.net  Sun Feb 22 20:06:17 2004
From: ben at chavet.net (Ben Chavet)
Date: Sun Feb 22 20:07:12 2004
Subject: [ansel] Tree view mods
In-Reply-To: <4039771E.6000809@outerspaceconsultants.com>
References: <40345430.1040900@outerspaceconsultants.com>
	<20040219155742.kxrd9a8w88kk48oo@marina.horde.org>
	<403541D6.7040909@outerspaceconsultants.com>
	<20040222210851.kn628s0cc0kook4g@mail.chavet.net>
	<4039771E.6000809@outerspaceconsultants.com>
Message-ID: <20040222220617.9c7lwc84gsksks84@mail.chavet.net>

> Perhaps, but is the concern strictly one of real-estate? I think one
> preview image should be sufficient for all the galleries.  I just didn't
> have the time to rewrite the entire "groupbyowner" part of the list.php
> code, so I kept it with a preview per owner (if you have it not
> groupbyowner, then there is only one preview image anyway).  It
> shouldn't be too difficult to rewrite that page to process the tree view
> different from the other views (list and tile) giving only a single
> preview image.

I guess I envision a groupbyowner tree view to be something like this:

     (-) [img] Parent Gallery
      |---[img] Sub-gallery1
      |-+-[img] Sub-gallery2
      | |---[img] Sub-subgallery
      |---[img] Sub-gallery3

With a simular tree for each user with a nice header with the owner's name.

Then a non-groupbyowner tree to be something like this:

    (+) User1
    (-) User2
     |---[img] Gallery1
       |---[img] Subgallery1

In other words, all of the groupbyowner trees bundled together into a 
tree.  It
could just be that we have different visions for the same idea.

> The idea of an icon image is noteworthy, I'm just not sure how much use
> a 20x20 version of a 3000x2000 image is going to be.

Yeah, I thought about that after I had already sent my message.  20x20 is
definitely too small, but perhaps some smaller version of the thumbnail.  I'm
just trying to think ahead.  I already have 60-70 galleries & that 
would take a
lot of screen realestate in a tree or list view.  As my gallery list grows, it
only gets worse.

> How about I see if I can make a quick re-write of tree code to ensure
> that there is only one preview page for all the galleries for both
> "groupbyowner" and otherwise.

The problem I see arising with having a single preview image for a tree 
is when
a user gets to have a lot of galleries.  The tree may be taller than the
browser window & to see the preview, you'd have to scroll back & forth.

--Ben

PS, I hope my sorry attempt at ascii-art doesn't get lost during the mailing
list process :)