This is part of the documentation project described at http://drupal.org/node/24768.
Please apply to DRUPAL-4-6 and HEAD.

CommentFileSizeAuthor
#3 image-module-help-text.diff1.12 KBStefan Nagtegaal
#2 image-cvs-26047.patch2.36 KBRobrecht Jacques
image.module_0.diff2.59 KBPunkCut25
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Robin Monks’s picture

Robin Monks CreditAttribution: Robin Monks commented

-1

Sorry, a few notes:

- Change 'admin/help#image': return t(' to be two seporate lines.
- Try to keep <p>sentences</p>'s together on one line for consistancy
- Add some extra whitespace before ordered list items (1 space should do it) for readibility
- title = "imagepage" Make real titles, for those with screen-readers, etc.
- Please remove the break;.

Robin

Robrecht Jacques’s picture

Robrecht Jacques CreditAttribution: Robrecht Jacques commented
Status: Active » Needs review
FileSize
image-cvs-26047.patch2.36 KB

Updated the patch.
- applies to current CVS,
- incorporated the remarks,
- not a native English speaker, so I didn't touch the text.

Stefan Nagtegaal’s picture

Stefan Nagtegaal CreditAttribution: Stefan Nagtegaal commented
Assigned: Unassigned » Stefan Nagtegaal
Status: Needs review » Reviewed & tested by the community
FileSize
image-module-help-text.diff1.12 KB

Okay, the helptext are up-to-date, but only aren't wrapped into <p>-aragraph tags.
Attached patch fixes this..

This simple patch is ready to be committed..

walkah’s picture

walkah CreditAttribution: walkah commented
Status: Reviewed & tested by the community » Fixed
Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)