9 replies [Last post]
dthomas31uk
Offline
Enthusiast
Last seen: 12 years 37 weeks ago
Joined: 2004-11-29
Posts: 144
Points: 0

Hi, Just created a website

http://www.suninvest.org.uk/index.html

Would be most appreciated if you could give me some feedback as to what anyone thinks.

Regards

Darren

Anonymous
Anonymous's picture
Guru

Please can you advise

The colors make me happy. Laughing out loud

You have one validation error. Looks like a missing </p> tag here:

<h1>GETTING THERE!</h1>
<p>With two main airports on this coast.  Alicante provides easy access to the Costa Blanca and Murcia to the Costa Calida with daily flights to and from the UK.
</div>

I didn't check the other pages. I like it. Nice work! Smile

dthomas31uk
Offline
Enthusiast
Last seen: 12 years 37 weeks ago
Joined: 2004-11-29
Posts: 144
Points: 0

Please can you advise

cheers for the tips triumph....much appreciated

fatfreddy
fatfreddy's picture
Offline
Enthusiast
Deep down under
Last seen: 13 years 26 weeks ago
Deep down under
Timezone: GMT+8
Joined: 2005-03-25
Posts: 390
Points: 0

Please can you advise

A nice looking site.

I would think about optimising the top three images on eachpage a little more - they are coming in at a fairly hefty size (27k + each)..

They load quite slowly on my 512k broadband connection and would annoy people on a 56k connection - just a thought

The only reason for time is so that everything doesn't happen at once
--Albert Einstein

Accessible webdesign Australia - webechodesigns

dthomas31uk
Offline
Enthusiast
Last seen: 12 years 37 weeks ago
Joined: 2004-11-29
Posts: 144
Points: 0

Please can you advise

Cheers for the advice fatfreddy. Thats a good point...will get that sorted.

Thanks

ABCruz
ABCruz's picture
Offline
Enthusiast
Pilipinas
Last seen: 6 years 37 weeks ago
Pilipinas
Timezone: GMT+8
Joined: 2005-06-16
Posts: 279
Points: 0

Please can you advise

Hello. I think the background color is too strong-- making it difficult for me to focus on the content. However, everything inside the page seems to be in order (I used both my hands to cover the background to see it Laughing out loud )

You should also do something about the flexibility of your containing elements because when I tried to resize the text, everything broke apart-- and I think many viewers will be resizing the text because it is too small.

Cheers. Smile

I am a Man of Truth.
I am a Free Human Person.
I am a Peacemaker.

http://www.abcruz.com

Hugo
Hugo's picture
Offline
Moderator
London
Last seen: 6 years 1 week ago
London
Joined: 2004-06-06
Posts: 15668
Points: 2806

Please can you advise

Agree with above you have gone entirely too small with text: body 74% and p .9em is too small., and you do need to re-think the flexibility of your divs the layout is breaking up to quickly you should aim for two comfortable increases.

Hugo.

Before you make your first post it is vital that you READ THE POSTING GUIDELINES!
----------------------------------------------------------------
Please post ALL your code - both CSS & HTML - in [code] tags
Please validate and ensure you have included a full Doctype before posting.
Why validate? Read Me

dthomas31uk
Offline
Enthusiast
Last seen: 12 years 37 weeks ago
Joined: 2004-11-29
Posts: 144
Points: 0

Please can you advise

Cheers for your advice abcruz and Hugo. Thanks for your constrcutive criticism.

'background color is too strong' Just tied it in with the suninvest logo, making it a lighter yellow. What would you recommend????

As for the flxibility of the containing elements....see what you mean...i agree....how can I get around this?????

Your advice is much appreciated tho' regards

ABCruz
ABCruz's picture
Offline
Enthusiast
Pilipinas
Last seen: 6 years 37 weeks ago
Pilipinas
Timezone: GMT+8
Joined: 2005-06-16
Posts: 279
Points: 0

Please can you advise

Hi again Darren,

I didn't have anything much to do tonight so I decided to attempt to recreate your stylesheet. I hope you don't mind. Here's what I came up with (as my suggestion on how to make your containers more flexible):

body {
  font-family : verdana, sans-serif ;
  font-size : small ;
  
  background : rgb(225,210,150) ;
  margin : 0px ;
  padding : 0px
}

h1 {
  font-family : serif ;
  font-size : 1.5em ;
  
  background : orange ;
  border-bottom : 1px rgb(0,0,0) solid ;
  margin : 0px ;
  padding : 5px
}

p {
  padding : 0px 20px ;
}

#container {
  width : 790px ;
  
  background : #0099D8 ;
  border : 2px rgb(150,140,100) solid ;
  border-bottom : 0px ;
  border-top : 0px ;
  margin : 0px auto ;
  padding : 0px
}

  #toprectangle {
    display : none ;
  }
  
  #navcontainer {
    background : rgb(255,255,255) ;
    border:0 ;
    margin: 0px ;
    padding: 0 ;
  }
  
      #navcontainer ul {
        text-align : center ;
        
        background: #0099D8 ;
        margin : 0px 0px 0px 135px ;
        padding : 2px 0px ;
      }
  
      #navcontainer ul li {
        list-style-type : none ;
        display : inline ;
        margin : 0px 5px
      }
      
      #navcontainer a { color : #FFEF10 ; text-decoration : none }
      #navcontainer a:hover { color : rgb(255,255,255) }
      
  #logo {
    height: 218px ;
    width:135px ;

    text-indent: -4000px ;
    text-decoration:none ;

    background: url("http://www.exploitation.org.uk/Test/Images/logo.gif");
    float:left;
    display: block;
    margin:0px ;
    padding:0px ;  
  }
  
  #imageBox {
    height:158px ;
    width: 655px ;

    background-color: #FFFFFF ;
    border-bottom: #FFE01C 11px solid ;
    float:left ;
    margin: 0px ;
    padding: 0px ;    
  }
  
      #imageBox1, #imageBox2, #imageBox3 {
         height:138px ;
         width: 184px ;

         border: #000 1px solid ;
         float:left ;
         margin: 10px 5px 0px 25px ;
         padding: 0px ;
       }
       
   #whiteBox {
     width : 655px ;
     
     background : rgb(255,255,255) ;
     float : right ;
   }
   
       #mainPicture, #gettingThere {
         width: 200px ;

         background-color:#FFFFFF ;
         border: #000 1px solid ;
         float:left ;
         margin: 10px ;
         padding:0px ;       
       }
       
       #mainText, #climate {
         width : 420px ;
       
         float : left ;
         border : 1px rgb(0,0,0) solid ;
         margin : 10px 0px ;
         padding : 0px
       }
       
       #health {
         clear : both ;
         border : 1px rgb(0,0,0) solid ;
         margin : 10px 
       }

   
   #col1 {
     width : 133px ;
     
     color : rgb(255,255,255) ;
     text-align : center ;
     
     float : left
   }
   
       #col1 p { font-weight :  bold }
   
       #col1 ul {
         line-height : 2em ;
         list-style-type : none ;

         margin : 0px ;
         padding : 0px
       }
       
       #col1 a { color : #FFEF10 ; text-decoration : none }
   
   #base {
     text-align : center ;
     
     background : rgb(255,255,255) ;
     border-top: #FFE01C 10px solid ;
     clear : both ;
     padding-bottom : 20px
   }
   
       #base p { font-weight : bold ; margin : .5em 0em 0em 0em }

Basically I removed the "height" for all elements except where they have to be specified. Cheers. Laughing out loud

I am a Man of Truth.
I am a Free Human Person.
I am a Peacemaker.

http://www.abcruz.com

dthomas31uk
Offline
Enthusiast
Last seen: 12 years 37 weeks ago
Joined: 2004-11-29
Posts: 144
Points: 0

Please can you advise

Ab............. Thank you very much for doing that for me.........much appreciated.

You are a star

Once again

Cheers Smile :) Smile