jramos/cve-dashboard
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: remove dual-mode checkbox — clicks always toggle selection, no more popover on first click

Browse Source
This commit is contained in:
jramos
2026-04-09 10:01:18 -06:00
parent d9fdaf5cbb
commit fc68097821
1 changed files with 1 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
frontend/src/components/pages/ReportingPage.js
View File

@@ -3107,13 +3107,6 @@ export default function VulnerabilityTriagePage({ filterDate, filterEXC }) {
style={{ padding: '0.45rem 0.5rem', textAlign: 'center', width: '36px' }} style={{ padding: '0.45rem 0.5rem', textAlign: 'center', width: '36px' }}
onClick={(e) => { onClick={(e) => {
if (queued) return; if (queued) return;
// If nothing selected and not shift-click, open single-add popover
if (selectedIds.size === 0 && !e.shiftKey) {
const rect = e.currentTarget.getBoundingClientRect();
setAddPopover({ finding, anchorRect: rect });
setQueueForm({ vendor: '', workflowType: 'FP' });
return;
}
// Shift-click range select // Shift-click range select
if (e.shiftKey && lastClickedId) { if (e.shiftKey && lastClickedId) {
const lastIdx = sorted.findIndex((f) => f.id === lastClickedId); const lastIdx = sorted.findIndex((f) => f.id === lastClickedId);
@@ -3130,7 +3123,7 @@ export default function VulnerabilityTriagePage({ filterDate, filterEXC }) {
}); });
} }
} else { } else {
// Toggle selection // Regular click — toggle selection
setSelectedIds((prev) => { setSelectedIds((prev) => {
const next = new Set(prev); const next = new Set(prev);
if (next.has(finding.id)) next.delete(finding.id); else next.add(finding.id); if (next.has(finding.id)) next.delete(finding.id); else next.add(finding.id);