Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
documentation
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
10 Feb 2012 at 21:25 UTC
Updated:
29 Jul 2014 at 20:23 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
scorchio commentedCould someone review this please? This patch adds the missing colons and rewraps the comments as necessary.
Comment #2
jhodgdonPlease read http://drupal.org/node/1354#lists again. The indentation is not correct. Thanks!
Comment #3
scorchio commentedOoops, sorry for that... Hopefully this one is a better try :)
Comment #4
jhodgdonThat's better! A couple of additional fixes are needed:
a)
This is all one paragraph. The 2nd line here needs to be moved up to the first line and then the rest re-wrapped to 80-character-max lines.
b)
The sub-list here should be preceded by : (at the end of the 2nd line). The next few list/sublists in the patch have the same problem.
Comment #5
kid_icarus commentedWell, here goes my second patch, hope it works out :)
Comment #6
cweagansTalked to kid_icarus about the trailing whitespace that I found. Other than that, this looks RTBC.
Comment #7
kid_icarus commentedFixes #6 as pointed out by cweagans.
Comment #8
cweagansPerfect. Welcome to core contribution, kid_icarus!
Comment #9
jhodgdonCommitted to 8.x and 7.x. Congratulations on your first core patch kid_icarus, and thanks scorchio for the original patch too!