Developer
a12d6a8169
- Wire MobileListView onItemDelete/onItemDuplicate callbacks to render action buttons in each list row. - Pass onItemDelete in ToolWorkshopMobileView list view. - Add CSS for mobile-list-item-action buttons. - Fix MobileEditView sticky bottom action bar that was hidden behind the 64px mobile navigation bar; raise to bottom: 64px and z-index 110. Quality gates: npm run typecheck, npm run lint, npm test -- --run (87 passed) Refs: openspec/changes/mobile-list-delete-button
39 lines
2.1 KiB
Markdown
39 lines
2.1 KiB
Markdown
# Restore Delete Button in Mobile List Views and Fix Edit Action Bar
|
|
|
|
## Summary
|
|
|
|
The `MobileListView` component accepted `onItemDelete` and `onItemDuplicate` callbacks but never rendered the corresponding action buttons. This change restores the delete button (and adds duplicate support) in each mobile list row for any view that supplies the callback.
|
|
|
|
In addition, the sticky bottom action bar added to `MobileEditView` in the previous change was positioned behind the 64px mobile navigation bar, making the Save and Delete buttons impossible to see on mobile. The bar is now raised above the navigation and given a higher z-index.
|
|
|
|
## Motivation
|
|
|
|
Users need a quick way to remove config profiles and tool types directly from the mobile list without first opening the edit view. The callback props already existed, so wiring them up restores intended behavior.
|
|
|
|
Users also reported that the Save/Delete buttons in the mobile edit view were not visible. Investigation showed the fixed-position action bar was rendering underneath the persistent mobile bottom navigation.
|
|
|
|
## Scope
|
|
|
|
- `MobileListView` shared component.
|
|
- `ToolWorkshopMobileView` (passes `onItemDelete`).
|
|
- `ConfigProfilesMobileView` already passed `onItemDelete`; it now actually renders.
|
|
- `MobileEditView` sticky action bar CSS (`utilities.css`).
|
|
- Mobile list item action CSS (`utilities.css`).
|
|
|
|
## Acceptance Criteria
|
|
|
|
- [x] `MobileListView` renders a delete button when `onItemDelete` is provided.
|
|
- [x] `MobileListView` renders a duplicate button when `onItemDuplicate` is provided.
|
|
- [x] Tapping delete/duplicate does not also trigger the row click.
|
|
- [x] `ToolWorkshopMobileView` list rows show the delete button.
|
|
- [x] `ConfigProfilesMobileView` list rows show the delete button.
|
|
- [x] Rows without delete/duplicate still show the chevron.
|
|
- [x] The `MobileEditView` sticky bottom action bar sits above the mobile navigation bar.
|
|
- [x] Save and Delete buttons in the edit view are visible and reachable.
|
|
|
|
## Related
|
|
|
|
- `openspec/changes/mobile-edit-default-bottom-actions`
|
|
- `openspec/changes/mobile-config-profiles-ui`
|
|
- `openspec/changes/mobile-tool-profile-ui`
|