Changeset 57

Timestamp:

06/17/07 00:20:47 (2 years ago)

Author:

Stuart Thiel

Message:

-Fixed #54
-The applet just loads the PlayerPanel? into the applet
-I made the applet dimensions much bigger (maybe this should be determined dynamically based on the size of the window and the size of the Zotero pullup)
-Additional change to #57
Instead of making a new tab or looking at how we click on items, we're now stealing the behaviour of the view button in the info tab. If it gets clicked for a "Video Recording" (type 28), then it'll load the VideAnnotate? applet with the URL as parameter.

Files:

• VideAnnotateXPI/trunk/srcExtension/chrome.manifest (modified) (1 diff)
• VideAnnotateXPI/trunk/srcExtension/chrome/VideAnnotate/content/applet.html (modified) (1 diff)
• VideAnnotateXPI/trunk/srcExtension/chrome/VideAnnotate/content/itemPane.xul (added)
• VideAnnotateXPI/trunk/srcExtension/chrome/VideAnnotate/content/overlay.js (modified) (1 diff)
• VideAnnotateXPI/trunk/srcExtension/chrome/VideAnnotate/content/overlay.xul (deleted)

VideAnnotateXPI/trunk/srcExtension/chrome.manifest

@@ -2 +2 @@
-
-overlay chrome://browser/content/browser.xul chrome://videannotate/content/overlay.xul
-
+
+
-
-skin videannotate global chrome/videannotate/skin/global/

VideAnnotateXPI/trunk/srcExtension/chrome/VideAnnotate/content/applet.html

@@ -8 +8 @@
-
-<applet code="org.videannotate.app.VideoAnnotater" 
-200 height=200
+"533" height="400"
-<script>
-var regexS3 = "[\\?&]url=([^&#]*)";

VideAnnotateXPI/trunk/srcExtension/chrome/VideAnnotate/content/overlay.js

@@ -13 +13 @@
-  isMineToDevour: function(item) {
-    // initialization code
+    if(item.getType() == 28) return true
-    if(!item) return false;
-    alert(item.getType());
-    return false;
-  }, 
-  
-
+
+
-    if(this.isMineToDevour(
-     XulThingy.itemsView._getItemAtRow(
-          XulThingy.itemsView.selection.currentIndex))) {
-
+
+
-    } else {
-Pane.itemSelected(
+ItemPane.onViewClick(button, event
-    }
-  }