Changeset 4517

Timestamp:

Dec 19, 2009, 2:20:19 PM (14 years ago)

Author:

grum

Message:

[Template:Gally] Fixes some bugs

Location:

extensions/gally

Files:

• comment_list.tpl (modified) (2 diffs)
• gallyjs-tpp.js (modified) (1 diff)
• picture.tpl (modified) (1 diff)
• release_notes.txt (modified) (1 diff)
• theme/graphite/theme.css (modified) (2 diffs)
• theme/grum-dark-II/theme.css (modified) (2 diffs)

extensions/gally/comment_list.tpl

@@ +1 @@
+{*
+ The comment list is defined inside a tabsheet made with the jQuery $().tabs()
+ With jQuery 1.2.6 (don't know how other release manage this) all the anchors <A> tags are considered as a tab.
+ So, can't use <A href="link"> because href is modified by the jQuery function.
+ Links are managed by an event onclick="document.location='link';"
+*}
 
 <ul class="commentList">
 {foreach from=$comments item=comment}
 <li>
-        <div class="commentDetail">
+  <div class="commentDetail">
     {if isset($comment.TN_SRC)}
     <div class="illustration">
@@ -16 +22 @@
         {if isset($comment.U_DELETE)}
         <li>
-          <a href="{$comment.U_DELETE}" title="{'delete this comment'|@translate}">
-            <img src="{$ROOT_URL}{$themeconf.icon_dir}/delete.png" class="button" alt="[delete]" />
-          </a>
+          <img style="cursor:pointer;" src="{$ROOT_URL}{$themeconf.icon_dir}/delete.png" class="button" alt="[delete]" onclick="document.location='{$comment.U_DELETE}';" title="{'delete this comment'|@translate}"/>
         </li>
         {/if}
         {if isset($comment.U_VALIDATE)}
         <li>
-          <a href="{$comment.U_VALIDATE}" title="validate this comment">
-            <img src="{$ROOT_URL}{$themeconf.icon_dir}/validate_s.png" class="button" alt="[validate]" />
-          </a>
+          <img style="cursor:pointer;" src="{$ROOT_URL}{$themeconf.icon_dir}/validate_s.png" class="button" alt="[validate]" onclick="document.location='{$comment.U_VALIDATE}';" title="{'validate this comment'|@translate}"/>
         </li>
         {/if}

extensions/gally/gallyjs-tpp.js

@@ -89 +89 @@
   if(options.animateDelay>0)
   {
-    $("#theTabs").tabs({ fx: { opacity: 'toggle', duration:options.animateDelay } });
+    $("#theTabs").tabs({ fx: { opacity: 'toggle', duration:options.animateDelay } } );
   }
   else

extensions/gally/picture.tpl

@@ -252 +252 @@
       "pwg.images.setPrivacyLevel", {image_id: id, level:level} ,
       {
+        method: "POST",
         onFailure: function(num, text) { selectElement.disabled = false; alert(num + " " + text); },
         onSuccess: function(result) { selectElement.disabled = false; }

extensions/gally/release_notes.txt

@@ -49 +49 @@
   |         |            | * Compatibility with the plugin VD language switch
   |         |            | * report some changes made on the "yoga" template
-  |         |            |   . r3060, r3121, r3137, r3147, r3196, r3538
+  |         |            |   . r3060, r3121, r3137, r3147, r3196, r3538, r4512
   |         |            |
   |         |            |

extensions/gally/theme/graphite/theme.css

@@ -454 +454 @@
 
 /*
+ * page :profile
+*/
+form#profile {
+  padding-bottom:20px;
+}
+
+
+/*
  * page :search
 */
@@ -481 +489 @@
 #menubar {
  border:1px #606060 solid;
+ padding-bottom:30px;
 }
 

extensions/gally/theme/grum-dark-II/theme.css

@@ -513 +513 @@
 
 /*
+ * page :profile
+*/
+form#profile {
+  padding-bottom:20px;
+}
+
+/*
  * page :search
 */
@@ -540 +547 @@
 #menubar {
   border:1px #303030 solid;
+  padding-bottom:30px;
 }