The admin/content screen has a section that lets you do bulk operations on nodes. But it has no explanation/description of the process, and I don't think it would be obvious to someone who hadn't already had experience using it in Drupal 6 or been tutored on it.

See screen shot.

I suggest either adding a description text to the UPDATE OPTIONS field, or renaming it, or both.

My suggestion would be to change the field to say

BULK OPERATIONS

And to add a short description below saying something like

Select the content items to operate on from the list below, select the operation to apply to those items, and click "Update" to perform the operation.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhodgdon’s picture

Title: Content admin screen not clear » Content admin screen not clear in bulk ops section
Issue tags: +ui-text
yoroy’s picture

Title: Content admin screen not clear in bulk ops section » Content admin screen not clear
Issue tags: -ui-text

Update options isn't ideal. main flaw is it doesn't link with having to select items in the table below. 'Bulk operations' are more difficult words.

'With selected items' ?

The proposed description is a variant on the 'this is a form that you have to fill out then click the button' pattern we've been actively weeding out from the UI.

jhodgdon’s picture

Issue tags: +ui-text

tag was removed, hopefully by mistake, by yoroy. Putting it back.

jhodgdon’s picture

Proposal:

Change the button text to "Update selected content" to explain what will happen if you click the button.
Change the label on the select list to "Choose action".

jhodgdon’s picture

Status: Active » Needs review
FileSize
10.56 KB
919 bytes

Here's a patch and a screen shot showing what it looks like.

jhodgdon’s picture

Component: node system » user interface text

Status: Needs review » Needs work
Issue tags: -ui-text

The last submitted patch, 677544.patch, failed testing.

Status: Needs work » Needs review
Issue tags: +ui-text

Re-test of 677544.patch from comment #5 was requested by jhodgdon.

jhodgdon’s picture

Reported failures were in Tracker module tests. Unlikely to be caused by this patch.

Status: Needs review » Needs work

The last submitted patch, 677544.patch, failed testing.

jhodgdon’s picture

Hmmm. Same failure. I will investigate.

jhodgdon’s picture

Assigned: Unassigned » jhodgdon

Real failure. I need to patch the Tracker test too. Coming soon...

jhodgdon’s picture

Status: Needs work » Needs review
FileSize
1.62 KB

This patch fixes the tests as well as the content screen. If someone could review the changes...

Bojhan’s picture

This patch will need to be applied to other places as well, say for example People

jhodgdon’s picture

Status: Needs review » Needs work

OK.

jhodgdon’s picture

FileSize
13.11 KB

Well, this is interesting. When I look at the People page, the options there are things like "Unblock the selected users". On the Content page, the options there are "Publish" (not "Publish the selected content"). See screen shot of people page.

So the People page is actually quite clear as it is, I think. So maybe a better fix for the Content page would be to ad "the selected content" to each of the options, rather than changing the button?

The Comments page is another spot (admin/content/comment), and that one is like People too.

I did a search for "Update options", and those three spots were all that came up.

So I think I'll fix node module so it has more verbose option names, and then I think we'll be consistent, clear, and happy. :)

jhodgdon’s picture

Status: Needs work » Needs review
FileSize
4.64 KB

Here's a patch. I also put the same items into the hook API documentation for the hook_node_operations hook.

Status: Needs review » Needs work

The last submitted patch, 677544consistency.patch, failed testing.

jhodgdon’s picture

Status: Needs work » Needs review
FileSize
4.61 KB

Here's a patch that should, as an added bonus, actually work. :)

Status: Needs review » Needs work
Issue tags: -ui-text

The last submitted patch, 677544syntax.patch, failed testing.

Status: Needs work » Needs review
Issue tags: +ui-text

Re-test of 677544syntax.patch from comment #19 was requested by jhodgdon.

jhodgdon’s picture

This test passed with the patch applied on my system...

cweagans’s picture

Status: Needs review » Reviewed & tested by the community

Patch applies with fuzz for me. Probably not an issue, though.

Everything works, and I like the changes to the strings.

jhodgdon’s picture

probably should wait for the retest to complete

jhodgdon’s picture

All of the locale tests definitely passed on my machine with this patch (that is what failed the last time).

cweagans’s picture

Um? Looks alright to me?

jhodgdon’s picture

Must have been a test glitch. Should be OK to go now!

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks!

Status: Fixed » Closed (fixed)
Issue tags: -ui-text

Automatically closed -- issue fixed for 2 weeks with no activity.