Fix details

FlexConfirmMail · Sep 20, 2024 · 2a07201 · 2a07201
1 parent 61b10f5
commit 2a07201
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 13 deletions.
diff --git a/src/web/dialog.html b/src/web/dialog.html
@@ -13,7 +13,7 @@
     <link rel="stylesheet" href="dialog.css">
 </head>
 <body>
-    <div id="myCardContainer" style="margin-left: 20px; margin-right: 20px;">
+    <div id="my-card-container" style="margin-left: 20px; margin-right: 20px;">
         <fluent-card>
             <div class="heading-container">
                 <h3>登録済みドメインのメールアドレス</h3><fluent-button onclick="onCheckAllTrusted()">全てチェック</fluent-button>

diff --git a/src/web/dialog.js b/src/web/dialog.js
@@ -105,19 +105,10 @@ function onCancel() {
 // eslint-disable-next-line @typescript-eslint/no-unused-vars
 function checkboxChanged(target_element) {
   const checkTargetLength = $("fluent-checkbox.check-target").length;
-  let checkedTargetLength = $("fluent-checkbox.check-target.checked").length;
+  const checkedTargetLength = $("fluent-checkbox.check-target.checked").length;
   // If the target is currently checked, the target is unchecked after this function and vice versa.
-  const is_currently_checked = $(target_element).hasClass('checked');
-
-  if (is_currently_checked)
-  {
-    checkedTargetLength -= 1;
-  }
-  else
-  {
-    checkedTargetLength += 1;
-  }
-  const hasUnchecked = checkTargetLength !== checkedTargetLength;
+  const adjustmentValue = $(target_element).hasClass('checked') ? -1 : 1;
+  const hasUnchecked = checkTargetLength !== (checkedTargetLength + adjustmentValue);
   $("#ok-button").prop("disabled", hasUnchecked);
 }