From 490356e6373b3017c05cef1534abdfc89f2222ce Mon Sep 17 00:00:00 2001
From: Thierry Bugier <tbugier@teclib.com>
Date: Tue, 28 Jan 2020 21:08:11 +0100
Subject: [PATCH] fix(question): show / hode specific properties

Signed-off-by: Thierry Bugier <tbugier@teclib.com>
---
 inc/question.class.php | 16 ++++++++++++----
 js/scripts.js.php      | 11 ++---------
 2 files changed, 14 insertions(+), 13 deletions(-)

diff --git a/inc/question.class.php b/inc/question.class.php
index 340b149cb..31c19a588 100644
--- a/inc/question.class.php
+++ b/inc/question.class.php
@@ -841,28 +841,36 @@ public function showForm($ID, $options = []) {
       echo '<tr>';
       // required
       echo '<td>';
-      echo '<label for="dropdown_required'.$rand.'" id="label_required">';
+      echo '<div class="plugin_formcreator_required">';
+      echo '<label for="dropdown_required'.$rand.'">';
       echo __('Required', 'formcreator');
       echo '</label>';
+      echo '</div>';
       echo '</td>';
 
-      echo '<td id="plugin_formcreator_required">';
+      echo '<td>';
+      echo '<div class="plugin_formcreator_required">';
       dropdown::showYesNo('required', $this->fields['required'], -1, [
          'rand'  => $rand,
       ]);
+      echo '</div>';
       echo '</td>';
 
       // show empty
       echo '<td>';
-      echo '<label for="dropdown_show_empty'.$rand.'" id="label_show_empty">';
+      echo '<div class="plugin_formcreator_mayBeEmpty">';
+      echo '<label for="dropdown_show_empty'.$rand.'">';
       echo __('Show empty', 'formcreator');
       echo '</label>';
+      echo '</div>';
       echo '</td>';
 
-      echo '<td id="plugin_formcreator_show_empty">';
+      echo '<td>';
+      echo '<div class="plugin_formcreator_mayBeEmpty">';
       dropdown::showYesNo('show_empty', $this->fields['show_empty'], -1, [
          'rand'  => $rand,
       ]);
+      echo '</div>';
       echo '</td>';
       echo '</tr>';
 
diff --git a/js/scripts.js.php b/js/scripts.js.php
index c57e398b2..42abd710b 100644
--- a/js/scripts.js.php
+++ b/js/scripts.js.php
@@ -1084,15 +1084,8 @@ function plugin_formcreator_changeQuestionType(rand) {
          return;
       }
 
-      $('#label_required').toggle(response.may_be_required);
-      $('#plugin_formcreator_required > [name="required"]').toggle(response.may_be_required);
-
-      $('#label_show_empty').toggle(response.may_be_empty);
-      $('#plugin_formcreator_show_empty > [name="show_empty"]').toggle(response.may_be_empty);
-
-      $('#plugin_formcreator_subtype_label').html(response.label);
-      $('#plugin_formcreator_subtype_value').html(response.field);
-
+      $('.plugin_formcreator_required').toggle(response.may_be_required);
+      $('.plugin_formcreator_mayBeEmpty').toggle(response.may_be_empty);
       plugin_formcreator_updateQuestionSpecific(response.additions);
    });
 }