mirror of
https://github.com/silverstripe/silverstripe-framework
synced 2024-10-22 12:05:37 +00:00
FormFields now allow setting of extra CSSClasses en masse
Each CSS class passed in to `addExtraClass` or `removeExtraClass` will be set as their own key in the `extraClasses` array Also make `Form` consistent with `FormField`
This commit is contained in:
parent
a6c3d1e269
commit
336ddf1a55
@ -1473,14 +1473,12 @@ class Form extends RequestHandler {
|
||||
* names delimited by a single space.
|
||||
*/
|
||||
public function addExtraClass($class) {
|
||||
$classes = explode(' ', $class);
|
||||
|
||||
foreach($classes as $class) {
|
||||
$value = trim($class);
|
||||
|
||||
$this->extraClasses[] = $value;
|
||||
//split at white space
|
||||
$classes = preg_split('/\s+/', $class);
|
||||
foreach ($classes as $class) {
|
||||
//add classes one by one
|
||||
$this->extraClasses[$class] = $class;
|
||||
}
|
||||
|
||||
return $this;
|
||||
}
|
||||
|
||||
@ -1491,8 +1489,12 @@ class Form extends RequestHandler {
|
||||
* @param string $class
|
||||
*/
|
||||
public function removeExtraClass($class) {
|
||||
$classes = explode(' ', $class);
|
||||
$this->extraClasses = array_diff($this->extraClasses, $classes);
|
||||
//split at white space
|
||||
$classes = preg_split('/\s+/', $class);
|
||||
foreach ($classes as $class) {
|
||||
//unset one by one
|
||||
unset($this->extraClasses[$class]);
|
||||
}
|
||||
return $this;
|
||||
}
|
||||
|
||||
|
@ -293,24 +293,32 @@ class FormField extends RequestHandler {
|
||||
}
|
||||
|
||||
/**
|
||||
* Add a CSS-class to the formfield-container.
|
||||
* Add one or more CSS-classes to the formfield-container.
|
||||
*
|
||||
* @param $class String
|
||||
*/
|
||||
public function addExtraClass($class) {
|
||||
$this->extraClasses[$class] = $class;
|
||||
//split at white space to extract all the classes
|
||||
$classes = preg_split('/\s+/', $class);
|
||||
foreach ($classes as $class) {
|
||||
//add each class one by one
|
||||
$this->extraClasses[$class] = $class;
|
||||
}
|
||||
return $this;
|
||||
}
|
||||
|
||||
/**
|
||||
* Remove a CSS-class from the formfield-container.
|
||||
* Remove one or more CSS-classes from the formfield-container.
|
||||
*
|
||||
* @param $class String
|
||||
*/
|
||||
public function removeExtraClass($class) {
|
||||
$pos = array_search($class, $this->extraClasses);
|
||||
if($pos !== false) unset($this->extraClasses[$pos]);
|
||||
|
||||
//split at white space to extract all the classes
|
||||
$classes = preg_split('/\s+/', $class);
|
||||
foreach ($classes as $class) {
|
||||
//unset each class one by one
|
||||
unset($this->extraClasses[$class]);
|
||||
}
|
||||
return $this;
|
||||
}
|
||||
|
||||
|
Loading…
x
Reference in New Issue
Block a user