From 48bba3036114075e6d5ae129fc3f9bbd0cd57956 Mon Sep 17 00:00:00 2001
From: riccardo <riccardo@parisson.com>
Date: Tue, 15 Mar 2011 18:37:45 +0100
Subject: [PATCH] added buttons, fixed marker problems

---
 telemeta/htdocs/css/telemeta.css                |  2 +-
 telemeta/htdocs/js/application.js               |  2 +-
 telemeta/htdocs/timeside/src/controller.js      | 11 +++++------
 telemeta/htdocs/timeside/src/markermap.js       | 17 ++++++++++-------
 .../telemeta_default/collection_add.html        | 14 +++++++++-----
 .../telemeta_default/collection_detail.html     | 10 +++++++---
 .../telemeta_default/collection_edit.html       | 14 +++++++++-----
 .../telemeta_default/collection_list.html       |  2 +-
 .../telemeta_default/mediaitem_add.html         |  4 ++--
 .../telemeta_default/mediaitem_copy.html        | 14 +++++++++-----
 .../telemeta_default/mediaitem_edit.html        |  2 +-
 .../telemeta_default/mediaitem_list.html        |  3 ++-
 12 files changed, 57 insertions(+), 38 deletions(-)

diff --git a/telemeta/htdocs/css/telemeta.css b/telemeta/htdocs/css/telemeta.css
index 49dbcd21..1c198e73 100644
--- a/telemeta/htdocs/css/telemeta.css
+++ b/telemeta/htdocs/css/telemeta.css
@@ -1002,7 +1002,7 @@ a.image-link {
     background-image: url('/images/cancel.png');
 }
 .mediaitem_button_save{
-    color:#6A0307 !important;
+    /*color:#6A0307 !important;*/
     background-image: url('/images/save.png');
 }
 .mediaitem_button_add{
diff --git a/telemeta/htdocs/js/application.js b/telemeta/htdocs/js/application.js
index b73a1b0c..b10f200a 100644
--- a/telemeta/htdocs/js/application.js
+++ b/telemeta/htdocs/js/application.js
@@ -21,7 +21,7 @@ Array.prototype.move = function(from, to){
     }
     if(from != to){
         var elm = this.splice(from,1)[0];
-        this.markers.splice(to,0,elm);
+        this.splice(to,0,elm);
         return to;
     }
     return from;
diff --git a/telemeta/htdocs/timeside/src/controller.js b/telemeta/htdocs/timeside/src/controller.js
index e8f0935d..7636460f 100644
--- a/telemeta/htdocs/timeside/src/controller.js
+++ b/telemeta/htdocs/timeside/src/controller.js
@@ -73,10 +73,8 @@ TimeSide(function($N) {
 //            }
             this.cfg.divmarkers.move(from,to);
             this.cfg.player.ruler.markers.move(from,to);
-//            this.cfg.player.ruler.move(from,to);
-//            var m = this.cfg.divmarkers.splice(from,1)[0]; //remove
-//            this.cfg.divmarkers.splice(to,0,m); //add
-            this.updateIndices(from,to);
+//           realIndex might not be equal to to
+            this.updateIndices(from,data.newIndex);
         },
 
         //called whenever a marker is added to the ruler BUT NOT in the map
@@ -132,11 +130,12 @@ TimeSide(function($N) {
             if(from===undefined || from==null){
                 from = 0;
             }
-            if(from>this.cfg.divmarkers.length){
+            var len = this.cfg.divmarkers.length-1;
+            if(from>len){
                 return;
             }
             if(to==undefined || to ==null){
-                to = this.cfg.divmarkers.length-1;
+                to = len;
             }
             if(to<from){
                 var tmp = to;
diff --git a/telemeta/htdocs/timeside/src/markermap.js b/telemeta/htdocs/timeside/src/markermap.js
index 230dc448..6bf72896 100644
--- a/telemeta/htdocs/timeside/src/markermap.js
+++ b/telemeta/htdocs/timeside/src/markermap.js
@@ -97,15 +97,16 @@ TimeSide(function($N, $J) {
 
         move: function(markerIndex, newOffset){
             var newIndex = this.indexOf(newOffset);
-            newIndex = this.markers.move(markerIndex,newIndex);
+            var realIndex = this.markers.move(markerIndex,newIndex);
 
-            var marker = this.markers[newIndex];
+            var marker = this.markers[realIndex];
             marker.offset = newOffset;
             marker.isSaved = marker.isEditable ? false : true;
             
             this.fire('moved', {
                 fromIndex: markerIndex,
-                toIndex: newIndex
+                toIndex: newIndex,
+                newIndex: realIndex
             });
 
 //            var newIndex = this.indexOf(newOffset);
@@ -195,10 +196,11 @@ TimeSide(function($N, $J) {
             //see http://stackoverflow.com/questions/4809157/i-need-to-pass-a-json-object-to-a-javascript-ajax-method-for-a-wcf-call-how-can
             //            var data2send = '{"id":"jsonrpc", "params":[{"item_id":"'+ itemid+'", "public_id": "'+marker.id+'", "time": "'+
             //            marker.offset+'","description": "'+marker.desc+'"}], "method":"telemeta.add_marker","jsonrpc":"1.0"}';
-           
-            var isSaved = marker.id !== undefined;
+
+           var id = marker.id;
+            var isSaved = id !== undefined;
             if(!isSaved){
-                marker.id=this.uniqid(); //defined in core;
+                id=this.uniqid(); //defined in core;
             }
             var method = isSaved ? "telemeta.update_marker" : "telemeta.add_marker";
             
@@ -210,7 +212,7 @@ TimeSide(function($N, $J) {
                 offset = "0.0";
             }
             var data2send = '{"id":"jsonrpc", "params":[{"item_id":"'+ s(itemid)+
-            '", "public_id": "'+s(marker.id)+'", "time": "'+s(offset)+
+            '", "public_id": "'+s(id)+'", "time": "'+s(offset)+
             '", "author": "'+s(marker.author)+
             '", "title": "'+s(marker.title)+
             '","description": "'+s(marker.desc)+'"}], "method":"'+method+'","jsonrpc":"1.0"}';
@@ -222,6 +224,7 @@ TimeSide(function($N, $J) {
                 data: data2send,
                 success: function(){
                     if(!isSaved){
+                        marker.id=id;
                         marker.isSaved = true;
                     }
                     if(functionOnSuccess){
diff --git a/telemeta/templates/telemeta_default/collection_add.html b/telemeta/templates/telemeta_default/collection_add.html
index 19b7d713..9a240238 100644
--- a/telemeta/templates/telemeta_default/collection_add.html
+++ b/telemeta/templates/telemeta_default/collection_add.html
@@ -3,8 +3,9 @@
 {% block submenu %}
     <div>    
      {% block tools %}
-     <a href="{% url telemeta-collections %}">
-          <img src="images/edit_cancel.png" style="vertical-align:middle" /></a>
+<!--     <a href="{% url telemeta-collections %}">
+          <img src="images/edit_cancel.png" style="vertical-align:middle" /></a>-->
+<a href="{% url telemeta-collections %}" class="mediaitem_button mediaitem_button_cancel">Cancel</a>
      {% endblock tools %}
     </div>
 {% endblock %}
@@ -12,7 +13,7 @@
 
 {% block infos %}
         <div class="infos">
-        <form method="POST" action="">{% csrf_token %}
+        <form method="POST" id ="_addCollectionForm" action="">{% csrf_token %}
         <table>
         {% for field in form %}
             {% if not field.html_name == "copied_from_item" %}
@@ -24,9 +25,12 @@
         {% endfor %}
         </table>
         <div align="center">
-         <a href="{% url telemeta-collections %}">
+<!--         <a href="{% url telemeta-collections %}">
           <img src="images/edit_cancel.png" style="vertical-align:middle" /></a>
-         <input type="image" src="images/save_changes.png" style="vertical-align:middle" value="save"/>
+         <input type="image" src="images/save_changes.png" style="vertical-align:middle" value="save"/>-->
+<a href="{% url telemeta-collections %}" class="mediaitem_button mediaitem_button_cancel">Cancel</a>
+         <a href="#" class="mediaitem_button mediaitem_button_save"
+   onclick="document.getElementById('_addCollectionForm').submit(); return false;">Save</a>
         </div>
         </form>
         </div>
diff --git a/telemeta/templates/telemeta_default/collection_detail.html b/telemeta/templates/telemeta_default/collection_detail.html
index 5abd52f0..63637713 100644
--- a/telemeta/templates/telemeta_default/collection_detail.html
+++ b/telemeta/templates/telemeta_default/collection_detail.html
@@ -12,13 +12,17 @@
 
 {% block submenu %}
     <div>
-    <a href="{% url telemeta-collection-dublincore collection.public_id %}">Dublin Core</a>
+    <!--<a href="{% url telemeta-collection-dublincore collection.public_id %}">Dublin Core</a>-->
+    <a href="{% url telemeta-collection-dublincore collection.public_id %}" class="mediaitem_button mediaitem_button_dublin_core">Dublin Core</a>
     
     {% if user.is_authenticated and perms.telemeta.change_mediacollection %}
      {% block tools %}
-     <a href="{% url telemeta-collection-edit collection.public_id %}">
+    <!-- <a href="{% url telemeta-collection-edit collection.public_id %}">
       <img src="images/edit_page.png" alt="EDIT" style="vertical-align:middle" /></a>
-      <button><a href="{% url telemeta-collection-copy collection.public_id %}">Copy</a></button>
+      <button><a href="{% url telemeta-collection-copy collection.public_id %}">Copy</a></button>-->
+
+       <a href="{% url telemeta-collection-edit collection.public_id %}" class="mediaitem_button mediaitem_button_edit">Edit</a>
+      <a href="{% url telemeta-collection-copy collection.public_id %}" class="mediaitem_button mediaitem_button_copy">Copy</a>
      {% endblock tools %}
     {% endif %}
     
diff --git a/telemeta/templates/telemeta_default/collection_edit.html b/telemeta/templates/telemeta_default/collection_edit.html
index dfb3f4e1..b58c2f94 100644
--- a/telemeta/templates/telemeta_default/collection_edit.html
+++ b/telemeta/templates/telemeta_default/collection_edit.html
@@ -1,13 +1,14 @@
 {% extends "telemeta/collection_detail.html" %}
 
 {% block tools %}
-    <a href="{% url telemeta-collection-detail collection.public_id %}">
-     <img src="images/edit_cancel.png" style="vertical-align:middle" /></a>
+<!--    <a href="{% url telemeta-collection-detail collection.public_id %}">
+     <img src="images/edit_cancel.png" style="vertical-align:middle" /></a>-->
+     <a href="{% url telemeta-collection-detail collection.public_id %}">Cancel</a>
 {% endblock %}
 
 {% block infos %}
         <div class="infos">
-        <form method="POST" action="">{% csrf_token %}
+        <form method="POST" id="_editCollectionForm" action="">{% csrf_token %}
        <table>
        {% for field in form %}
         {% if not field.html_name == "copied_from_item" %}
@@ -19,9 +20,12 @@
        {% endfor %}
        </table>
         <div align="center">
-         <a href="{% url telemeta-collection-detail collection.public_id %}">
+<!--         <a href="{% url telemeta-collection-detail collection.public_id %}">
           <img src="images/edit_cancel.png" style="vertical-align:middle" /></a>
-         <input type="image" src="images/save_changes.png" style="vertical-align:middle" value="save"/>
+         <input type="image" src="images/save_changes.png" style="vertical-align:middle" value="save"/>-->
+         <a href="{% url telemeta-collection-detail collection.public_id %}">Cancel</a>
+         <a href="#" class="mediaitem_button mediaitem_button_save"
+   onclick="document.getElementById('_editCollectionForm').submit(); return false;">Save</a>
         </div>
         </form>
         </div>
diff --git a/telemeta/templates/telemeta_default/collection_list.html b/telemeta/templates/telemeta_default/collection_list.html
index 7cf243b6..1528c049 100644
--- a/telemeta/templates/telemeta_default/collection_list.html
+++ b/telemeta/templates/telemeta_default/collection_list.html
@@ -8,7 +8,7 @@
     <div class="buttons">  
     {% if user.is_authenticated and perms.telemeta.add_mediacollection %}
 <!--     <button><a href="{% url telemeta-collection-add %}">Add</a></button>-->
-      <a href="{% url telemeta-collection-add %}" class="roundBorder6 mediaitem_button mediaitem_button_add">Add</a>
+      <a href="{% url telemeta-collection-add %}" class="mediaitem_button mediaitem_button_add">Add</a>
 
     {% endif %}
     </div>
diff --git a/telemeta/templates/telemeta_default/mediaitem_add.html b/telemeta/templates/telemeta_default/mediaitem_add.html
index 3b7eafc5..2bc1a027 100644
--- a/telemeta/templates/telemeta_default/mediaitem_add.html
+++ b/telemeta/templates/telemeta_default/mediaitem_add.html
@@ -41,8 +41,8 @@
          <img src="images/edit_cancel.png" style="vertical-align:middle" /></a>
         <input type="image" src="images/save_changes.png" style="vertical-align:middle" value="save" />-->
         <a href="{% url telemeta-item-detail item.public_id %}"
-           class="roundBorder6 mediaitem_button mediaitem_button_cancel">Cancel</a>
-<a href="#" class="roundBorder6 mediaitem_button mediaitem_button_save"
+           class="mediaitem_button mediaitem_button_cancel">Cancel</a>
+<a href="#" class="mediaitem_button mediaitem_button_save"
    onclick="document.getElementById('_addItemForm').submit(); return false;">Save</a>
        </div>
      </form>
diff --git a/telemeta/templates/telemeta_default/mediaitem_copy.html b/telemeta/templates/telemeta_default/mediaitem_copy.html
index bb4c8797..28f6b182 100644
--- a/telemeta/templates/telemeta_default/mediaitem_copy.html
+++ b/telemeta/templates/telemeta_default/mediaitem_copy.html
@@ -11,8 +11,9 @@
 <div>
     {% if user.is_authenticated and perms.telemeta.add_mediaitem %}
     {% block tools %}
-     <a href="{% url telemeta-item-detail item.public_id %}">
-     <img src="images/edit_cancel.png" /></a>
+<!--     <a href="{% url telemeta-item-detail item.public_id %}">
+     <img src="images/edit_cancel.png" /></a>-->
+    <a href="{% url telemeta-item-detail item.public_id %}">Cancel</a>
     {% endblock tools %}
     {% endif %}
 </div>
@@ -23,7 +24,7 @@
   
   {% block infos %}    
     <div class="infos">
-     <form enctype="multipart/form-data" method="POST" action="">{% csrf_token %}
+     <form enctype="multipart/form-data" id="_mediaItemCopyForm" method="POST" action="">{% csrf_token %}
       <table>
        {% for field in form %}
         {% if not field.html_name == "copied_from_item" %}
@@ -35,9 +36,12 @@
        {% endfor %}
        </table>
        <div align="center">
-        <a href="{% url telemeta-item-detail item.public_id %}">
+<!--        <a href="{% url telemeta-item-detail item.public_id %}">
          <img src="images/edit_cancel.png" style="vertical-align:middle" /></a>
-        <input type="image" src="images/save_changes.png" style="vertical-align:middle" value="save" />
+        <input type="image" src="images/save_changes.png" style="vertical-align:middle" value="save" />-->
+<a href="{% url telemeta-item-detail item.public_id %}">Cancel</a>
+         <a href="#" class="mediaitem_button mediaitem_button_save"
+   onclick="document.getElementById('_mediaItemCopyForm').submit(); return false;">Save</a>
        </div>
      </form>
     </div>
diff --git a/telemeta/templates/telemeta_default/mediaitem_edit.html b/telemeta/templates/telemeta_default/mediaitem_edit.html
index e3b80b22..1b320363 100644
--- a/telemeta/templates/telemeta_default/mediaitem_edit.html
+++ b/telemeta/templates/telemeta_default/mediaitem_edit.html
@@ -3,7 +3,7 @@
     {% block tools %}
 <!--     <a href="{% url telemeta-item-detail item.public_id %}">
          <img src="images/edit_cancel.png" /></a>-->
- <a href="{% url telemeta-item-detail item.public_id %}"  class="mediaitem_button mediaitem_button_cancel">Cancel</a>
+ <a href="{% url telemeta-item-detail item.public_id %}" class="mediaitem_button mediaitem_button_cancel">Cancel</a>
     {% endblock tools %}
 
   {% block infos %}    
diff --git a/telemeta/templates/telemeta_default/mediaitem_list.html b/telemeta/templates/telemeta_default/mediaitem_list.html
index 5bbf9228..188bef52 100644
--- a/telemeta/templates/telemeta_default/mediaitem_list.html
+++ b/telemeta/templates/telemeta_default/mediaitem_list.html
@@ -8,7 +8,8 @@
 {% block submenu %}
     <div class="buttons">  
     {% if user.is_authenticated and perms.telemeta.add_mediaitem %}
-     <button><a href="{% url telemeta-item-add %}">Add</a></button>
+<!--     <button><a href="{% url telemeta-item-add %}">Add</a></button>-->
+<a href="{% url telemeta-item-add %}" class="mediaitem_button mediaitem_button_add">Add</a>
     {% endif %}
     </div>
 {% endblock %}
-- 
2.39.5