Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(331)

Unified Diff: samples/third_party/lightbox/lightbox-iframe.js

Issue 3083012: Tidying up. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/o3d/
Patch Set: '' Created 10 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « samples/third_party/codemirror/js/tokenize.js ('k') | samples/third_party/xmljs/COPYING » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: samples/third_party/lightbox/lightbox-iframe.js
===================================================================
--- samples/third_party/lightbox/lightbox-iframe.js (revision 55159)
+++ samples/third_party/lightbox/lightbox-iframe.js (working copy)
@@ -68,7 +68,7 @@
Event.observe(ctrl, 'click', this.activate.bindAsEventListener(this), false);
ctrl.onclick = function(){return false;};
},
-
+
// Turn everything on - mainly the IE fixes
activate: function(){
if (browser == 'Internet Explorer'){
@@ -79,18 +79,18 @@
}
this.displayLightbox("block");
},
-
+
// Ie requires height to 100% and overflow hidden or else you can scroll down past the lightbox
prepareIE: function(height, overflow){
bod = document.getElementsByTagName('body')[0];
bod.style.height = height;
bod.style.overflow = overflow;
-
+
htm = document.getElementsByTagName('html')[0];
htm.style.height = height;
- htm.style.overflow = overflow;
+ htm.style.overflow = overflow;
},
-
+
// In IE, select elements hover on top of the lightbox
hideSelects: function(visibility){
selects = document.getElementsByTagName('select');
@@ -98,37 +98,37 @@
selects[i].style.visibility = visibility;
}
},
-
+
// Taken from lightbox implementation found at http://www.huddletogether.com/projects/lightbox/
getScroll: function(){
if (self.pageYOffset) {
this.yPos = self.pageYOffset;
} else if (document.documentElement && document.documentElement.scrollTop){
- this.yPos = document.documentElement.scrollTop;
+ this.yPos = document.documentElement.scrollTop;
} else if (document.body) {
this.yPos = document.body.scrollTop;
}
},
-
+
setScroll: function(x, y){
- window.scrollTo(x, y);
+ window.scrollTo(x, y);
},
-
+
displayLightbox: function(display){
$('overlay').style.display = display;
$('lightbox').style.display = display;
if(display != 'none') this.loadInfo();
},
-
+
// Write an iFrame instead of using an AJAX call to pull the content
loadInfo: function() {
info = "<div id='lbContent'><center><a href='#' class='lbAction' rel='deactivate'>close (x)</a></center><iframe frameborder=\"0\" width=\"100%\" height=\"100%\" src=\"" + this.content + "\"</iframe></div>";
new Insertion.Before($('lbLoadMessage'), info)
- $('lightbox').className = "done";
- this.actions();
-
+ $('lightbox').className = "done";
+ this.actions();
+
},
-
+
// Search through new links within the lightbox, and attach click event
actions: function(){
lbActions = document.getElementsByClassName('lbAction');
@@ -139,29 +139,29 @@
}
},
-
+
// Example of creating your own functionality once lightbox is initiated
insert: function(e){
link = Event.element(e).parentNode;
Element.remove($('lbContent'));
-
+
var myAjax = new Ajax.Request(
link.href,
{method: 'post', parameters: "", onComplete: this.processInfo.bindAsEventListener(this)}
);
-
+
},
-
+
// Example of creating your own functionality once lightbox is initiated
deactivate: function(){
Element.remove($('lbContent'));
-
+
if (browser == "Internet Explorer"){
this.setScroll(0,this.yPos);
this.prepareIE("auto", "auto");
this.hideSelects("visible");
}
-
+
this.displayLightbox("none");
}
}
@@ -196,4 +196,4 @@
lbcontainer.appendChild(lb);
bod.appendChild(overlay);
bod.appendChild(lbcontainer);
-}
+}
« no previous file with comments | « samples/third_party/codemirror/js/tokenize.js ('k') | samples/third_party/xmljs/COPYING » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698