daveadmin/dobetternorge-tools
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: remove browser-native required validation blocking doc picker submissions

Browse Source 
Forms lacked novalidate and textareas had required, so the browser fired
HTML5 validation before tools.js could intercept — blocking submissions
where text came from the doc picker or file upload rather than the textarea.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
davegilligan
2026-05-24 00:34:38 +02:00
parent 2e8fda72d2
commit fa42c7223a
3 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
includes/tool_form.php
+2 -2
View File
@@ -1,4 +1,4 @@
<form id="toolForm" class="tool-form">
<form id="toolForm" class="tool-form" novalidate>
<div class="control-row" id="languageControl">
<span class="control-label">Language</span>
<label><input type="radio" name="language" value="en" checked> English</label>
@@ -82,7 +82,7 @@
</div>
<label class="input-label" for="toolInput" id="inputLabel">Question</label>
<textarea id="toolInput" name="toolInput" rows="10" required></textarea>
<textarea id="toolInput" name="toolInput" rows="10"></textarea>
<div class="form-footer">
<p id="toolStatus" class="form-status" role="status" aria-live="polite"></p>
redact.php
+2 -2
View File
@@ -6,7 +6,7 @@ $toolKind = 'Redaction Assistant';
$toolBadge = 'deterministic first';
require_once __DIR__ . '/includes/layout.php';
?>
<form id="toolForm" class="tool-form">
<form id="toolForm" class="tool-form" novalidate>
<div class="lang-switcher" id="redactLangSwitcher" role="group" aria-label="UI language">
<button type="button" class="lang-btn is-active" data-lang="en">&#127468;&#127463; EN</button>
@@ -108,7 +108,7 @@ require_once __DIR__ . '/includes/layout.php';
</div>
<label class="input-label" for="toolInput" id="inputLabel" data-i18n="redactInputLabel">Pasted text</label>
<textarea id="toolInput" name="toolInput" rows="10" required data-i18n-placeholder="redactInputPlaceholder" placeholder="Paste text containing names, phone numbers, emails, addresses, or national ID numbers."></textarea>
<textarea id="toolInput" name="toolInput" rows="10" data-i18n-placeholder="redactInputPlaceholder" placeholder="Paste text containing names, phone numbers, emails, addresses, or national ID numbers."></textarea>
<div class="form-footer">
<p id="toolStatus" class="form-status" role="status" aria-live="polite"></p>
timeline.php
+2 -2
View File
@@ -6,7 +6,7 @@ $toolKind = 'Timeline Builder';
$toolBadge = 'process-and-forget';
require_once __DIR__ . '/includes/layout.php';
?>
<form id="toolForm" class="tool-form">
<form id="toolForm" class="tool-form" novalidate>
<div class="lang-switcher" id="timelineLangSwitcher" role="group" aria-label="UI language">
<button type="button" class="lang-btn is-active" data-lang="en">&#127468;&#127463; EN</button>
@@ -87,7 +87,7 @@ require_once __DIR__ . '/includes/layout.php';
</div>
<label class="input-label" for="toolInput" id="inputLabel" data-i18n="timelineInputLabel">Pasted text</label>
<textarea id="toolInput" name="toolInput" rows="10" required data-i18n-placeholder="timelineInputPlaceholder" placeholder="Paste case notes, court decisions, or correspondence containing dates and events."></textarea>
<textarea id="toolInput" name="toolInput" rows="10" data-i18n-placeholder="timelineInputPlaceholder" placeholder="Paste case notes, court decisions, or correspondence containing dates and events."></textarea>
<div id="timelineNotesSection">
<label class="input-label" for="timelineNotes" data-i18n="timelineNotesLabel">Context notes <small class="control-hint">(optional)</small></label>