Error reductions found via analytics (#3351)

# Description of Changes

Please provide a summary of the changes, including:

- What was changed
- Why the change was made
- Any challenges encountered

Closes #(issue_number)

---

## Checklist

### General

- [ ] I have read the [Contribution
Guidelines](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/CONTRIBUTING.md)
- [ ] I have read the [Stirling-PDF Developer
Guide](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/DeveloperGuide.md)
(if applicable)
- [ ] I have read the [How to add new languages to
Stirling-PDF](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/HowToAddNewLanguage.md)
(if applicable)
- [ ] I have performed a self-review of my own code
- [ ] My changes generate no new warnings

### Documentation

- [ ] I have updated relevant docs on [Stirling-PDF's doc
repo](https://github.com/Stirling-Tools/Stirling-Tools.github.io/blob/main/docs/)
(if functionality has heavily changed)
- [ ] I have read the section [Add New Translation
Tags](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/HowToAddNewLanguage.md#add-new-translation-tags)
(for new translation tags only)

### UI Changes (if applicable)

- [ ] Screenshots or videos demonstrating the UI changes are attached
(e.g., as comments or direct attachments in the PR)

### Testing (if applicable)

- [ ] I have tested my changes locally. Refer to the [Testing
Guide](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/DeveloperGuide.md#6-testing)
for more details.

src/main/resources/static/js/downloader.js

@@ -132,7 +132,9 @@
         }
       } catch (error) {
         clearTimeout(timeoutId);
-        showGameBtn.style.display = 'none';
+        if(showGameBtn){
+          showGameBtn.style.display = 'none';
+        }
         submitButton.textContent = originalButtonText;
         submitButton.disabled = false;
         handleDownloadError(error);

src/main/resources/static/js/fileInput.js

@@ -170,7 +170,7 @@ function setupFileInput(chooser) {
 	inputContainer.querySelector('#fileInputText').innerHTML = window.fileInput.loading;
 
     async function checkZipFile() {
-      const hasZipFiles = allFiles.some(file => zipTypes.includes(file.type));
+      const hasZipFiles = allFiles.some(file => file.type && zipTypes.includes(file.type));
 
       // Only change to extractPDF message if we actually have zip files
       if (hasZipFiles) {

src/main/resources/static/js/homecard-legacy.js

@@ -255,5 +255,12 @@ document.addEventListener('DOMContentLoaded', function () {
     });
   }, 500);
 
+  Array.from(document.querySelectorAll('.feature-group-header')).forEach((header) => {
+    const parent = header.parentNode;
+    header.onclick = () => {
+      expandCollapseToggle(parent);
+    };
+  });
+
   showFavoritesOnly();
 });

src/main/resources/static/js/homecard.js

@@ -241,10 +241,5 @@ document.addEventListener('DOMContentLoaded', async function () {
       console.error('Material Symbols Rounded font failed to load.');
     });
 
-  Array.from(document.querySelectorAll('.feature-group-header')).forEach((header) => {
-    const parent = header.parentNode;
-    header.onclick = () => {
-      expandCollapseToggle(parent);
-    };
-  });
+
 });

src/main/resources/static/js/languageSelection.js

@@ -57,11 +57,15 @@ function initLanguageSettings() {
 
 function sortLanguageDropdown() {
   document.addEventListener('DOMContentLoaded', function () {
-    const dropdownMenu = document.querySelector('.dropdown-menu .dropdown-item.lang_dropdown-item').parentElement;
+    const dropdownMenu = document.getElementById('languageSelection');
     if (dropdownMenu) {
-      const items = Array.from(dropdownMenu.children).filter((child) => child.matches('a'));
+      const items = Array.from(dropdownMenu.children).filter((child) => child.querySelector('a'));
       items
-        .sort((a, b) => a.dataset.bsLanguageCode.localeCompare(b.dataset.bsLanguageCode))
+        .sort((wrapperA, wrapperB) => {
+          const a = wrapperA.querySelector('a');
+          const b = wrapperB.querySelector('a');
+          return a.dataset.bsLanguageCode.localeCompare(b.dataset.bsLanguageCode);
+        })
         .forEach((node) => dropdownMenu.appendChild(node));
     }
   });

src/main/resources/static/js/multitool/commands/delete-page.js

@@ -21,12 +21,10 @@ export class DeletePageCommand extends Command {
     this.pagesContainer.removeChild(this.element);
     if (this.pagesContainer.childElementCount === 0) {
       const filenameInput = document.getElementById("filename-input");
-      const filenameParagraph = document.getElementById("filename");
       const downloadBtn = document.getElementById("export-button");
 
       filenameInput.disabled = true;
       filenameInput.value = "";
-      filenameParagraph.innerText = "";
 
       downloadBtn.disabled = true;
     }
@@ -43,13 +41,10 @@ export class DeletePageCommand extends Command {
     }
 
     const filenameInput = document.getElementById("filename-input");
-    const filenameParagraph = document.getElementById("filename");
     const downloadBtn = document.getElementById("export-button");
 
     filenameInput.disabled = false;
     filenameInput.value = this.filenameInputValue;
-    if (this.filenameParagraph)
-      filenameParagraph.innerText = this.filenameParagraphText;
 
     downloadBtn.disabled = false;
   }
@@ -63,12 +58,10 @@ export class DeletePageCommand extends Command {
     this.pagesContainer.removeChild(this.element);
     if (this.pagesContainer.childElementCount === 0) {
       const filenameInput = document.getElementById("filename-input");
-      const filenameParagraph = document.getElementById("filename");
       const downloadBtn = document.getElementById("export-button");
 
       filenameInput.disabled = true;
       filenameInput.value = "";
-      filenameParagraph.innerText = "";
 
       downloadBtn.disabled = true;
     }

src/main/resources/static/js/pages/home.js

@@ -112,10 +112,10 @@ function setAsDefault(value) {
 
 function adjustVisibleElements() {
   const container = document.querySelector('.recent-features');
+  if(!container) return; 
   const subElements = Array.from(container.children);
 
   let totalWidth = 0;
-  const containerWidth = container.offsetWidth;
 
   subElements.forEach((element) => {
     totalWidth += 12 * parseFloat(getComputedStyle(document.documentElement).fontSize);

src/main/resources/templates/fragments/navbar.html

@@ -143,7 +143,7 @@
             </a>
             <div class="dropdown-menu dropdown-menu-tp" aria-labelledby="languageDropdown">
               <div class="dropdown-menu-wrapper px-xl-2 px-2">
-                <div class="scrollable-y lang_dropdown-mw scalable-languages-container">
+                <div id="languageSelection" class="scrollable-y lang_dropdown-mw scalable-languages-container">
                   <th:block th:insert="~{fragments/languages :: langs}"></th:block>
                 </div>
               </div>

src/main/resources/templates/login.html

@@ -143,7 +143,7 @@
             </button>
             <div class="dropdown-menu" aria-labelledby="languageDropdown">
               <!-- Here's where the fragment will be included -->
-              <div class="scrollable-y">
+              <div id="languageSelection" class="scrollable-y" >
               <th:block th:replace="~{fragments/languages :: langs}"></th:block>
               </div>
             </div>

src/main/resources/templates/misc/add-image.html

@@ -27,7 +27,7 @@
             </div>
             <script type="module" th:src="@{'/pdfjs-legacy/pdf.mjs'}"></script>
             <script type="module" th:src="@{'/js/pages/add-image.js'}"></script>
-            <div class="tab-group show-on-file-selected">
+            <div class="show-on-file-selected">
               <div
                 th:replace="~{fragments/common :: fileSelector(name='image-upload', disableMultipleFiles=false, multipleInputsForSingleRequest=true, accept='image/*', inputText=#{imgPrompt})}">
               </div>