Merge pull request jdorn#696 from stroebjo/fix/checkbox_with_ids

fix: checkbox/label should have unqiue id/for attribues for clickable label

Author: schmunk42

src/editors/checkbox.js

@@ -27,11 +27,14 @@ export class CheckboxEditor extends AbstractEditor {
 
   build () {
     this.label = this.header = this.theme.getCheckboxLabel(this.getTitle(), this.isRequired())
+    this.label.htmlFor = this.formname
+
     if (this.schema.description) this.description = this.theme.getFormInputDescription(this.schema.description)
     if (this.options.infoText && !this.options.compact) this.infoButton = this.theme.getInfoButton(this.options.infoText)
     if (this.options.compact) this.container.classList.add('compact')
 
     this.input = this.theme.getCheckbox()
+    this.input.id = this.formname
     this.control = this.theme.getFormControl(this.label, this.input, this.description, this.infoButton)
 
     if (this.schema.readOnly || this.schema.readonly) {

src/editors/multiselect.js

@@ -61,9 +61,12 @@ export class MultiSelectEditor extends AbstractEditor {
       this.inputs = {}
       this.controls = {}
       for (i = 0; i < this.option_keys.length; i++) {
+        const id = this.formname + i.toString()
         this.inputs[this.option_keys[i]] = this.theme.getCheckbox()
+        this.inputs[this.option_keys[i]].id = id
         this.select_options[this.option_keys[i]] = this.inputs[this.option_keys[i]]
         const label = this.theme.getCheckboxLabel(this.option_titles[i])
+        label.htmlFor = id
         this.controls[this.option_keys[i]] = this.theme.getFormControl(label, this.inputs[this.option_keys[i]])
       }