Rails Javascriptを使ってフォーム項目の表示／非常時

Question

RailsでJavascriptを使ってフォーム項目の表示／非表示の切り替えを行いたいです。

ラジオボタンでvalueがtrueの時は表示、falseの時は非常時としたいです。
一度は正常に切り替えることが出来たのですがCSSのラジオボタンを装飾するためにコードを変更したら動かなくなってしまいました。
HTMLのclass, valueの指定はあってるように見えるのですが原因を教えていただければと思います。

■装飾したラジオボタン

■new.html.erb

<div class="form-group">
  <label for="constraction" class="font-weight-bold">工事の希望</label><br>
  <input type="radio" value="true" name="work" id="work_true" class="constraction">
  <label class="radio" for="work_true"><%= f.radio_button :work, "true" %>希望</label>
  <input type="radio" value="false" name="work" id="work_false" class="constraction">
  <label class="radio" for="work_false"><%= f.radio_button :work, "false" %>無し</label>
</div>

■出力されたHTML

<div class="form-group">
  <label for="constraction" class="font-weight-bold">工事の希望</label>
    <br>
    <input type="radio" value="true" name="work" id="work_true" class="constraction">
    <label class="radio" for="work_true"><input type="radio" value="true" name="work" id="work_true">希望</label>
    ::before
    <input type="radio" value="true" name="work" id="work_true">
    希望
    ::after
    </label>
    <input type="radio" value="false" name="work" id="work_false" class="constraction">
    <label class="radio" for="work_false"><input type="radio" value="false" name="work" id="work_false">無し</label>
    ::before
    <input type="radio" value="false" name="work" id="work_false">
    無し
    ::after
  </label>
</div>  

<div id="work_on">...</div>

■Javascript

$(function () {
  // ラジオボタンを選択変更したら実行
  $(".constraction input").change(function () {
    let val = $(this).val();

    if (val == "true"){
      $("#work_on").show();
    } else {
      $("#work_on").hide();
    };
  });
});

■CSS（ラジオボタン装飾部分）

input[type=radio] {
  display: none;
}

.radio {
  box-sizing: border-box;
  -webkit-transition: background-color 0.2s linear;
  transition: background-color 0.2s linear;
  position: relative;
  display: inline-block;
  margin: 0 15px 5px 0;
  padding: 8px 14px 8px 38px;
  border-radius: 8px;
  background-color: #f6f7f8;
  vertical-align: middle;
  cursor: pointer;
}
.radio:hover {
  background-color: #e2edd7;
}
.radio:hover:after {
  border-color: #e2edd7;
}
.radio:after {
  -webkit-transition: border-color 0.2s linear;
  transition: border-color 0.2s linear;
  position: absolute;
  top: 50%;
  left: 15px;
  display: block;
  margin-top: -10px;
  width: 16px;
  height: 16px;
  border: 2px solid #bbb;
  border-radius: 6px;
  content: '';
}

.radio:before {
  -webkit-transition: opacity 0.2s linear;
  transition: opacity 0.2s linear;
  position: absolute;
  top: 45%;
  left: 18px;
  display: block;
  margin-top: -5px;
  width: 10px;
  height: 10px;
  border-radius: 50%;
  background-color: #53b300;
  content: '';
  opacity: 0;
}

input[type=radio]:checked + .radio:before {
  opacity: 1;
}

input[type="radio"]:checked + label {
background: #e2edd7;
color: #000000;
}

Answer 1

スクリプトの問題以前に、根本的に間違っている箇所があります。

<div class="form-group">
  <label for="constraction" class="font-weight-bold">工事の希望</label><br>
  <%= f.radio_button :work, "true" %>
  <label class="radio" for="work_true">希望</label>
  <%= f.radio_button :work, "false" %>
  <label class="radio" for="work_false">無し</label>
</div>

JavaScriptはセレクタの指定が間違っていますね。

// $(".constraction input")
//         ↓
   $("input.constraction")

Comments

[Komama]

いつもありがとうございます！
無事動作しました！

[kei344]

あ、セレクタは
$('input[name="work"].constraction')
のほうがよいかも。