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 CreditAttribution: sun.core commentedComment #2
amateescu CreditAttribution: amateescu commentedI wanted to do this for a very long time, so, finally, here's a patch :)
Comment #4
amateescu CreditAttribution: amateescu commentedNow with passing tests..
Comment #5
xjmAha. See: #1375134-11: Random test failure in TaxonomyTermTestCase::testNodeTermCreationAndDeletion()
Comment #6
amateescu CreditAttribution: 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 CreditAttribution: BTMash commented#6: 490182-add_delete_operation_to_taxonomy_overview_terms-3.patch queued for re-testing.
Comment #10
BTMash CreditAttribution: 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 CreditAttribution: BTMash commentedSetting back to needs work.
Comment #12
amateescu CreditAttribution: amateescu commentedRe-rolled against latest HEAD.
Comment #13
BTMash CreditAttribution: 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 CreditAttribution: Bojhan commentedLooks fine, wish we would document a standard for this
Comment #15
Dries CreditAttribution: Dries commentedCommitted to 8.x. Thanks! Glad to see some UX improvements go in.
Comment #16
Bojhan CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: Bojhan commentedonly 8
Comment #19
scottalan CreditAttribution: scottalan commentedWorking...
Comment #20
xjmHi scottalan,
Are you still working on this issue?
Comment #21
scottalan CreditAttribution: 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.