summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--includes/form.inc5
-rw-r--r--misc/collapse.js97
-rw-r--r--modules/comment/comment.admin.inc3
-rw-r--r--modules/node/node.admin.inc3
-rw-r--r--modules/node/node.module2
-rw-r--r--modules/path/path.admin.inc10
-rw-r--r--modules/search/search.module11
-rw-r--r--modules/search/search.pages.inc2
-rw-r--r--modules/simpletest/simpletest.test6
-rw-r--r--modules/system/system-behavior-rtl.css4
-rw-r--r--modules/system/system-behavior.css30
-rw-r--r--modules/system/system.admin.inc3
-rw-r--r--modules/trigger/trigger.admin.inc4
-rw-r--r--modules/user/user.admin.inc3
-rw-r--r--modules/user/user.test2
-rw-r--r--themes/garland/style-rtl.css4
-rw-r--r--themes/garland/style.css20
-rw-r--r--themes/seven/ie6.css1
-rw-r--r--themes/seven/reset.css7
-rw-r--r--themes/seven/style.css84
-rw-r--r--themes/seven/template.php25
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 = '<fieldset' . drupal_attributes($element['#attributes']) . '>';
if (!empty($element['#title'])) {
- $output .= '<legend>' . $element['#title'] . '</legend>';
+ // Always wrap fieldset legends in a SPAN for CSS positioning.
+ $output .= '<legend><span class="fieldset-legend">' . $element['#title'] . '</span></legend>';
}
+ $output .= '<div class="fieldset-wrapper">';
if (!empty($element['#description'])) {
$output .= '<div class="fieldset-description">' . $element['#description'] . '</div>';
}
@@ -1956,6 +1958,7 @@ function theme_fieldset($variables) {
if (isset($element['#value'])) {
$output .= $element['#value'];
}
+ $output .= '</div>';
$output .= "</fieldset>\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 = $('<span class="summary"></span>');
- 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($('<a href="#">' + text + '</a>')
- .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($('<span class="element-invisible"></span>')
- .append(fieldset.hasClass('collapsed') ? Drupal.t('Show') : Drupal.t('Hide'))
- .after(' ')
- )
- )
- .append(summary)
- .after(
- $('<div class="fieldset-wrapper"></div>')
- .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);
+
+ $('<span class="fieldset-legend-prefix element-invisible"></span>')
+ .append($fieldset.hasClass('collapsed') ? Drupal.t('Show') : Drupal.t('Hide'))
+ .prependTo($legend)
+ .after(' ');
+
+ // .wrapInner() does not retain bound events.
+ var $link = $('<a class="fieldset-title" href="#"></a>')
+ .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' => '<div class="container-inline">',
- '#suffix' => '</div>',
+ '#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' => '<div class="container-inline">',
- '#suffix' => '</div>',
+ '#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' => '<div class="container-inline">', '#suffix' => '</div>');
- $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' => '<div class="container-inline">', '#suffix' => '</div>');
- $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' => '<div class="container-inline">',
- '#suffix' => '</div>',
+ '#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' => "<div class='container-inline'>",
- '#suffix' => '</div>',
+ '#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' => '<div class="container-inline">',
- '#suffix' => '</div>',
+ '#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('<fieldset id="edit-account"><legend>Account information</legend>', 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));
}
@@ -76,29 +78,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 = '<fieldset' . drupal_attributes($element['#attributes']) . '>';
- if (!empty($element['#title'])) {
- $output .= '<legend><span>' . $element['#title'] . '</span></legend>';
- }
- if (!empty($element['#description'])) {
- $output .= '<div class="fieldset-description">' . $element['#description'] . '</div>';
- }
- $output .= $element['#children'];
- if (isset($element['#value'])) {
- $output .= $element['#value'];
- }
- $output .= "</fieldset>\n";
- return $output;
-}
-
-/**
* Implements hook_css_alter().
*/
function seven_css_alter(&$css) {