Skip to content

fix(knowledge): prevent navigation on context menu actions and widen tags modal #4015

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
waleedlatif1 merged 4 commits into from
Apr 7, 2026
Merged

fix(knowledge): prevent navigation on context menu actions and widen tags modal #4015

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
5343bc9
fix(knowledge): prevent navigation on context menu actions and widen …
waleedlatif1 Apr 7, 2026
dea71d1
fix(knowledge): guard onCopyId against navigation and use setTimeout …
waleedlatif1 Apr 7, 2026
7673726
refactor(knowledge): extract withActionGuard helper to deduplicate co…
waleedlatif1 Apr 7, 2026
2a82159
fix(knowledge): wrap withActionGuard callback in try/finally to preve…
waleedlatif1 Apr 7, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
refactor(knowledge): extract withActionGuard helper to deduplicate co… 
…ntext menu guard
  • Loading branch information
@waleedlatif1
waleedlatif1 committed Apr 7, 2026
commit 7673726a868a358b308fad9ad60b2b3eebd25caa
53 changes: 23 additions & 30 deletions apps/sim/app/workspace/[workspaceId]/knowledge/components/base-card/base-card.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,20 @@ export function BaseCard({
const [isDeleteModalOpen, setIsDeleteModalOpen] = useState(false)
const [isTagsModalOpen, setIsTagsModalOpen] = useState(false)
const [isDeleting, setIsDeleting] = useState(false)

/**
* Guards against context menu actions triggering card navigation.
* The card's onClick fires synchronously during the click event bubble phase,
* so the ref is checked before the setTimeout-0 callback resets it.
*/
const actionTakenRef = useRef(false)
const withActionGuard = useCallback((fn: () => void) => {
actionTakenRef.current = true
fn()
setTimeout(() => {
actionTakenRef.current = false
}, 0)
}, [])
Comment thread
waleedlatif1 marked this conversation as resolved.

const searchParams = new URLSearchParams({
kbName: title,
Expand Down Expand Up @@ -131,45 +144,25 @@ export function BaseCard({
)

const handleOpenInNewTab = useCallback(() => {
actionTakenRef.current = true
window.open(href, '_blank')
setTimeout(() => {
actionTakenRef.current = false
}, 0)
}, [href])
withActionGuard(() => window.open(href, '_blank'))
}, [href, withActionGuard])

const handleViewTags = useCallback(() => {
actionTakenRef.current = true
setIsTagsModalOpen(true)
setTimeout(() => {
actionTakenRef.current = false
}, 0)
}, [])
withActionGuard(() => setIsTagsModalOpen(true))
}, [withActionGuard])

const handleEdit = useCallback(() => {
actionTakenRef.current = true
setIsEditModalOpen(true)
setTimeout(() => {
actionTakenRef.current = false
}, 0)
}, [])
withActionGuard(() => setIsEditModalOpen(true))
}, [withActionGuard])

const handleDelete = useCallback(() => {
actionTakenRef.current = true
setIsDeleteModalOpen(true)
setTimeout(() => {
actionTakenRef.current = false
}, 0)
}, [])
withActionGuard(() => setIsDeleteModalOpen(true))
}, [withActionGuard])

const handleCopyId = useCallback(() => {
if (!id) return
actionTakenRef.current = true
navigator.clipboard.writeText(id)
setTimeout(() => {
actionTakenRef.current = false
}, 0)
}, [id])
withActionGuard(() => navigator.clipboard.writeText(id))
}, [id, withActionGuard])

const handleConfirmDelete = useCallback(async () => {
if (!id || !onDelete) return
Expand Down
Loading