Review:

CommentFileSizeAuthor
#3 book-settings-ui-text.patch1.64 KByoroy
booksettingsuitextreview-1.png108.26 KByoroy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

yoroy’s picture

yoroy CreditAttribution: yoroy commented

Pff, what's up with images not showing in initial posts?

john_jacobs’s picture

john_jacobs CreditAttribution: john_jacobs commented
Assigned: yoroy » Unassigned
Status: Needs review » Active

These changes read well to me, I agree that they are all an improvement, thanks for your work yoroy.

yoroy’s picture

yoroy CreditAttribution: yoroy commented
Assigned: Unassigned » yoroy
Status: Active » Needs review
FileSize
book-settings-ui-text.patch1.64 KB

Ok, here's a patch, this should be the result:

book-settings-after

I couldn't come up with a description that really added useful info to the child page content type selection so I removed it. Since it's radios, so a single choice, I decided to remove 'Default' from the label.

Dries’s picture

Dries CreditAttribution: Dries commented
Assigned: Unassigned » yoroy
Status: Active » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)
Issue tags: -Usability

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

Issue tags: +Usability

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.