That’s a massive improvement dod!!!
Some thoughts and I am sure it is still very much work in progress so if I hit on something that you have on your “to do” list then I apologise.
The layout is neat but I would replace the heading with a banner, this one thing will really spice up the whole page and also lead to giving your site an identity, don’t forget to use “alt” command for all graphics too, that way if they don’t load for any reason at least the viewer will know what was their as it will display text and not just a red X.
Instead of making the site alter its width with the browser, just make the whole background the same grey colour, that way it will fit in all browsers and keep things simple for you. If you really want to have the whole lot expand use “width 100%” instead of a set amount.
On your links page the colour of them is different to the colour of the link in your contact, I realise one takes you to email and the others are browser links but try to keep a running identity/colour scheme throughout the site. It will also be more effective if you have the links change colour when you hover over them.
You can easily alter that by adding in an extra bit to your css
this is what is effecting it currently..
Code:
.style6 {
font-family: Tahoma;
color: #FFFFFF;
}
by adding another bit of code, something like what I have put below (this is in addition as it will control the "on mouse over" or "hover" part
Code:
.style6 a:hover {
color:#cc6666;
background-color: #ffffff;
border-bottom: 1px dotted #cc6666;
}
obviously you would need to change the colours to suit your scheme (especially the background as I have used white in my example) but by adding stuff like the above it will allow you to have the links change colour on hover. My example also adds a dotted line underneath at the same time.
Another thing I would do at this point to keep everything simple is to move all the style commands from the pages themselves into a separate style sheet and just make one call to that at the beginning of each page. Not only is that the correct way to do it, it is also easy to work with as you have all the styling in one place and only have to alter one file to change the complete look of your site.
Name your style sheet anything you want but keep the .css extention, drop it into the same directory as the rest of your pages. Then on each of your pages just add one line into your page header
Code:
<link rel="stylesheet" type="text/css" href="whatever.css" media="screen" title="George Michie Photography.com" />
Finally, for now
You have not yet? Made a way for your downloads to be used, it says click to access choices but there is nowhere to click. I suspect that you just have not got that far yet.
Hope that helps
never done this before so any tips/hints/recommendations would be hugely appreciated
