Perhaps it's by design, but it doesn't seem natural to have "We read" as the default block label for each of the read, reviewed and recommended blocks that are available.

The attached patch provides some (to me) more natural block labels. It also uses the block label variables with an "Amazon:" prefix when listing the blocks on admin/block (to make it easier to identify them when ordering/enabling).

Thanks,
Curtis.

CommentFileSizeAuthor
amazon.module.diff1.97 KBAnonymous (not verified)

Comments

Prometheus6’s picture

The default labels are just defaults. It doesn't matter much what they are one way or the other. Making it clearer which is which in the block configuration form is a good thought, though.

Prometheus6’s picture

Assigned: Unassigned » Prometheus6
Status: Reviewed & tested by the community » Closed (fixed)

Okay, that's done.