Download & Extend

Empty table pattern for "Edit [book's] order and titles" admin page

Project:Drupal core
Version:7.x-dev
Component:book.module
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)
Issue tags:7.17 release notes, d8ux, needs backport to D7, Novice, UI pattern, Usability

Issue Summary

If there are no subpages in a book (just a root page), the book admin page admin/content/book/%nid shows a table header with no content in table. According to http://drupal.org/node/604342 and #504428: Roll-out 'Empty text' pattern to other admin pages. empty table should contain an explanation and/or call to action.

Proposed wording if there is no content in the table:

No book content available.

Why not call to action link? Concistency: we currently do not do it for content admin page. Create a separate issue to contest / sync this.

AttachmentSizeStatusTest resultOperations
empty_book.png41.21 KBIgnored: Check issue status.NoneNone

Comments

#1

Issue tags:+Novice

#2

Status:active» needs review

This one was messier but here's the patch.

AttachmentSizeStatusTest resultOperations
drupal-book-empty-pattern-1473198-2.patch566 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 35,140 pass(es).View details

#3

Status:needs review» reviewed & tested by the community

A call to action link would make sense here, but i agree: let's keep it consistent.

The patch applies cleanly and works as expected. RTBC.

#4

Version:8.x-dev» 7.x-dev
Status:reviewed & tested by the community» patch (to be ported)
Issue tags:+needs backport to D7

Looks good, thanks!

Committed and pushed to 8.x.

I think we need to backport this to D7 as well?

#5

Status:patch (to be ported)» needs review

Agree. Here's the patch for D7 :)

AttachmentSizeStatusTest resultOperations
drupal-book-empty-pattern-D7-1473198-5.patch546 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 39,385 pass(es).View details

#6

Status:needs review» reviewed & tested by the community

The patch in #5 applied cleanly and added the empty text to the "edit order and titles" pages.

#7

Status:reviewed & tested by the community» fixed

Committed and pushed to 7.x. Thanks!

#8

Status:fixed» closed (fixed)

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

#9

Issue tags:+7.17 release notes

This is already mentioned in CHANGELOG.txt, so adding to the 7.17 release notes as well.

nobody click here