Duplicate letters now pulse differently

Pedro270707 · Oct 25, 2023 · 8abd315 · 8abd315
1 parent c404e28
commit 8abd315
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 8 deletions.
diff --git a/css/senha.css b/css/senha.css
@@ -54,6 +54,10 @@
  border-color: #ff0000;
 }
 
+.character-slot.duplicate-letter {
+ border-color: #ff9900;
+}
+
 .character-slots {
  display: flex;
  gap: 0.25rem;

diff --git a/js/senha.js b/js/senha.js
@@ -191,15 +191,12 @@ document.addEventListener("keyup", (event) => {
  }
  if (hasDuplicateLetters(charactersInSlotArray)) {
  let characterSlots = getCharacterSlots();
- let seenLetters = '';
  for (const slot of Array.from(characterSlots.children)) {
  if (slot.innerHTML === '&nbsp;') {
  continue;
  }
- if (seenLetters.includes(slot.innerHTML)) {
- pulseSlotError(slot);
- } else {
- seenLetters += slot.innerHTML;
+ if (charactersInSlotArray.split(slot.innerHTML).length - 1 > 1) {
+ pulseSlotError(slot, "duplicate-letter");
  }
  }
  }
@@ -224,12 +221,12 @@ document.addEventListener("keyup", (event) => {
  }
 });
 
-function pulseSlotError(slot) {
+function pulseSlotError(slot, errorClass = "error") {
  slot.style.transition = "none";
- slot.classList.add("error");
+ slot.classList.add(errorClass);
  setTimeout(() => {
  slot.style.transition = "border-color 1s";
- slot.classList.remove("error");
+ slot.classList.remove(errorClass);
  }, 20);
 }