-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Closed
Closed
Copy link
Labels
BugSomething is broken and not working as intended in the system.Something is broken and not working as intended in the system.Priority: MediumNon-blocking regression, noticable inconcistency, or important featureNon-blocking regression, noticable inconcistency, or important featureno-issue-activity
Description
Issue summary
The index table data-primary-link prop does not trigger a click/redirect after the first time. We think this is because of the isNavigating.current assignment on l101 of Row.tsx.
I'm not entirely sure of what the purpose is with isNavigating, but, it looks like it could almost be removed? Alternatively, it may just be able to be moved down below l110 after the if/else.
Expected behavior
It should keep triggering after the first click.
Actual behavior
It does not work after the first click.
Steps to reproduce the problem
- Use
data-primary-linksetup inIndexTable - Click the link to add a URL param or use an
onClickhandler to stay on the same page - Link no longer works
Reduced test case
https://codesandbox.io/s/unselectable-index-table-forked-utd6z
Metadata
Metadata
Assignees
Labels
BugSomething is broken and not working as intended in the system.Something is broken and not working as intended in the system.Priority: MediumNon-blocking regression, noticable inconcistency, or important featureNon-blocking regression, noticable inconcistency, or important featureno-issue-activity