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


Issue tags:+Novice

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

This one was messier but here's the patch.

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.

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?

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

Agree. Here's the patch for D7 :)

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.

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.

Issue tags:+7.17 release notes

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