problem deleting species in the new GUI

[hispanomys]

I tried deleting a species in one list (La Romieu) in the new GUI and nothing happens. Maybe I am doing something wrong, but when I am in edit mode and press the minus sign in front of the species I want to delete it brings me back to the beginning of the table.

[karilint]

At locality/?tab=2

• When in edit mode, and one presses the Actions (minus) in the Species section, the cursor jumps randomly to another species row and highlights it in red colour

[karilint]

[
{
"id": "T1",
"title": "Fix row-action target mapping in EditableTable",
"summary": "Update edit-mode row action logic so minus/plus actions mutate the correct source row when the table uses visible_data with different ordering.",
"app": "frontend",
"files_touched": [
"frontend/src/components/DetailView/common/EditableTable.tsx"
],
"migrations": false,
"permissions": [
"No role model changes",
"Behavior applies only to users already allowed to edit"
],
"acceptance_criteria": [
"Clicking minus on a rendered row always toggles that exact row's rowState",
"No unrelated row is marked removed/cancelled/new due to sorting/filtering order differences"
],
"test_plan": [
"Add/extend unit tests for row action mapping with visible_data and sorted order",
"Run frontend unit tests relevant to EditableTable"
],
"estimate_hours": 2.5,
"priority": "high"
},
{
"id": "T2",
"title": "Harden Locality Species tab integration",
"summary": "Ensure Locality->Species tab passes data in a way that preserves stable row identity for edit actions.",
"app": "frontend",
"files_touched": [
"frontend/src/components/Locality/Tabs/SpeciesTab.tsx"
],
"migrations": false,
"permissions": [
"No endpoint or auth middleware changes"
],
"acceptance_criteria": [
"At /locality/:id?tab=2 in edit mode, pressing minus on a species row affects only that row",
"Row coloring is consistent with that row's own rowState"
],
"test_plan": [
"Component-level test for species row action in sorted display",
"Manual verification in locality species tab with multiple rows"
],
"estimate_hours": 1.5,
"priority": "high"
},
{
"id": "T3",
"title": "Add end-to-end regression for locality species row actions",
"summary": "Extend Cypress locality tests to assert minus action does not jump to another row in tab=2.",
"app": "frontend-e2e",
"files_touched": [
"cypress/e2e/locality.cy.js"
],
"migrations": false,
"permissions": [
"Uses existing test login roles with edit capability"
],
"acceptance_criteria": [
"Cypress test fails on old behavior and passes on fixed behavior",
"Test asserts the clicked row is the row receiving removed/cancelled visual state"
],
"test_plan": [
"Run targeted Cypress spec for locality page",
"Optionally run full locality e2e suite in CI"
],
"estimate_hours": 2.0,
"priority": "high"
},
{
"id": "T4",
"title": "Update release notes/changelog",
"summary": "Record bugfix scope, affected UI path, and regression test addition.",
"app": "documentation",
"files_touched": [
"documentation/CHANGELOG.md"
],
"migrations": false,
"permissions": [
"N/A"
],
"acceptance_criteria": [
"Changelog entry references Locality -> Species edit-mode row action fix",
"Entry mentions regression coverage"
],
"test_plan": [
"Peer review changelog clarity"
],
"estimate_hours": 0.5,
"priority": "medium"
}
]

[karilint]

Summary

Fixed edit-mode row-action mapping in EditableTable so minus/plus actions resolve the correct source row even when visible_data is reordered (e.g., sorted), preventing unrelated rows from being toggled red/removed. The new logic first resolves by object identity in the editable array, then falls back to idFieldName matching, and preserves existing useObject behavior.