Skip to main content
Code Review

Return to Answer

deleted 2 characters in body
Source Link
Jamal
Jamal
  • 35.2k
  • 13
  • 134
  • 238

A few semantic issues:

  1. Do not use <b> tags.
    If you must, you can use <strong>or, or even better, <span class="bold"></span>, and then style the class in your cssCSS.

  2. Do not use <i> tags.
    If you must, you can use <em>or, or even better, <span class="italic"></span>, and then style the class in your cssCSS.

  3. Do not use <br /> tags for spacing.
    If you require spacing, use margin-bottom, or padding-bottom.

A few semantic issues:

  1. Do not use <b> tags.
    If you must you can use <strong>or even better <span class="bold"></span> and then style the class in your css.

  2. Do not use <i> tags.
    If you must you can use <em>or even better <span class="italic"></span> and then style the class in your css.

  3. Do not use <br /> tags for spacing.
    If you require spacing use margin-bottom, or padding-bottom.

A few semantic issues:

  1. Do not use <b> tags.
    If you must, you can use <strong>, or even better, <span class="bold"></span>, and then style the class in your CSS.

  2. Do not use <i> tags.
    If you must, you can use <em>, or even better, <span class="italic"></span>, and then style the class in your CSS.

  3. Do not use <br /> tags for spacing.
    If you require spacing, use margin-bottom or padding-bottom.

Source Link
Hailwood
Hailwood
  • 467
  • 4
  • 11

A few semantic issues:

  1. Do not use <b> tags.
    If you must you can use <strong>or even better <span class="bold"></span> and then style the class in your css.

  2. Do not use <i> tags.
    If you must you can use <em>or even better <span class="italic"></span> and then style the class in your css.

  3. Do not use <br /> tags for spacing.
    If you require spacing use margin-bottom, or padding-bottom.