add delete button on details and edit views#2733
Conversation
I thought about that, but i found it more clear to have it on the top for many reasons: |
|
I think having the delete button in the bottom row with existing buttons sounds good. We could right align it to create distance. If it doesn't already, the delete process could also require a confirmation step to prevent accidental deletion. |
This comment was marked as resolved.
This comment was marked as resolved.
Fixes pallets-eco#1327 fix showing confirm message on Modal Align "Delete" button with other button groups in Edit page fix
4c860c9 to
dfd324a
Compare
|
I think merging #2847 would make this feature more straightforward. |
I don't think so, I think 2847 only affects inline editable on the list view, while this change is for the edit and detail views. |
Oh, I actually was talking about HTMX and ease of use that come with it 🤓. |


Fixes #1327
Tests case are added to make sure button is appeared in both edit and details views with both Modal and non-modal mode