Client's site is up, peruse through or make suggestions

Discussion in 'Web Design and Development' started by Mr. Mister, Aug 26, 2006.

  1. Mr. Mister macrumors 6502

    Feb 15, 2006

    Just put that up a week ago, it should pretty much work. Not too many issues I'm aware of, I'm pretty satisfied for it because while I'm 16 and new to the business I think it looks pretty decent.

    Mainly I'm looking for formatting problems but little recommendations are welcome as well. This is my first PHP-driven gallery, so there might be some broken links, report those here.

    Also, tell me about load time, I'm thinking I might compress some things a bit further if people are reporting long loads.
  2. Lixivial macrumors 6502a


    Jan 13, 2005
    Between cats, dogs and wanderlust.
    I like it. Simple and has a touch of elegance to it. Let me be the first to say that, in my opinion, you have done a superb job. Here're a few things I noticed, take them for what they're worth (which isn't usually much. ;))

    As for load times, on my DSL the sidebar images are loading perceptibly slow. I look at the file size of the sidebar images and they're not that taxing -- 120kbs -- but I can only imagine what they'd do on dial-up.

    - Some of the randomized sidebar images conflict with the "Plan Design Build Remodel" words and obscure them. Here's an example. But it looks as though it renders properly on all the major browsers.

    - You may want to handle photo 0 as it brings up a Red X and tells me that there are no more photos in the series. However by doing this it breaks the layout: here's an example. The problem is that the "Previous Photo" link is always displaying, even if I'm at the last photo in the series and this causes errors, because from photo 0 I can go to Photo -1, and keep going. This also happens when I'm at the last photo in the series, despite being told so. Here's an example. (only in this case, the "Next Photo" is always displaying) :)

    - I'd really think a link back to the gallery main page should be displayed on the "Previous/Next" photo browser. There should always be an escape route, because I like to use tabbed browsing and sometimes I want to start a fresh history with a newly opened tab and not mess with my previous tab's back/forward state.

    - Some of the photos are only half displaying. Here's an example. Or is it supposed to show the gray?

    - I think you should choose a bit more a browser-friendly resolution for some of the high-res gallery photos. Look at how this one displays. Here's an example. This would not only help with that layout problem, but also loading times as well. I realize high-res photos are great, but sometimes they're a bit too high-res when they're integrated into the website's layout. Some of the images are displaying well, such as these, but you should go back through and make sure all your images are looking that way. I did not peruse the entire gallery, but there are a number that are too wide. In the end, though, a user shouldn't have to scroll right and left unless it's completely unavoidable... (they're running at 800x600 for instance)

    -Nitpick: I know you like to display that your site is XHTML compliant and that it's just a beautifully standards-compliant site, but I'd advise against advertising this. What does it have to do with the client's site? That they were able to hire a proper web designer who can code to standards? Plus, not all your pages are valid. Here's an example. (I clicked that link from one of the photo display pages). Also, shouldn't clicking the "CSS" link take me to the CSS validator and the XHTML link take me to the XHTML validator?

    - Clicking beach houses and then clicking the picture above "View all pictures >>" throws an error. It links to an invalid IP address:; Condos does the same thing and so does Residential.

    - The remodeling tab is broken, or the layout breaks for some reason, and it contains no sidebar image. That's a bit "shocking".

    - Nitpick: I know that this is a nitpick, but I really like it when the website's main image links back to the root of the URL and not to its index page. ... ... I know, that's terrible...

    On the whole, once you work out a few minor kinks, it is a great and wonderful site. Good job!

    EDIT: Added a few points of clarification.
  3. decksnap macrumors 68040


    Apr 11, 2003
    Posting 'this site designed by' is a little amateurish. I would remove that.
  4. virus1 macrumors 65816


    Jun 24, 2004
    Agreed. But other than that, its an excelent site!

    Edit: 1,000th post!
  5. Mr. Mister thread starter macrumors 6502

    Feb 15, 2006
    Thing is, it would be completely normal to put a "Designed by Studio 39024" type of crap there.

    Lixivial: Thanks a lot for your feedback. The broken links are links that reference files on my local computer (that's what the is) and obviously were completely transparent to me during testing because it loads up the page fine on this comp because the files are there. Looking into fixing those.

    Overall the "fullsize image" part of the gallery is something I'm going to be improving, the images need to be smaller than 1024x768 AT LEAST and there's some stuff going on with some older CSS classes that I never undid.
  6. decksnap macrumors 68040


    Apr 11, 2003
    No, I don't think it would. The only reason to do that is if it was a completely pro bono type of thing. Perhaps it is?

    It's definitely not professional to put that there otherwise.
  7. cr2sh macrumors 68030


    May 28, 2002
    In the project gallery, I don't like the two seperate pages for "before" and "after" or "interior" and "exterior." I want to look at the images, see inside the house.. and having to navigate through (for no reason at all) slows me down.

    Put the galleries of each house on their own page and maybe seperate the interior/exteriors with headings... :)
  8. billy_d_goat macrumors member

    Mar 23, 2005
    A little nitpicky, but you might want to bold/sharpen up the font used for your red buttons at the top. Especially with the gloss effect, the text looks a bit lost or scrambled. Other than that, it looks pretty good!
  9. tuartboy macrumors 6502a


    May 10, 2005
    Check for null values on your querystring and display real error messages.
  10. eto macrumors member

    Apr 27, 2006
    20,000 leagues under the sea and or Duluth

    I agree... even if it is pro bono, you should make the credit less prominent.

    the layout is a little inflexible, if anything set up your navigation tabs to resize with the text... this can be as simple as using tiled backgrounds in your tabs and percentages for text size instead of px.

    nice design and color palette. oh yeah, nice integration of logo with the favicon. did you make the favicon?
  11. dornoforpyros macrumors 68040


    Oct 19, 2004
    Calgary, AB
    hmm to bad that it seems the OP has been banned, on the off chance he reads this, good job, keep with it, you've got a future in web/graphics if your only 16 and pumping out work of this quality.

    yeah you've got room to grow of course, but your off to a good start.

Share This Page