From c2aaaeb4ec71141e4caeaaeabd16ee022eb068f1 Mon Sep 17 00:00:00 2001
From: Catalin Constantin Mititiuc <webdevcat@proton.me>
Date: Mon, 16 Jun 2025 22:41:33 -0700
Subject: [PATCH] Extract reveal record logic into a function

---
 src/modules/record_sheet.js | 25 ++++++++++++++-----------
 1 file changed, 14 insertions(+), 11 deletions(-)

diff --git a/src/modules/record_sheet.js b/src/modules/record_sheet.js
index 841a693..8caf395 100644
--- a/src/modules/record_sheet.js
+++ b/src/modules/record_sheet.js
@@ -252,14 +252,7 @@ function clear() {
   //document.querySelector('#defender-record .name').textContent = 'defender';
 }
 
-function select(data) {
-  const record = data && getRecord(data);
-  const isSelected = record?.classList.contains('selected');
-
-  deselect();
-
-  if (isSelected || !data) return;
-
+function reveal(record) {
   const currentSquadView = document.querySelector(`#record-sheet #${record.dataset.allegiance}-record .records-header .squad-number text`);
   const records = document.querySelector(`#record-sheet #${record.dataset.allegiance}-record .records`);
   const target = records.querySelector(`.squad-${record.dataset.squad}`);
@@ -302,10 +295,20 @@ function select(data) {
 
   if (currentSquad !== target)
     showSquad(currentSquad, target, direction);
-  else {
-    record.classList.add('selected');
+  else
     record.scrollIntoView({ behavior: 'smooth' });
-  }
+}
+
+function select(data) {
+  const record = data && getRecord(data);
+  const isSelected = record?.classList.contains('selected');
+
+  deselect();
+
+  if (isSelected || !data) return;
+
+  reveal(record);
+  record.classList.add('selected');
 }
 
 function endMove() {