Skip to content

Formatting improvements to BIP-75#14

Merged
techguy613 merged 1 commit intotechguy613:masterfrom
jonathancross:patch-2
Mar 11, 2016
Merged

Formatting improvements to BIP-75#14
techguy613 merged 1 commit intotechguy613:masterfrom
jonathancross:patch-2

Conversation

@jonathancross
Copy link
Copy Markdown

  • Fixing a few extra closing </b> tags and converting others to wiki bold syntax.
  • Linking "see below" and "see above" items to the actual section of the BIP.
  • Consistent capitalization of "Bitcoin".
  • "requester" => "requester* (more common outside of legal writing)
  • "concious" => "conscious"
  • "Foward" => "Forward"
  • "Satoshis" => "satoshis" (as unit of bitcoin, not the name of creator)
  • Removing unnecessary </img> which can actually cause problems.
  • Adding required alt attribute to img tags.
  • Fix wrapping of long lines (some were wrapped at 112 chars) - No effect on final rendering users see.

 * Fixing a few extra closing `b` tags and converting others to wiki bold syntax.
 * Linking "see below" and "see above" items to the actual section of the BIP.
 * Consistent capitalization of "Bitcoin".
 * "requester" => "requester* (more common outside of legal writing)
 * "concious" => "conscious"
 * "Foward" => "Forward"
 * "Satoshis" => "satoshis" (as unit of bitcoin, not the name of creator)
 * Removing unnecessary </img> which can actually cause problems.
 * Adding required `alt` attribute to img tags.
 * Fix wrapping of long lines (some were wrapped at 112 chars) - No effect on final rendering users see.
@jmacwhyte
Copy link
Copy Markdown

These look good, thanks for the review!

@techguy613
Copy link
Copy Markdown
Owner

Agreed with @jmacwhyte. Thanks for the review @jonathancross, merging this PR now.

techguy613 added a commit that referenced this pull request Mar 11, 2016
Formatting improvements to BIP-75
@techguy613 techguy613 merged commit d1e1c7c into techguy613:master Mar 11, 2016
@jonathancross
Copy link
Copy Markdown
Author

Thanks for the quick review and merge!
Cheers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants