Jump to content

jvincent

New Members
  • Posts

    3
  • Joined

  • Last visited

Posts posted by jvincent

  1. I like the look of the page.

     

    The navigation is not contained in the actual content, and when viewed with a wider resolution, it moves too far to the left:

    thrift.jpg

     

    There are several validation errors that should be fixed.

     

    Justifying text can easily cause display issues - at least add the omitted space between couches and entertainment - that'll help a bit.

     

    thrift2.jpg

     

    Can you find a photo without a date stamp or remove the one you're showing now?

     

    It's best to put all the styling into your external stylesheet, not inline. The only time inline is ok is if a certain formatting happens one time only on your entire site.

     

    Header tags (h1, h2, h3, etc) are SEO and formatting tools and should be used to define headings on your page. Content like 'Open Everyday' and '10:30 - 5:30' doesn't quite meet header criteria. But that aside, format your header tags in your stylesheet not inline.

     

    The scroll marquee isn't working in any of the newer browsers, and that's a good thing - scrolling is kind of a frowned-upon outdated feature from the 90s.

     

    There is an empty box underneath the Facebook link.

     

    Your CSS could use some streamlining. For example, anything 'normal' is default, so no need to even specify it.

     

    Under "#login_box h1 and h2" you first declare text-align left, and in the next line, center. Stylesheets are 'read' from top to bottom, so the lowest style applies, in this case 'center'.

     

    there is not 'float: center"

     

    Check your entire HTML layout - for example, your footer is styled as 800 px wide and it actually spreads underneath your entire contend. You probably don't need all those floats in your CSS, either.

     

     

    Thank you very much, I'll work on it right away. I really appreciate your review. Have a great new year.

×
×
  • Create New...