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.
Comment | File | Size | Author |
---|---|---|---|
#5 | drupal-book-empty-pattern-D7-1473198-5.patch | 546 bytes | roborn |
#2 | drupal-book-empty-pattern-1473198-2.patch | 566 bytes | kika |
empty_book.png | 41.21 KB | kika |
Comments
Comment #1
kika CreditAttribution: kika commentedComment #2
kika CreditAttribution: kika commentedThis one was messier but here's the patch.
Comment #3
roborn CreditAttribution: roborn commentedA 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.
Comment #4
webchickLooks good, thanks!
Committed and pushed to 8.x.
I think we need to backport this to D7 as well?
Comment #5
roborn CreditAttribution: roborn commentedAgree. Here's the patch for D7 :)
Comment #6
Devin Carlson CreditAttribution: Devin Carlson commentedThe patch in #5 applied cleanly and added the empty text to the "edit order and titles" pages.
Comment #7
webchickCommitted and pushed to 7.x. Thanks!
Comment #9
David_Rothstein CreditAttribution: David_Rothstein commentedThis is already mentioned in CHANGELOG.txt, so adding to the 7.17 release notes as well.