From 444e22d8317de54093238c03f7628a2b61b7c7cd Mon Sep 17 00:00:00 2001 From: Thomas Gelf Date: Wed, 16 Mar 2016 17:20:11 +0100 Subject: [PATCH] ConfigForm: use fieldset instead of HTML header --- application/forms/ConfigForm.php | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/application/forms/ConfigForm.php b/application/forms/ConfigForm.php index 6f1d1a3e..bd361ad2 100644 --- a/application/forms/ConfigForm.php +++ b/application/forms/ConfigForm.php @@ -16,7 +16,6 @@ class ConfigForm extends QuickForm $config = $this->config(); $resources = $this->enumResources(); - $this->addHtml(sprintf('

%s

', $this->translate('Database backend'))); $this->addElement('select', 'resource', array( 'required' => true, 'label' => $this->translate('DB Resource'), @@ -25,6 +24,16 @@ class ConfigForm extends QuickForm 'value' => $config->get('db', 'resource') )); + $this->addDisplayGroup(array('resource'), 'config', array( + 'decorators' => array( + 'FormElements', + array('HtmlTag', array('tag' => 'dl')), + 'Fieldset', + ), + 'order' => 40, + 'legend' => $this->translate('Database backend') + )); + if (empty($resources)) { $this->getElement('resource')->addError( $this->translate('This has to be a MySQL or PostgreSQL database')