Skip to main content
Code Review

Return to Answer

Irrelevant
Source Link
edit approved Aug 18, 2014 at 15:01
user3845582
edit approved Aug 18, 2014 at 15:01
user3845582
  • 105
  • 1
  • 6

A few other things I can see right off the bat:

  • you're missing a <head> block; it's supposed to be before the <body> block, and the <title> block is supposed to be inside it.
  • you've duplicated the ending </html>
  • what's the purpose of the HTML comments? They don't explain anything.
  • Is <a href=""> blank in the real HTML or was it just redacted for posting to CodeReview?
  • are you using &nbsp; for spacing purposes? If so, don't -- use the proper CSS attributes to control spacing.
  • Tell your lawyers that the "Taking any action based on this email is strictly prohibited" disclaimer makes them look like fools.

A few other things I can see right off the bat:

  • you're missing a <head> block; it's supposed to be before the <body> block, and the <title> block is supposed to be inside it.
  • you've duplicated the ending </html>
  • what's the purpose of the HTML comments? They don't explain anything.
  • Is <a href=""> blank in the real HTML or was it just redacted for posting to CodeReview?
  • are you using &nbsp; for spacing purposes? If so, don't -- use the proper CSS attributes to control spacing.
  • Tell your lawyers that the "Taking any action based on this email is strictly prohibited" disclaimer makes them look like fools.

A few other things I can see right off the bat:

  • you're missing a <head> block; it's supposed to be before the <body> block, and the <title> block is supposed to be inside it.
  • you've duplicated the ending </html>
  • what's the purpose of the HTML comments? They don't explain anything.
  • Is <a href=""> blank in the real HTML or was it just redacted for posting to CodeReview?
  • are you using &nbsp; for spacing purposes? If so, don't -- use the proper CSS attributes to control spacing.
Source Link
Snowbody
Snowbody
  • 8.7k
  • 25
  • 50

A few other things I can see right off the bat:

  • you're missing a <head> block; it's supposed to be before the <body> block, and the <title> block is supposed to be inside it.
  • you've duplicated the ending </html>
  • what's the purpose of the HTML comments? They don't explain anything.
  • Is <a href=""> blank in the real HTML or was it just redacted for posting to CodeReview?
  • are you using &nbsp; for spacing purposes? If so, don't -- use the proper CSS attributes to control spacing.
  • Tell your lawyers that the "Taking any action based on this email is strictly prohibited" disclaimer makes them look like fools.