Index: includes/menu.inc
===================================================================
RCS file: /cvs/drupal/drupal/includes/menu.inc,v
retrieving revision 1.335
diff -u -p -r1.335 menu.inc
--- includes/menu.inc 19 Aug 2009 23:29:13 -0000 1.335
+++ includes/menu.inc 22 Aug 2009 17:56:11 -0000
@@ -113,6 +113,11 @@ define('MENU_CREATED_BY_ADMIN', 0x0040);
define('MENU_IS_LOCAL_TASK', 0x0080);
/**
+ * Internal menu flag -- menu item is a local action.
+ */
+define('MENU_IS_LOCAL_ACTION', 0x0100);
+
+/**
* @} End of "Menu flags".
*/
@@ -168,6 +173,14 @@ define('MENU_LOCAL_TASK', MENU_IS_LOCAL_
define('MENU_DEFAULT_LOCAL_TASK', MENU_IS_LOCAL_TASK | MENU_LINKS_TO_PARENT);
/**
+ * Menu type -- An action specific to the parent, usually rendered as a link.
+ *
+ * Local actions are menu items that describe actions on the parent item such
+ * as adding a new user, taxonomy term, etc.
+ */
+define('MENU_LOCAL_ACTION', MENU_IS_LOCAL_TASK | MENU_IS_LOCAL_ACTION);
+
+/**
* @} End of "Menu item types".
*/
@@ -1306,6 +1319,15 @@ function theme_menu_local_task($link, $a
}
/**
+ * Generate the HTML output for a single local action link.
+ *
+ * @ingroup themeable
+ */
+function theme_menu_local_action($link, $active = FALSE) {
+ return '
' . $link . "\n";
+}
+
+/**
* Generates elements for the $arg array in the help hook.
*/
function drupal_help_arg($arg = array()) {
@@ -1437,27 +1459,39 @@ function menu_navigation_links($menu_nam
}
/**
- * Collects the local tasks (tabs) for a given level.
+ * Collects the local tasks (tabs), action links, and the root path.
*
* @param $level
* The level of tasks you ask for. Primary tasks are 0, secondary are 1.
- * @param $return_root
- * Whether to return the root path for the current page.
* @return
- * Themed output corresponding to the tabs of the requested level, or
- * router path if $return_root == TRUE. This router path corresponds to
- * a parent tab, if the current page is a default local task.
- */
-function menu_local_tasks($level = 0, $return_root = FALSE) {
- $tabs = &drupal_static(__FUNCTION__);
- $root_path = &drupal_static(__FUNCTION__ . ':root_path');
+ * An array containing
+ * - tabs: Local tasks for the requested level:
+ * - count: The number of local tasks.
+ * - output: The themed output of local tasks.
+ * - actions: Action links for the requested level:
+ * - count: The number of action links.
+ * - output: The themed output of action links.
+ * - root_path: The router path for the current page. If the current page is
+ * a default local task, then this corresponds to the parent tab.
+ */
+function menu_local_tasks($level = 0) {
+ $data = &drupal_static(__FUNCTION__);
+ $root_path = &drupal_static(__FUNCTION__ . ':root_path', '');
+ $empty = array(
+ 'tabs' => array('count' => 0, 'output' => ''),
+ 'actions' => array('count' => 0, 'output' => ''),
+ 'root_path' => &$root_path,
+ );
- if (!isset($tabs)) {
+ if (!isset($data)) {
+ $data = array();
+ // Set defaults in case there are no actions or tabs.
+ $actions = $empty['actions'];
$tabs = array();
$router_item = menu_get_item();
if (!$router_item || !$router_item['access']) {
- return '';
+ return $empty;
}
// Get all tabs and the root page.
$result = db_select('menu_router', NULL, array('fetch' => PDO::FETCH_ASSOC))
@@ -1480,7 +1514,6 @@ function menu_local_tasks($level = 0, $r
// Store the translated item for later use.
$tasks[$item['path']] = $item;
}
-
// Find all tabs below the current path.
$path = $router_item['path'];
// Tab parenting may skip levels, so the number of parts in the path may not
@@ -1488,31 +1521,42 @@ function menu_local_tasks($level = 0, $r
$depth = 1001;
while (isset($children[$path])) {
$tabs_current = '';
+ $actions_current = '';
$next_path = '';
- $count = 0;
+ $tab_count = 0;
+ $action_count = 0;
foreach ($children[$path] as $item) {
if ($item['access']) {
- $count++;
// The default task is always active.
if ($item['type'] == MENU_DEFAULT_LOCAL_TASK) {
- // Find the first parent which is not a default local task.
+ // Find the first parent which is not a default local task or action.
for ($p = $item['tab_parent']; $tasks[$p]['type'] == MENU_DEFAULT_LOCAL_TASK; $p = $tasks[$p]['tab_parent']);
$link = theme('menu_item_link', array('href' => $tasks[$p]['href']) + $item);
$tabs_current .= theme('menu_local_task', $link, TRUE);
$next_path = $item['path'];
+ $tab_count++;
}
else {
$link = theme('menu_item_link', $item);
- $tabs_current .= theme('menu_local_task', $link);
+ if ($item['type'] == MENU_LOCAL_TASK) {
+ $tabs_current .= theme('menu_local_task', $link);
+ $tab_count++;
+ }
+ else {
+ $actions_current .= theme('menu_local_action', $link);
+ $action_count++;
+ }
}
}
}
$path = $next_path;
- $tabs[$depth]['count'] = $count;
+ $tabs[$depth]['count'] = $tab_count;
$tabs[$depth]['output'] = $tabs_current;
+ $actions['count'] = $action_count;
+ $actions['output'] = $actions_current;
$depth++;
}
-
+ $data['actions'] = $actions;
// Find all tabs at the same level or above the current one.
$parent = $router_item['tab_parent'];
$path = $router_item['path'];
@@ -1524,6 +1568,9 @@ function menu_local_tasks($level = 0, $r
$next_parent = '';
$count = 0;
foreach ($children[$parent] as $item) {
+ if ($item['type'] == MENU_LOCAL_ACTION) {
+ continue;
+ }
if ($item['access']) {
$count++;
if ($item['type'] == MENU_DEFAULT_LOCAL_TASK) {
@@ -1560,36 +1607,51 @@ function menu_local_tasks($level = 0, $r
ksort($tabs);
// Remove the depth, we are interested only in their relative placement.
$tabs = array_values($tabs);
+ $data['tabs'] = $tabs;
}
- if ($return_root) {
- return $root_path;
- }
- else {
- // We do not display single tabs.
- return (isset($tabs[$level]) && $tabs[$level]['count'] > 1) ? $tabs[$level]['output'] : '';
+ if (isset($data['tabs'][$level])) {
+ return array(
+ 'tabs' => $data['tabs'][$level],
+ 'actions' => $data['actions'],
+ 'root_path' => $root_path,
+ );
}
+ return $empty;
}
/**
* Returns the rendered local tasks at the top level.
*/
function menu_primary_local_tasks() {
- return menu_local_tasks(0);
+ $links = menu_local_tasks(0);
+ // Do not display single tabs.
+ return ($links['tabs']['count'] > 1 ? $links['tabs']['output'] : '');
}
/**
* Returns the rendered local tasks at the second level.
*/
function menu_secondary_local_tasks() {
- return menu_local_tasks(1);
+ $links = menu_local_tasks(1);
+ // Do not display single tabs.
+ return ($links['tabs']['count'] > 1 ? $links['tabs']['output'] : '');
+}
+
+/**
+ * Returns the rendered local actions at the current level.
+ */
+function menu_local_actions() {
+ $links = menu_local_tasks();
+ return $links['actions']['output'];
}
/**
* Returns the router path, or the path of the parent tab of a default local task.
*/
function menu_tab_root_path() {
- return menu_local_tasks(0, TRUE);
+ $links = menu_local_tasks();
+ return $links['root_path'];
}
/**
Index: includes/theme.inc
===================================================================
RCS file: /cvs/drupal/drupal/includes/theme.inc,v
retrieving revision 1.509
diff -u -p -r1.509 theme.inc
--- includes/theme.inc 22 Aug 2009 15:55:37 -0000 1.509
+++ includes/theme.inc 22 Aug 2009 17:47:49 -0000
@@ -2039,6 +2039,7 @@ function template_preprocess_page(&$vari
$variables['messages'] = $variables['show_messages'] ? theme('status_messages') : '';
$variables['main_menu'] = theme_get_setting('toggle_main_menu') ? menu_main_menu() : array();
$variables['secondary_menu'] = theme_get_setting('toggle_secondary_menu') ? menu_secondary_menu() : array();
+ $variables['action_links'] = menu_local_actions();
$variables['search_box'] = (theme_get_setting('toggle_search') ? drupal_render(drupal_get_form('search_theme_form')) : '');
$variables['site_name'] = (theme_get_setting('toggle_name') ? filter_xss_admin(variable_get('site_name', 'Drupal')) : '');
$variables['site_slogan'] = (theme_get_setting('toggle_slogan') ? filter_xss_admin(variable_get('site_slogan', '')) : '');
Index: modules/menu/menu.module
===================================================================
RCS file: /cvs/drupal/drupal/modules/menu/menu.module,v
retrieving revision 1.196
diff -u -p -r1.196 menu.module
--- modules/menu/menu.module 22 Aug 2009 14:34:20 -0000 1.196
+++ modules/menu/menu.module 22 Aug 2009 17:35:58 -0000
@@ -65,7 +65,7 @@ function menu_menu() {
'page callback' => 'drupal_get_form',
'page arguments' => array('menu_edit_menu', 'add'),
'access arguments' => array('administer menu'),
- 'type' => MENU_LOCAL_TASK,
+ 'type' => MENU_LOCAL_ACTION,
);
$items['admin/structure/menu/settings'] = array(
'title' => 'Settings',
@@ -94,7 +94,7 @@ function menu_menu() {
'page callback' => 'drupal_get_form',
'page arguments' => array('menu_edit_item', 'add', NULL, 3),
'access arguments' => array('administer menu'),
- 'type' => MENU_LOCAL_TASK,
+ 'type' => MENU_LOCAL_ACTION,
);
$items['admin/structure/menu-customize/%menu/edit'] = array(
'title' => 'Edit menu',
Index: modules/system/page.tpl.php
===================================================================
RCS file: /cvs/drupal/drupal/modules/system/page.tpl.php,v
retrieving revision 1.29
diff -u -p -r1.29 page.tpl.php
--- modules/system/page.tpl.php 22 Aug 2009 14:34:22 -0000 1.29
+++ modules/system/page.tpl.php 22 Aug 2009 17:48:56 -0000
@@ -68,6 +68,8 @@
* - $secondary_menu (array): An array containing the Secondary menu links for
* the site, if they have been configured.
* - $breadcrumb: The breadcrumb trail for the current page.
+ * - $action_links: Actions local to the page, such as 'Add menu' on the menu
+ * administration interface.
*
* Page content (in order of occurrence in the default page.tpl.php):
* - $title: The page title, for use in the actual HTML content.
@@ -171,6 +173,7 @@
+
Index: themes/garland/page.tpl.php
===================================================================
RCS file: /cvs/drupal/drupal/themes/garland/page.tpl.php,v
retrieving revision 1.31
diff -u -p -r1.31 page.tpl.php
--- themes/garland/page.tpl.php 3 Aug 2009 03:04:34 -0000 1.31
+++ themes/garland/page.tpl.php 22 Aug 2009 17:49:24 -0000
@@ -54,6 +54,7 @@
+
Index: themes/seven/page.tpl.php
===================================================================
RCS file: /cvs/drupal/drupal/themes/seven/page.tpl.php,v
retrieving revision 1.2
diff -u -p -r1.2 page.tpl.php
--- themes/seven/page.tpl.php 20 Aug 2009 15:59:58 -0000 1.2
+++ themes/seven/page.tpl.php 22 Aug 2009 17:49:41 -0000
@@ -33,6 +33,7 @@
+
Index: themes/seven/style.css
===================================================================
RCS file: /cvs/drupal/drupal/themes/seven/style.css,v
retrieving revision 1.12
diff -u -p -r1.12 style.css
--- themes/seven/style.css 21 Aug 2009 07:59:47 -0000 1.12
+++ themes/seven/style.css 22 Aug 2009 17:50:58 -0000
@@ -541,7 +541,8 @@ div.form-item div.description {
}
/* Buttons */
-input.form-submit {
+input.form-submit,
+ul.action-links a {
cursor: pointer;
padding: 4px 17px;
color: #5a5a5a;
@@ -568,7 +569,8 @@ div.node-form input#edit-submit-1 {
color: #133B54;
}
-input.form-submit:active {
+input.form-submit:active,
+ul.action-links a:active {
background: #666;
color: #fff;
border-color: #555;
@@ -608,6 +610,14 @@ html.js input.throbbing {
background-position: 100% -16px;
}
+ul.action-links {
+ padding: 15px 0;
+}
+
+ul.action-links a:hover {
+ text-decoration: none;
+}
+
/* Exceptions */
#diff-inline-form select,
#block-system-main div.filter-options select {