Fix test results behavior - remove summary from upload results and hide tests on new image
✅ Fixed Summary Message Placement: - Removed summary message from custom upload results (displayResults function) - Keep dynamic message only in 'Run All Tests' results - Upload results now show only detection stats and details ✅ Hide Test Results on New Image: - Test results automatically hidden when new image is uploaded - Test results hidden when file upload is reset - User must click 'Run All Tests' again to see results for new image ✅ Improved User Experience: - Clear separation between upload results and test results - Dynamic messages only appear where expected (Run All Tests) - Clean workflow: upload → test results disappear → click test → see results - No confusing duplicate summary messages ✅ Behavior Flow: 1. Upload image → No summary message in upload results 2. Click 'Run All Tests' → Shows '✅ Found X memory modules' or '❌ No memory modules' 3. Upload new image → Test results disappear automatically 4. Click 'Run All Tests' again → Fresh results with dynamic message Perfect test results behavior as requested
This commit is contained in:
+12
-14
@@ -185,6 +185,12 @@ function handleFile(file) {
|
|||||||
|
|
||||||
uploadedFile = file;
|
uploadedFile = file;
|
||||||
|
|
||||||
|
// Hide test results when new image is uploaded
|
||||||
|
const testResultsSection = document.getElementById('testResultsSection');
|
||||||
|
if (testResultsSection) {
|
||||||
|
testResultsSection.style.display = 'none';
|
||||||
|
}
|
||||||
|
|
||||||
// Show file info with change file option
|
// Show file info with change file option
|
||||||
const uploadArea = document.getElementById('uploadArea');
|
const uploadArea = document.getElementById('uploadArea');
|
||||||
uploadArea.innerHTML = `
|
uploadArea.innerHTML = `
|
||||||
@@ -231,6 +237,12 @@ function resetFileUpload() {
|
|||||||
// Hide results if showing
|
// Hide results if showing
|
||||||
document.getElementById('resultsSection').style.display = 'none';
|
document.getElementById('resultsSection').style.display = 'none';
|
||||||
|
|
||||||
|
// Hide test results when file is reset
|
||||||
|
const testResultsSection = document.getElementById('testResultsSection');
|
||||||
|
if (testResultsSection) {
|
||||||
|
testResultsSection.style.display = 'none';
|
||||||
|
}
|
||||||
|
|
||||||
// Reinitialize the upload area with fresh event listeners
|
// Reinitialize the upload area with fresh event listeners
|
||||||
initializeUploadArea();
|
initializeUploadArea();
|
||||||
|
|
||||||
@@ -316,18 +328,6 @@ function displayResults(result, title) {
|
|||||||
const resultsSection = document.getElementById('resultsSection');
|
const resultsSection = document.getElementById('resultsSection');
|
||||||
const resultsContent = document.getElementById('resultsContent');
|
const resultsContent = document.getElementById('resultsContent');
|
||||||
|
|
||||||
// Create smart summary message
|
|
||||||
let summaryMessage = '';
|
|
||||||
if (result.success) {
|
|
||||||
if (result.num_detections > 0) {
|
|
||||||
summaryMessage = `<div class="summary-message success">✅ Found ${result.num_detections} memory modules</div>`;
|
|
||||||
} else {
|
|
||||||
summaryMessage = `<div class="summary-message no-memory">❌ No memory modules</div>`;
|
|
||||||
}
|
|
||||||
} else {
|
|
||||||
summaryMessage = `<div class="summary-message error">❌ Error: ${result.error}</div>`;
|
|
||||||
}
|
|
||||||
|
|
||||||
let detectionsHtml = '';
|
let detectionsHtml = '';
|
||||||
if (result.detections && result.detections.length > 0) {
|
if (result.detections && result.detections.length > 0) {
|
||||||
detectionsHtml = result.detections.map((detection, index) => `
|
detectionsHtml = result.detections.map((detection, index) => `
|
||||||
@@ -347,8 +347,6 @@ function displayResults(result, title) {
|
|||||||
<span class="badge">${new Date().toLocaleTimeString()}</span>
|
<span class="badge">${new Date().toLocaleTimeString()}</span>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
${summaryMessage}
|
|
||||||
|
|
||||||
<div class="result-stats">
|
<div class="result-stats">
|
||||||
<div class="stat-item">
|
<div class="stat-item">
|
||||||
<div class="stat-value">${result.num_detections}</div>
|
<div class="stat-value">${result.num_detections}</div>
|
||||||
|
|||||||
Reference in New Issue
Block a user