Wikipedia affirms the commonality of CRUD in user interfaces: http://en.wikipedia.org/wiki/Create,_read,_update_and_delete#User_interface. However, admin/content/taxonomy/tanonomyID is missing one of the atomic CRUD operations: delete. That operation requires a three step process:
- Click edit link.
- Click Delete button.
- Click Delete button (on different page).
Please expose delete as a first-order function on the taxonomy list page.
I suspect this is a Drupal design pattern. If so, I hope it can be revisited for the entire product.
| Comment | File | Size | Author |
|---|---|---|---|
| #13 | terms-list-post-patch-3.png | 52.05 KB | btmash |
| #13 | terms-list-post-patch-2.png | 42.8 KB | btmash |
| #13 | terms-list-post-patch-1.png | 56.23 KB | btmash |
| #13 | terms-list-pre-patch-3.png | 42.46 KB | btmash |
| #13 | terms-list-pre-patch-2.png | 73.77 KB | btmash |
Comments
Comment #1
sun.core commentedComment #2
amateescu commentedI wanted to do this for a very long time, so, finally, here's a patch :)
Comment #4
amateescu commentedNow with passing tests..
Comment #5
xjmAha. See: #1375134-11: Random test failure in TaxonomyTermTestCase::testNodeTermCreationAndDeletion()
Comment #6
amateescu commentedRerolled after #1375134: Random test failure in TaxonomyTermTestCase::testNodeTermCreationAndDeletion(). And this looks more like a task to me, we're not asking for anything more than UX consistency here :)
Comment #7
xjmBefore-and-after screenshots would be helpful here. Tagging as novice to:
Comment #8
btmash commented#6: 490182-add_delete_operation_to_taxonomy_overview_terms-3.patch queued for re-testing.
Comment #10
btmash commentedI ran into an issue while trying to apply the patch so I've queued the patch for retesting to ensure it is not user error. I suspect this may have to do with the crud api coming in.
Comment #11
btmash commentedSetting back to needs work.
Comment #12
amateescu commentedRe-rolled against latest HEAD.
Comment #13
btmash commentedOk, so after some user related issues (cache clearing), I can confirm that the test works. My work flow was:
1) Go to the term list page.

2) I see the delete action is missing. So I click on edit.


3) I see the delete action button and click on it.
I would finally be able to delete the term.
At this stage I apply the patch.


4) I go back to the term list page.
5) I saw the delete action link and clicked on it.
6) Just to confirm, I tried to delete the term and it was successful.
Comment #14
Bojhan commentedLooks fine, wish we would document a standard for this
Comment #15
dries commentedCommitted to 8.x. Thanks! Glad to see some UX improvements go in.
Comment #16
Bojhan commentedAwesome! Lets make sure we document this new standard in http://drupal.org/ui-standards - that we have delete as a operations in all tables.
Comment #17
btmash commentedJust so I understand, are ux improvements like this only being done for 8.x or will there be backports to 7.x?
Comment #18
Bojhan commentedonly 8
Comment #19
scottalan commentedWorking...
Comment #20
xjmHi scottalan,
Are you still working on this issue?
Comment #21
scottalan commentedI have gotten off track... I will touch base with zendoodles on this one to get more direction.
Comment #22
jibranCan we please create a new issue for #16?
Comment #23
xjmActually we don't need a new issue as this has been replaced with the Dropbutton pattern anyway.