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.

#5 drupal-book-empty-pattern-D7-1473198-5.patch546 bytesroborn
PASSED: [[SimpleTest]]: [MySQL] 39,385 pass(es). View
#2 drupal-book-empty-pattern-1473198-2.patch566 byteskika
PASSED: [[SimpleTest]]: [MySQL] 35,140 pass(es). View
empty_book.png41.21 KBkika
Members fund testing for the Drupal project. Drupal Association Learn more


kika’s picture

Issue tags: +Novice
kika’s picture

Status: Active » Needs review
566 bytes
PASSED: [[SimpleTest]]: [MySQL] 35,140 pass(es). View

This one was messier but here's the patch.

roborn’s picture

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.

webchick’s picture

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?

roborn’s picture

Status: Patch (to be ported) » Needs review
546 bytes
PASSED: [[SimpleTest]]: [MySQL] 39,385 pass(es). View

Agree. Here's the patch for D7 :)

Devin Carlson’s picture

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.

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 7.x. Thanks!

Status: Fixed » Closed (fixed)

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

David_Rothstein’s picture

Issue tags: +7.17 release notes

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