summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJennifer Hodgdon2012-10-17 16:43:42 (GMT)
committer Jennifer Hodgdon2012-10-17 16:43:42 (GMT)
commit100823ea1fc20495a01bcd78cd4d2d14d7136b3e (patch)
treeb668cb9a62feb7f27ad43857b5ce68f2e90ee00b
parent469589eea16e302b172809e868373d2ab5a57f1a (diff)
Issue #1398404 by dcam, batigolix, NROTC_Webmaster, aLearner: Clean up API docs for overlay module
-rw-r--r--modules/overlay/overlay-child-rtl.css5
-rw-r--r--modules/overlay/overlay-child.css5
-rw-r--r--modules/overlay/overlay-child.js4
-rw-r--r--modules/overlay/overlay-parent.css5
-rw-r--r--modules/overlay/overlay-parent.js5
-rw-r--r--modules/overlay/overlay.install2
-rw-r--r--modules/overlay/overlay.module53
7 files changed, 60 insertions, 19 deletions
diff --git a/modules/overlay/overlay-child-rtl.css b/modules/overlay/overlay-child-rtl.css
index 7fd39f4..2e5a267 100644
--- a/modules/overlay/overlay-child-rtl.css
+++ b/modules/overlay/overlay-child-rtl.css
@@ -1,4 +1,9 @@
+/**
+ * @file
+ * RTL styling for Overlay child pages.
+ */
+
html {
direction: rtl;
}
diff --git a/modules/overlay/overlay-child.css b/modules/overlay/overlay-child.css
index a2b23b4..959ebdd 100644
--- a/modules/overlay/overlay-child.css
+++ b/modules/overlay/overlay-child.css
@@ -1,4 +1,9 @@
+/**
+ * @file
+ * Basic styling for the Overlay child pages.
+ */
+
html.js {
background: transparent !important;
overflow-y: scroll;
diff --git a/modules/overlay/overlay-child.js b/modules/overlay/overlay-child.js
index e78e383..4ff6cf0 100644
--- a/modules/overlay/overlay-child.js
+++ b/modules/overlay/overlay-child.js
@@ -1,3 +1,7 @@
+/**
+ * @file
+ * Attaches the behaviors for the Overlay child pages.
+ */
(function ($) {
diff --git a/modules/overlay/overlay-parent.css b/modules/overlay/overlay-parent.css
index dad6d55..9459a7a 100644
--- a/modules/overlay/overlay-parent.css
+++ b/modules/overlay/overlay-parent.css
@@ -1,4 +1,9 @@
+/**
+ * @file
+ * Basic styling for the Overlay module.
+ */
+
html.overlay-open,
html.overlay-open body {
height: 100%;
diff --git a/modules/overlay/overlay-parent.js b/modules/overlay/overlay-parent.js
index 76e1cf2..521bd76 100644
--- a/modules/overlay/overlay-parent.js
+++ b/modules/overlay/overlay-parent.js
@@ -1,3 +1,8 @@
+/**
+ * @file
+ * Attaches the behaviors for the Overlay parent pages.
+ */
+
(function ($) {
/**
diff --git a/modules/overlay/overlay.install b/modules/overlay/overlay.install
index 8939e2a..2df860b 100644
--- a/modules/overlay/overlay.install
+++ b/modules/overlay/overlay.install
@@ -2,7 +2,7 @@
/**
* @file
- * Install, update and uninstall functions for the overlay module.
+ * Install, update, and uninstall functions for the Overlay module.
*/
/**
diff --git a/modules/overlay/overlay.module b/modules/overlay/overlay.module
index 6acc260..c07cc6c 100644
--- a/modules/overlay/overlay.module
+++ b/modules/overlay/overlay.module
@@ -302,7 +302,10 @@ function overlay_page_alter(&$page) {
}
/**
- * Menu callback; dismisses the overlay accessibility message for this user.
+ * Page callback: Dismisses the overlay accessibility message for this user.
+ *
+ * @return
+ * A render array for a page containing a list of content.
*/
function overlay_user_dismiss_message() {
global $user;
@@ -327,10 +330,12 @@ function overlay_user_dismiss_message() {
* If the current user can access the overlay and has not previously indicated
* that this message should be dismissed, this function returns a message
* containing a link to disable the overlay. Nothing is returned for anonymous
- * users, because the links control per-user settings. Therefore, because some
- * screen readers are unable to properly read overlay contents, site builders
- * are discouraged from granting the "access overlay" permission to the
- * anonymous role. See http://drupal.org/node/890284.
+ * users, because the links control per-user settings. Because some screen
+ * readers are unable to properly read overlay contents, site builders are
+ * discouraged from granting the "access overlay" permission to the anonymous
+ * role.
+ *
+ * @see http://drupal.org/node/890284
*/
function overlay_disable_message() {
global $user;
@@ -385,7 +390,13 @@ function overlay_disable_message() {
/**
* Returns the HTML for the message about how to disable the overlay.
*
- * @see overlay_disable_message()
+ * @param $variables
+ * An associative array with an 'element' element, which itself is an
+ * associative array containing:
+ * - profile_link: The link to this user's account.
+ * - dismiss_message_link: The link to dismiss the overlay.
+ *
+ * @ingroup themeable
*/
function theme_overlay_disable_message($variables) {
$element = $variables['element'];
@@ -473,8 +484,12 @@ function overlay_preprocess_maintenance_page(&$variables) {
}
/**
- * Preprocesses template variables for overlay.tpl.php
+ * Implements template_preprocess_HOOK() for overlay.tpl.php
*
+ * If the current page request is inside the overlay, add appropriate classes
+ * to the <body> element, and simplify the page title.
+ *
+ * @see template_process_overlay()
* @see overlay.tpl.php
*/
function template_preprocess_overlay(&$variables) {
@@ -485,20 +500,21 @@ function template_preprocess_overlay(&$variables) {
}
/**
- * Processes variables for overlay.tpl.php
+ * Implements template_process_HOOK() for overlay.tpl.php
+ *
+ * Places the rendered HTML for the page body into a top level variable.
*
* @see template_preprocess_overlay()
* @see overlay.tpl.php
*/
function template_process_overlay(&$variables) {
- // Place the rendered HTML for the page body into a top level variable.
$variables['page'] = $variables['page']['#children'];
}
/**
* Implements hook_preprocess_page().
*
- * Hide tabs inside the overlay.
+ * If the current page request is inside the overlay, hide the tabs.
*
* @see overlay_get_mode()
*/
@@ -509,7 +525,7 @@ function overlay_preprocess_page(&$variables) {
}
/**
- * Callback to request that the overlay display an empty page.
+ * Stores and returns whether an empty page override is needed.
*
* This is used to prevent a page request which closes the overlay (for
* example, a form submission) from being fully re-rendered before the overlay
@@ -550,7 +566,7 @@ function overlay_page_delivery_callback_alter(&$callback) {
}
/**
- * Delivery callback to display an empty page.
+ * Prints an empty page.
*
* This function is used to print out a bare minimum empty page which still has
* the scripts and styles necessary in order to trigger the overlay to close.
@@ -562,7 +578,7 @@ function overlay_deliver_empty_page() {
}
/**
- * Get the current overlay mode.
+ * Gets the current overlay mode.
*
* @see overlay_set_mode()
*/
@@ -688,12 +704,13 @@ function overlay_overlay_child_initialize() {
}
/**
- * Callback to request that the overlay close as soon as the page is displayed.
+ * Requests that the overlay overlay closes when the page is displayed.
*
* @param $redirect
* (optional) The path that should open in the parent window after the
* overlay closes. If not set, no redirect will be performed on the parent
* window.
+ *
* @param $redirect_options
* (optional) An associative array of options to use when generating the
* redirect URL.
@@ -757,7 +774,7 @@ function overlay_supplemental_regions() {
}
/**
- * Helper function for returning a list of page regions related to the overlay.
+ * Returns a list of page regions related to the overlay.
*
* @param $type
* The type of regions to return. This can either be 'overlay_regions' or
@@ -923,7 +940,7 @@ function overlay_store_rendered_content($id = NULL, $content = NULL) {
}
/**
- * Request that the parent window refresh a particular page region.
+ * Requests that the parent window refreshes a particular page region.
*
* @param $region
* The name of the page region to refresh. The parent window will trigger a
@@ -938,7 +955,7 @@ function overlay_request_refresh($region) {
}
/**
- * Request that the entire parent window be reloaded when the overlay closes.
+ * Requests that the entire parent window is reloaded when the overlay closes.
*
* @see overlay_trigger_refresh()
*/
@@ -947,7 +964,7 @@ function overlay_request_page_refresh() {
}
/**
- * Check if the parent window needs to be refreshed on this page load.
+ * Checks if the parent window needs to be refreshed on this page load.
*
* If the previous page load requested that any page regions be refreshed, or
* if it requested that the entire page be refreshed when the overlay closes,