Allow decimal feedback days (0.1 step) in Brief Advisor
- Add step="0.1" to all 4 feedback day inputs (static, video, HTML, translation) - Change parseInt → parseFloat so fractional values like 0.5 or 2.5 are preserved - Business day arithmetic naturally rounds up fractions to next whole business day Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
parent
80d90da95e
commit
ac9a3f057b
2 changed files with 8 additions and 8 deletions
|
|
@ -190,10 +190,10 @@ function formatDate(date) {
|
|||
// ---- Read user-supplied feedback days (defaults to 0 if toggle is off) ----
|
||||
function getUserFeedbackDays() {
|
||||
return {
|
||||
static: document.getElementById('needsStatic').checked ? parseInt(document.getElementById('feedbackDaysStatic').value) || 0 : 0,
|
||||
video: document.getElementById('needsVideo').checked ? parseInt(document.getElementById('feedbackDaysVideo').value) || 0 : 0,
|
||||
html: document.getElementById('needsHTML').checked ? parseInt(document.getElementById('feedbackDaysHTML').value) || 0 : 0,
|
||||
translation: document.getElementById('needsTranslation').checked ? parseInt(document.getElementById('feedbackDaysTranslation').value) || 0 : 0
|
||||
static: document.getElementById('needsStatic').checked ? parseFloat(document.getElementById('feedbackDaysStatic').value) || 0 : 0,
|
||||
video: document.getElementById('needsVideo').checked ? parseFloat(document.getElementById('feedbackDaysVideo').value) || 0 : 0,
|
||||
html: document.getElementById('needsHTML').checked ? parseFloat(document.getElementById('feedbackDaysHTML').value) || 0 : 0,
|
||||
translation: document.getElementById('needsTranslation').checked ? parseFloat(document.getElementById('feedbackDaysTranslation').value) || 0 : 0
|
||||
};
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -109,7 +109,7 @@
|
|||
<div id="feedbackStatic" class="hidden border-t border-gray-200 dark:border-gray-600 bg-gray-50 dark:bg-gray-700/30 px-3 py-2.5">
|
||||
<label class="flex items-center justify-between">
|
||||
<span class="text-xs font-medium text-gray-600 dark:text-gray-400">Days needed for feedback</span>
|
||||
<input type="number" id="feedbackDaysStatic" min="0" max="30" value="3" class="w-16 text-center rounded-md border border-gray-300 dark:border-gray-600 bg-white dark:bg-gray-700 px-2 py-1 text-sm focus:ring-2 focus:ring-brand-500 outline-none">
|
||||
<input type="number" id="feedbackDaysStatic" min="0" max="30" step="0.1" value="3" class="w-16 text-center rounded-md border border-gray-300 dark:border-gray-600 bg-white dark:bg-gray-700 px-2 py-1 text-sm focus:ring-2 focus:ring-brand-500 outline-none">
|
||||
</label>
|
||||
</div>
|
||||
</div>
|
||||
|
|
@ -130,7 +130,7 @@
|
|||
<div id="feedbackVideo" class="hidden border-t border-gray-200 dark:border-gray-600 bg-gray-50 dark:bg-gray-700/30 px-3 py-2.5">
|
||||
<label class="flex items-center justify-between">
|
||||
<span class="text-xs font-medium text-gray-600 dark:text-gray-400">Days needed for feedback</span>
|
||||
<input type="number" id="feedbackDaysVideo" min="0" max="30" value="3" class="w-16 text-center rounded-md border border-gray-300 dark:border-gray-600 bg-white dark:bg-gray-700 px-2 py-1 text-sm focus:ring-2 focus:ring-brand-500 outline-none">
|
||||
<input type="number" id="feedbackDaysVideo" min="0" max="30" step="0.1" value="3" class="w-16 text-center rounded-md border border-gray-300 dark:border-gray-600 bg-white dark:bg-gray-700 px-2 py-1 text-sm focus:ring-2 focus:ring-brand-500 outline-none">
|
||||
</label>
|
||||
</div>
|
||||
</div>
|
||||
|
|
@ -151,7 +151,7 @@
|
|||
<div id="feedbackHTML" class="hidden border-t border-gray-200 dark:border-gray-600 bg-gray-50 dark:bg-gray-700/30 px-3 py-2.5">
|
||||
<label class="flex items-center justify-between">
|
||||
<span class="text-xs font-medium text-gray-600 dark:text-gray-400">Days needed for feedback</span>
|
||||
<input type="number" id="feedbackDaysHTML" min="0" max="30" value="3" class="w-16 text-center rounded-md border border-gray-300 dark:border-gray-600 bg-white dark:bg-gray-700 px-2 py-1 text-sm focus:ring-2 focus:ring-brand-500 outline-none">
|
||||
<input type="number" id="feedbackDaysHTML" min="0" max="30" step="0.1" value="3" class="w-16 text-center rounded-md border border-gray-300 dark:border-gray-600 bg-white dark:bg-gray-700 px-2 py-1 text-sm focus:ring-2 focus:ring-brand-500 outline-none">
|
||||
</label>
|
||||
</div>
|
||||
</div>
|
||||
|
|
@ -172,7 +172,7 @@
|
|||
<div id="feedbackTranslation" class="hidden border-t border-gray-200 dark:border-gray-600 bg-gray-50 dark:bg-gray-700/30 px-3 py-2.5">
|
||||
<label class="flex items-center justify-between">
|
||||
<span class="text-xs font-medium text-gray-600 dark:text-gray-400">Days needed for feedback</span>
|
||||
<input type="number" id="feedbackDaysTranslation" min="0" max="30" value="5" class="w-16 text-center rounded-md border border-gray-300 dark:border-gray-600 bg-white dark:bg-gray-700 px-2 py-1 text-sm focus:ring-2 focus:ring-brand-500 outline-none">
|
||||
<input type="number" id="feedbackDaysTranslation" min="0" max="30" step="0.1" value="5" class="w-16 text-center rounded-md border border-gray-300 dark:border-gray-600 bg-white dark:bg-gray-700 px-2 py-1 text-sm focus:ring-2 focus:ring-brand-500 outline-none">
|
||||
</label>
|
||||
</div>
|
||||
</div>
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue