From 7818e2d4603d352c0a20574051ec72c9727a190d Mon Sep 17 00:00:00 2001 From: Dries Buytaert Date: Wed, 3 Mar 2010 19:46:26 +0000 Subject: - Patch #676800 by casey, sun, james.elliott, cosmicdreams, seutje, Jacine: made fieldsets work on all browser. --- includes/form.inc | 5 +- misc/collapse.js | 97 ++++++++++++++++------------------ modules/comment/comment.admin.inc | 3 +- modules/node/node.admin.inc | 3 +- modules/node/node.module | 2 +- modules/path/path.admin.inc | 10 ++-- modules/search/search.module | 11 ++-- modules/search/search.pages.inc | 2 +- modules/simpletest/simpletest.test | 6 +-- modules/system/system-behavior-rtl.css | 4 +- modules/system/system-behavior.css | 30 +++++------ modules/system/system.admin.inc | 3 +- modules/trigger/trigger.admin.inc | 4 +- modules/user/user.admin.inc | 3 +- modules/user/user.test | 2 +- themes/garland/style-rtl.css | 4 +- themes/garland/style.css | 20 ++----- themes/seven/ie6.css | 1 + themes/seven/reset.css | 7 ++- themes/seven/style.css | 84 ++++++++++++++++------------- themes/seven/template.php | 25 +-------- 21 files changed, 145 insertions(+), 181 deletions(-) diff --git a/includes/form.inc b/includes/form.inc index 0955dfd0f..c30974cc7 100644 --- a/includes/form.inc +++ b/includes/form.inc @@ -1947,8 +1947,10 @@ function theme_fieldset($variables) { $output = ''; if (!empty($element['#title'])) { - $output .= '' . $element['#title'] . ''; + // Always wrap fieldset legends in a SPAN for CSS positioning. + $output .= '' . $element['#title'] . ''; } + $output .= '
'; if (!empty($element['#description'])) { $output .= '
' . $element['#description'] . '
'; } @@ -1956,6 +1958,7 @@ function theme_fieldset($variables) { if (isset($element['#value'])) { $output .= $element['#value']; } + $output .= '
'; $output .= "\n"; return $output; } diff --git a/misc/collapse.js b/misc/collapse.js index a976345dc..ffacce9cb 100644 --- a/misc/collapse.js +++ b/misc/collapse.js @@ -5,40 +5,33 @@ * Toggle the visibility of a fieldset using smooth animations. */ Drupal.toggleFieldset = function (fieldset) { - if ($(fieldset).is('.collapsed')) { - // Action div containers are processed separately because of a IE bug - // that alters the default submit button behavior. - var content = $('> div:not(.action)', fieldset); - $(fieldset) + var $fieldset = $(fieldset); + if ($fieldset.is('.collapsed')) { + var $content = $('> .fieldset-wrapper', fieldset).hide(); + $fieldset .removeClass('collapsed') .trigger({ type: 'collapsed', value: false }) - .find('> legend > a > span.element-invisible') - .empty() - .append(Drupal.t('Hide')); - content.hide(); - content.slideDown({ + .find('> legend span.fieldset-legend-prefix').html(Drupal.t('Hide')); + $content.slideDown({ duration: 'fast', easing: 'linear', complete: function () { - Drupal.collapseScrollIntoView(this.parentNode); - this.parentNode.animating = false; - $('div.action', fieldset).show(); + Drupal.collapseScrollIntoView(fieldset); + fieldset.animating = false; }, step: function () { // Scroll the fieldset into view. - Drupal.collapseScrollIntoView(this.parentNode); + Drupal.collapseScrollIntoView(fieldset); } }); } else { - $('div.action', fieldset).hide(); - $(fieldset).trigger({ type: 'collapsed', value: true }); - var content = $('> div:not(.action)', fieldset).slideUp('fast', function () { - $(this.parentNode).addClass('collapsed') - .find('> legend > a > span.element-invisible') - .empty() - .append(Drupal.t('Show')); - this.parentNode.animating = false; + $fieldset.trigger({ type: 'collapsed', value: true }); + $('> .fieldset-wrapper', fieldset).slideUp('fast', function () { + $fieldset + .addClass('collapsed') + .find('> legend span.fieldset-legend-prefix').html(Drupal.t('Show')); + fieldset.animating = false; }); } }; @@ -63,45 +56,45 @@ Drupal.collapseScrollIntoView = function (node) { Drupal.behaviors.collapse = { attach: function (context, settings) { - $('fieldset.collapsible > legend', context).once('collapse', function () { - var fieldset = $(this.parentNode); + $('fieldset.collapsible', context).once('collapse', function () { + var $fieldset = $(this); // Expand if there are errors inside. - if ($('input.error, textarea.error, select.error', fieldset).size() > 0) { - fieldset.removeClass('collapsed'); + if ($('.error', $fieldset).length) { + $fieldset.removeClass('collapsed'); } var summary = $(''); - fieldset. + $fieldset. bind('summaryUpdated', function () { - var text = $.trim(fieldset.getSummary()); + var text = $.trim($fieldset.getSummary()); summary.html(text ? ' (' + text + ')' : ''); }) .trigger('summaryUpdated'); - // Turn the legend into a clickable link and wrap the contents of the - // fieldset in a div for easier animation. - var text = this.innerHTML; - $(this).empty() - .append($('' + text + '') - .click(function () { - var fieldset = $(this).parents('fieldset:first')[0]; - // Don't animate multiple times. - if (!fieldset.animating) { - fieldset.animating = true; - Drupal.toggleFieldset(fieldset); - } - return false; - }) - .prepend($('') - .append(fieldset.hasClass('collapsed') ? Drupal.t('Show') : Drupal.t('Hide')) - .after(' ') - ) - ) - .append(summary) - .after( - $('
') - .append(fieldset.children(':not(legend):not(.action)')) - ); + // Turn the legend into a clickable link, but retain span.fieldset-legend + // for CSS positioning. + var $legend = $('> legend .fieldset-legend', this); + + $('') + .append($fieldset.hasClass('collapsed') ? Drupal.t('Show') : Drupal.t('Hide')) + .prependTo($legend) + .after(' '); + + // .wrapInner() does not retain bound events. + var $link = $('') + .prepend($legend.contents()) + .appendTo($legend) + .click(function () { + var fieldset = $fieldset.get(0); + // Don't animate multiple times. + if (!fieldset.animating) { + fieldset.animating = true; + Drupal.toggleFieldset(fieldset); + } + return false; + }); + + $legend.append(summary); }); } }; diff --git a/modules/comment/comment.admin.inc b/modules/comment/comment.admin.inc index d68e4e0fb..764a5adbd 100644 --- a/modules/comment/comment.admin.inc +++ b/modules/comment/comment.admin.inc @@ -39,8 +39,7 @@ function comment_admin_overview($form, &$form_state, $arg) { $form['options'] = array( '#type' => 'fieldset', '#title' => t('Update options'), - '#prefix' => '
', - '#suffix' => '
', + '#attributes' => array('class' => array('container-inline')), ); $options = array(); foreach (comment_operations($arg == 'approval' ? 'publish' : 'unpublish') as $key => $value) { diff --git a/modules/node/node.admin.inc b/modules/node/node.admin.inc index b4aa7733c..ad982dedf 100644 --- a/modules/node/node.admin.inc +++ b/modules/node/node.admin.inc @@ -388,8 +388,7 @@ function node_admin_nodes() { $form['options'] = array( '#type' => 'fieldset', '#title' => t('Update options'), - '#prefix' => '
', - '#suffix' => '
', + '#attributes' => array('class' => array('container-inline')), '#access' => $admin_access, ); $options = array(); diff --git a/modules/node/node.module b/modules/node/node.module index 6c3609bab..3e290f77b 100644 --- a/modules/node/node.module +++ b/modules/node/node.module @@ -2611,7 +2611,7 @@ function node_search_validate($form, &$form_state) { $keys .= ' "' . str_replace('"', ' ', $form_state['values']['phrase']) . '"'; } if (!empty($keys)) { - form_set_value($form['basic']['inline']['processed_keys'], trim($keys), $form_state); + form_set_value($form['basic']['processed_keys'], trim($keys), $form_state); } } diff --git a/modules/path/path.admin.inc b/modules/path/path.admin.inc index bb31d1bfd..b400aa344 100644 --- a/modules/path/path.admin.inc +++ b/modules/path/path.admin.inc @@ -231,23 +231,23 @@ function path_admin_delete_confirm_submit($form, &$form_state) { function path_admin_filter_form($form, &$form_state, $keys = '') { $form['#attributes'] = array('class' => array('search-form')); $form['basic'] = array('#type' => 'fieldset', - '#title' => t('Filter aliases') + '#title' => t('Filter aliases'), + '#attributes' => array('class' => array('container-inline')), ); - $form['basic']['inline'] = array('#prefix' => '
', '#suffix' => '
'); - $form['basic']['inline']['filter'] = array( + $form['basic']['filter'] = array( '#type' => 'textfield', '#title' => '', '#default_value' => $keys, '#maxlength' => 128, '#size' => 25, ); - $form['basic']['inline']['submit'] = array( + $form['basic']['submit'] = array( '#type' => 'submit', '#value' => t('Filter'), '#submit' => array('path_admin_filter_form_submit_filter'), ); if ($keys) { - $form['basic']['inline']['reset'] = array( + $form['basic']['reset'] = array( '#type' => 'submit', '#value' => t('Reset'), '#submit' => array('path_admin_filter_form_submit_reset'), diff --git a/modules/search/search.module b/modules/search/search.module index 36a0c1820..c82e85d1a 100644 --- a/modules/search/search.module +++ b/modules/search/search.module @@ -901,19 +901,18 @@ function search_form($form, &$form_state, $action = '', $keys = '', $type = NULL $form['#action'] = $action; $form['#attributes']['class'][] = 'search-form'; $form['module'] = array('#type' => 'value', '#value' => $type); - $form['basic'] = array('#type' => 'item', '#title' => $prompt, '#id' => 'edit-keys'); - $form['basic']['inline'] = array('#prefix' => '
', '#suffix' => '
'); - $form['basic']['inline']['keys'] = array( + $form['basic'] = array('#type' => 'container', '#attributes' => array('class' => array('container-inline'))); + $form['basic']['keys'] = array( '#type' => 'textfield', - '#title' => '', + '#title' => $prompt, '#default_value' => $keys, '#size' => $prompt ? 40 : 20, '#maxlength' => 255, ); // processed_keys is used to coordinate keyword passing between other forms // that hook into the basic search form. - $form['basic']['inline']['processed_keys'] = array('#type' => 'value', '#value' => array()); - $form['basic']['inline']['submit'] = array('#type' => 'submit', '#value' => t('Search')); + $form['basic']['processed_keys'] = array('#type' => 'value', '#value' => array()); + $form['basic']['submit'] = array('#type' => 'submit', '#value' => t('Search')); return $form; } diff --git a/modules/search/search.pages.inc b/modules/search/search.pages.inc index 7c1c95048..355fcdc5d 100644 --- a/modules/search/search.pages.inc +++ b/modules/search/search.pages.inc @@ -122,7 +122,7 @@ function template_preprocess_search_result(&$variables) { * value for the basic search form. */ function search_form_validate($form, &$form_state) { - form_set_value($form['basic']['inline']['processed_keys'], trim($form_state['values']['keys']), $form_state); + form_set_value($form['basic']['processed_keys'], trim($form_state['values']['keys']), $form_state); } /** diff --git a/modules/simpletest/simpletest.test b/modules/simpletest/simpletest.test index b1a1f264d..375dab881 100644 --- a/modules/simpletest/simpletest.test +++ b/modules/simpletest/simpletest.test @@ -220,11 +220,11 @@ class SimpleTestFunctionalTest extends DrupalWebTestCase { if ($this->parse()) { if ($fieldset = $this->getResultFieldSet()) { // Code assumes this is the only test in group. - $results['summary'] = $this->asText($fieldset->div[1]); + $results['summary'] = $this->asText($fieldset->div->div[1]); $results['name'] = $this->asText($fieldset->legend); $results['assertions'] = array(); - $tbody = $fieldset->table->tbody; + $tbody = $fieldset->div->table->tbody; foreach ($tbody->tr as $row) { $assertion = array(); $assertion['message'] = $this->asText($row->td[0]); @@ -243,8 +243,6 @@ class SimpleTestFunctionalTest extends DrupalWebTestCase { /** * Get the fieldset containing the results for group this test is in. - * - * @return fieldset containing the results for group this test is in. */ function getResultFieldSet() { $fieldsets = $this->xpath('//fieldset'); diff --git a/modules/system/system-behavior-rtl.css b/modules/system/system-behavior-rtl.css index 7238a341c..9951071f2 100644 --- a/modules/system/system-behavior-rtl.css +++ b/modules/system/system-behavior-rtl.css @@ -14,12 +14,12 @@ html.js input.throbbing { /** * Collapsing fieldsets */ -html.js fieldset.collapsible legend a { +html.js fieldset.collapsible .fieldset-legend { padding-left: 0; padding-right: 15px; background-position: 98% 75%; } -html.js fieldset.collapsed legend a { +html.js fieldset.collapsed .fieldset-legend { background-image: url(../../misc/menu-collapsed-rtl.png); background-position: 98% 50%; } diff --git a/modules/system/system-behavior.css b/modules/system/system-behavior.css index 7850a56b8..0ff2f875b 100644 --- a/modules/system/system-behavior.css +++ b/modules/system/system-behavior.css @@ -46,35 +46,28 @@ html.js fieldset.collapsed { margin-bottom: 0; height: 1em; } -html.js fieldset.collapsed * { +html.js fieldset.collapsed .fieldset-wrapper { display: none; } -html.js fieldset.collapsed legend, html.js fieldset.collapsed legend a span.element-invisible { +html.js fieldset.collapsible .fieldset-legend { display: block; - overflow: hidden; -} -html.js fieldset.collapsible legend a { - display: inline; padding-left: 15px; /* LTR */ background: url(../../misc/menu-expanded.png) 5px 75% no-repeat; /* LTR */ } +html.js fieldset.collapsed .fieldset-legend { + background-image: url(../../misc/menu-collapsed.png); /* LTR */ + background-position: 5px 50%; /* LTR */ +} html.js fieldset.collapsible legend span.summary { - display: inline; font-size: 0.9em; color: #999; margin-left: 0.5em; } -html.js fieldset.collapsed legend a { - background-image: url(../../misc/menu-collapsed.png); /* LTR */ - background-position: 5px 50%; /* LTR */ -} -/* Note: IE-only fix due to '* html' (breaks Konqueror otherwise). */ -* html.js fieldset.collapsed legend, -* html.js fieldset.collapsed legend *, +/* IE6: Fix due to '* html' (breaks Konqueror otherwise). */ * html.js fieldset.collapsed table * { display: inline; } -/* For Safari 2 to prevent collapsible fieldsets containing tables from dissapearing due to tableheader.js. */ +/* Safari 2: Tables in collapsible fieldsets disappear due to tableheader.js. */ html.js fieldset.collapsible { position: relative; } @@ -267,9 +260,14 @@ div.password-confirm { /** * Inline items (need to override above) */ -.container-inline div, .container-inline label { +.container-inline div, +.container-inline label { display: inline; } +/* Fieldset contents always need to be rendered as block. */ +.container-inline .fieldset-wrapper { + display: block; +} .nowrap { white-space: nowrap; diff --git a/modules/system/system.admin.inc b/modules/system/system.admin.inc index 5e7fa8a2e..41ac3a4a0 100644 --- a/modules/system/system.admin.inc +++ b/modules/system/system.admin.inc @@ -2900,8 +2900,7 @@ function system_actions_manage_form($form, &$form_state, $options = array()) { $form['parent'] = array( '#type' => 'fieldset', '#title' => t('Create an advanced action'), - '#prefix' => '
', - '#suffix' => '
', + '#attributes' => array('class' => array('container-inline')), ); $form['parent']['action'] = array( '#type' => 'select', diff --git a/modules/trigger/trigger.admin.inc b/modules/trigger/trigger.admin.inc index f716c84a4..7568bbb2b 100644 --- a/modules/trigger/trigger.admin.inc +++ b/modules/trigger/trigger.admin.inc @@ -160,8 +160,8 @@ function trigger_assign_form($form, $form_state, $module, $hook, $label) { } $form[$hook]['parent'] = array( - '#prefix' => "
", - '#suffix' => '
', + '#type' => 'container', + '#attributes' => array('class' => array('container-inline')), ); // List possible actions that may be assigned. if (count($options) != 0) { diff --git a/modules/user/user.admin.inc b/modules/user/user.admin.inc index 44575850b..1a6f7ca3d 100644 --- a/modules/user/user.admin.inc +++ b/modules/user/user.admin.inc @@ -168,8 +168,7 @@ function user_admin_account() { $form['options'] = array( '#type' => 'fieldset', '#title' => t('Update options'), - '#prefix' => '
', - '#suffix' => '
', + '#attributes' => array('class' => array('container-inline')), ); $options = array(); foreach (module_invoke_all('user_operations') as $operation => $array) { diff --git a/modules/user/user.test b/modules/user/user.test index 8ee7629b7..70925ad8c 100644 --- a/modules/user/user.test +++ b/modules/user/user.test @@ -143,7 +143,7 @@ class UserRegistrationTestCase extends DrupalWebTestCase { variable_set('user_default_timezone', DRUPAL_USER_TIMEZONE_SELECT); $this->drupalLogout(); $this->drupalGet('user/register'); - $this->assertRaw('
Account information', t('Account settings fieldset was not hidden.')); + $this->assertText(t('Account information'), t('Account settings fieldset was not hidden.')); } } diff --git a/themes/garland/style-rtl.css b/themes/garland/style-rtl.css index 130e15e66..970781a4a 100644 --- a/themes/garland/style-rtl.css +++ b/themes/garland/style-rtl.css @@ -227,13 +227,13 @@ ul.links li, ul.inline li { margin-right: 25px; } -html.js fieldset.collapsible legend a { +html.js fieldset.collapsible .fieldset-legend { padding-left: 0; padding-right: 2em; background: url("images/menu-expanded.gif") no-repeat 100% 50%; } -html.js fieldset.collapsed legend a { +html.js fieldset.collapsed .fieldset-legend { background: url("images/menu-collapsed-rtl.gif") no-repeat 100% 50%; } diff --git a/themes/garland/style.css b/themes/garland/style.css index 4fc79748c..8b105cc59 100644 --- a/themes/garland/style.css +++ b/themes/garland/style.css @@ -892,38 +892,28 @@ fieldset { background-position: 0 0; } -*:first-child+html fieldset > .description, *:first-child+html fieldset .fieldset-wrapper .description { +*:first-child+html fieldset .fieldset-wrapper .fieldset-description { padding-top: 1em; } -fieldset legend { - /* Fix disappearing legend in FFox */ - display: block; -} - -*:first-child+html fieldset legend, *:first-child+html fieldset.collapsed legend { - display: inline; -} - html.js fieldset.collapsed { background: transparent; padding-top: 0; padding-bottom: .6em; } -html.js fieldset.collapsible legend a { +html.js fieldset.collapsible .fieldset-legend { padding-left: 2em; /* LTR */ background: url(images/menu-expanded.gif) no-repeat 0% 50%; /* LTR */ } +html.js fieldset.collapsed .fieldset-legend { + background: url(images/menu-collapsed.gif) no-repeat 0% 50%; /* LTR */ +} html.js fieldset.collapsible legend span.summary { color: #898989; } -html.js fieldset.collapsed legend a { - background: url(images/menu-collapsed.gif) no-repeat 0% 50%; /* LTR */ -} - /** * Vertical tabs. */ diff --git a/themes/seven/ie6.css b/themes/seven/ie6.css index d4df28084..fb4d2e56a 100644 --- a/themes/seven/ie6.css +++ b/themes/seven/ie6.css @@ -4,6 +4,7 @@ ul.menu li, ul.menu li a, ul.links li, ul.links li a, +.action-links, #page { height: 1%; } diff --git a/themes/seven/reset.css b/themes/seven/reset.css index a0a5ea7cd..9ad87c21c 100644 --- a/themes/seven/reset.css +++ b/themes/seven/reset.css @@ -214,7 +214,6 @@ body { * to sneak the clearfix class into Drupal's markup. * From http://www.positioniseverything.net/easyclearing.html */ -div.form-item:after, ul.links:after, div.admin-panel .body:after, .clearfix:after { @@ -232,7 +231,7 @@ ul.inline:after { clear: none; } -div.form-item, +.form-item, ul.links, div.admin-panel .body, .clearfix { @@ -240,14 +239,14 @@ div.admin-panel .body, } /* Hides from IE-mac \*/ -* html div.form-item, +* html .form-item, * html ul.links, * html div.admin-panel .body, * html .clearfix { height: 1%; } -div.form-item, +.form-item, ul.links, div.admin-panel .body, .clearfix { diff --git a/themes/seven/style.css b/themes/seven/style.css index a09f043f4..38839c6bd 100644 --- a/themes/seven/style.css +++ b/themes/seven/style.css @@ -603,88 +603,100 @@ table tr.selected td { } /** - * Forms. + * Fieldsets. + * + * Fieldset legends are displayed like containers in Seven. However, several + * browsers do not support styling of LEGEND elements. To achieve the desired + * styling: + * - All fieldsets use 'position: relative'. + * - All legend labels are wrapped in a single span.fieldset-legend that uses + * 'position: absolute', which means that the LEGEND element itself is not + * rendered by browsers. + * - Due to using 'position: absolute', collapsed fieldsets do not have a + * height; the fieldset requires a 'padding-top' to make the absolute + * positioned .fieldset-legend appear as though it would have a height. + * - Various browsers are positioning the legend differently if there is a + * 'padding-left'/'padding-right' applied on a fieldset and inherit the + * positioning even to absolute positioned elements within; we therefore have + * to apply all padding to the inner .fieldset-wrapper instead. */ -/* Fieldsets & Form items */ fieldset { border: 1px solid #ccc; - padding: 30px 13px 13px 14px; - margin: 0 0 10px; + padding: 2.5em 0 0 0; + position: relative; + margin: 1em 0; } -fieldset legend span, -fieldset legend a { +fieldset .fieldset-legend { + margin-top: 0.5em; + padding-left: 15px; position: absolute; - margin-top: 9px; + text-transform: uppercase; } -fieldset legend a span { - position: relative; - margin-top: 0; +fieldset .fieldset-wrapper { + padding: 0 13px 13px 15px; } fieldset.collapsed { - background: transparent; + background-color: transparent; } -html.js fieldset.collapsed legend, -html.js fieldset.collapsed legend * { - display: block; -} html.js fieldset.collapsed { border-width: 1px; + height: auto; margin-bottom: 10px; - padding: 13px; } fieldset fieldset { - background: #fff; + background-color: #fff; } fieldset fieldset fieldset { - background: #f8f8f8; + background-color: #f8f8f8; } html.js fieldset.collapsible .fieldset-wrapper { overflow: visible; } -div.form-item { +/** + * Form elements. + */ + +.form-item { padding: 9px 0; margin: 0 0 10px; } -.filter-wrapper div.form-item, -div.teaser-checkbox div.form-item, -div.form-item div.form-item, -fieldset div.form-item { +.filter-wrapper .form-item, +div.teaser-checkbox .form-item, +.form-item .form-item { padding: 5px 0; margin: 0; border: 0; } - +.form-type-checkbox { + padding: 0; +} .text-format-wrapper .form-item { padding-bottom: 0; } -div.form-item label { +.form-item label { margin: 0; padding: 0; } -fieldset legend { - text-transform: uppercase; -} - -div.form-item label.option { +.form-item label.option { text-transform: none; } -div.form-item label.option { +.form-item label.option { font-size: 12px; } -div.form-item label.option input { +.form-item label.option input { vertical-align: middle; } @@ -700,7 +712,7 @@ div.form-item label.option input { padding-left: 6px; } -.filter-wrapper div.form-item, +.filter-wrapper .form-item, .filter-wrapper .filter-guidelines, .filter-wrapper .filter-help { font-size: 12px; @@ -709,7 +721,7 @@ div.form-item label.option input { ul.tips, div.description, -div.form-item div.description { +.form-item div.description { margin: 5px 0; line-height: 15px; font-size: 12px; @@ -856,10 +868,6 @@ div.admin-panel h3 { padding-bottom: 9px; } -.container-inline fieldset { - display: block; -} - /* admin/appearance */ #system-themes-page h2 { font-weight: normal; diff --git a/themes/seven/template.php b/themes/seven/template.php index 66ad3feeb..88a67a09d 100644 --- a/themes/seven/template.php +++ b/themes/seven/template.php @@ -5,6 +5,8 @@ * Override or insert variables into the html template. */ function seven_preprocess_html(&$vars) { + // Add conditional CSS for IE8 and below. + drupal_add_css(path_to_theme() . '/ie.css', array('weight' => CSS_THEME, 'browsers' => array('IE' => 'lte IE 8', '!IE' => FALSE), 'preprocess' => FALSE)); // Add conditional CSS for IE6. drupal_add_css(path_to_theme() . '/ie6.css', array('weight' => CSS_THEME, 'browsers' => array('IE' => 'lt IE 7', '!IE' => FALSE), 'preprocess' => FALSE)); } @@ -75,29 +77,6 @@ function seven_tablesort_indicator($variables) { } } -/** - * Override of theme_fieldset(). - * - * Add span to legend tag, so we can style it to be inside the fieldset. - */ -function seven_fieldset($variables) { - $element = $variables['element']; - - $output = ''; - if (!empty($element['#title'])) { - $output .= '' . $element['#title'] . ''; - } - if (!empty($element['#description'])) { - $output .= '
' . $element['#description'] . '
'; - } - $output .= $element['#children']; - if (isset($element['#value'])) { - $output .= $element['#value']; - } - $output .= "
\n"; - return $output; -} - /** * Implements hook_css_alter(). */ -- cgit v1.2.3