Skip to main content
Code Review

Return to Answer

edited body
Source Link
user34073
user34073

Your CSS seems very good, although do you really need both hl and hr if they both oddo the same thing? Why don't you just get rid of one of them?

You have one error in your HTML - an unclosed div (http://validator.w3.org/check):

  <div id="fizzOut" >Change a value to get output (a snippet thing)
  <script>fizzing();</script>
</body>

That div should be closed:

<div id="fizzOut">Change a value to get output (a snippet thing)</div>

Your CSS seems very good, although do you really need both hl and hr if they both od the same thing? Why don't you just get rid of one of them?

You have one error in your HTML - an unclosed div (http://validator.w3.org/check):

  <div id="fizzOut" >Change a value to get output (a snippet thing)
  <script>fizzing();</script>
</body>

That div should be closed:

<div id="fizzOut">Change a value to get output (a snippet thing)</div>

Your CSS seems very good, although do you really need both hl and hr if they both do the same thing? Why don't you just get rid of one of them?

You have one error in your HTML - an unclosed div (http://validator.w3.org/check):

  <div id="fizzOut" >Change a value to get output (a snippet thing)
  <script>fizzing();</script>
</body>

That div should be closed:

<div id="fizzOut">Change a value to get output (a snippet thing)</div>
Source Link
user34073
user34073

Your CSS seems very good, although do you really need both hl and hr if they both od the same thing? Why don't you just get rid of one of them?

You have one error in your HTML - an unclosed div (http://validator.w3.org/check):

  <div id="fizzOut" >Change a value to get output (a snippet thing)
  <script>fizzing();</script>
</body>

That div should be closed:

<div id="fizzOut">Change a value to get output (a snippet thing)</div>