From e0adb9306466fc738004e071e3f9b063441814e2 Mon Sep 17 00:00:00 2001
From: Dries Buytaert <dries@buytaert.net>
Date: Fri, 11 Jun 2010 14:07:32 +0000
Subject: - Patch #668640 David_Rothstein: follow-up on overlay.

---
 modules/overlay/overlay-child.css |  3 ++-
 modules/overlay/overlay-parent.js | 16 +++++++---------
 modules/overlay/overlay.tpl.php   |  2 +-
 3 files changed, 10 insertions(+), 11 deletions(-)

(limited to 'modules')

diff --git a/modules/overlay/overlay-child.css b/modules/overlay/overlay-child.css
index c8924b10f..ce6945e1c 100644
--- a/modules/overlay/overlay-child.css
+++ b/modules/overlay/overlay-child.css
@@ -1,3 +1,4 @@
+/* $Id$ */
 
 html.js {
   background: transparent !important;
@@ -15,7 +16,7 @@ html.js body {
   position: relative;
   padding: .2em;
   padding-right: 26px;
-  width: 78%;
+  width: 88%;
 }
 #overlay-titlebar {
   padding: 0 20px;
diff --git a/modules/overlay/overlay-parent.js b/modules/overlay/overlay-parent.js
index e39978993..33cc598fc 100644
--- a/modules/overlay/overlay-parent.js
+++ b/modules/overlay/overlay-parent.js
@@ -386,7 +386,7 @@ Drupal.overlay.eventhandlerAlterDisplacedElements = function (event) {
       (data.drupalOverlay = data.drupalOverlay || {}).maxWidth = true;
     }
 
-    // Use a more rigerous approach if the displaced element still overlaps
+    // Use a more rigorous approach if the displaced element still overlaps
     // window's scrollbar; clip the element on the right.
     var offset = $(this).offset();
     var offsetRight = offset.left + $(this).outerWidth();
@@ -398,7 +398,7 @@ Drupal.overlay.eventhandlerAlterDisplacedElements = function (event) {
 };
 
 /**
- * Event handler: restores size of displaced elements as they where before
+ * Event handler: restores size of displaced elements as they were before
  * overlay was opened.
  *
  * @param event
@@ -419,8 +419,6 @@ Drupal.overlay.eventhandlerRestoreDisplacedElements = function (event) {
   }
 };
 
-Drupal.overlay.linksToRestore = [];
-
 /**
  * Event handler: overrides href of administrative links to be opened in
  * the overlay.
@@ -609,7 +607,7 @@ Drupal.overlay.eventhandlerRestrictKeyboardNavigation = function (event) {
   }
 
   if (event.keyCode && event.keyCode == $.ui.keyCode.TAB) {
-    // Whenever the focus is not inside the overlay (or an displaced element)
+    // Whenever the focus is not inside the overlay (or a displaced element)
     // move the focus along until it is.
     var direction = event.shiftKey ? -1 : 1;
     var current = this.$tabbables.index(event.target);
@@ -639,13 +637,13 @@ Drupal.overlay.eventhandlerDispatchEvent = function (event) {
 };
 
 /**
- * Make a regular admin link into an URL that will trigger the overlay to open.
+ * Make a regular admin link into a URL that will trigger the overlay to open.
  *
  * @param link
  *   A Javascript Link object (i.e. an <a> element).
  *
  * @return
- *   An URL that will trigger the overlay (in the form
+ *   A URL that will trigger the overlay (in the form
  *   /node/1#overlay=admin/config).
  */
 Drupal.overlay.fragmentizeLink = function (link) {
@@ -781,14 +779,14 @@ Drupal.overlay.getDisplacement = function (region) {
  * Theme function to create the overlay iframe element.
  */
 Drupal.theme.prototype.overlayContainer = function () {
-  return '<div id="overlay-container" role="dialog"><div class="overlay-modal-background"/></div>';
+  return '<div id="overlay-container" role="dialog"><div class="overlay-modal-background"></div></div>';
 };
 
 /**
  * Theme function to create an overlay iframe element.
  */
 Drupal.theme.prototype.overlayElement = function (url) {
-  return '<iframe class="overlay-element" frameborder="0" scrolling="auto" allowtransparency="true" role="document"/>';
+  return '<iframe class="overlay-element" frameborder="0" scrolling="auto" allowtransparency="true" role="document"></iframe>';
 };
 
 })(jQuery);
diff --git a/modules/overlay/overlay.tpl.php b/modules/overlay/overlay.tpl.php
index 0c25b0eb8..851285128 100644
--- a/modules/overlay/overlay.tpl.php
+++ b/modules/overlay/overlay.tpl.php
@@ -6,7 +6,7 @@
  * Default theme implementation to display a page in the overlay.
  *
  * Available variables:
- * - $title: the (sanitized) title of the node.
+ * - $title: the (sanitized) title of the page.
  * - $page: The rendered page content.
  * - $tabs (array): Tabs linking to any sub-pages beneath the current page
  *   (e.g., the view and edit tabs when displaying a node).
-- 
cgit v1.2.3