Jump to content

please review


mabell

Recommended Posts

Hi - I am a new member of this forum, but I would very much like to hear what other members have to say about my site.

My site

It is a fairly new site, I keep adding to it through the blog and also new pages as I find them needed.

I am using php includes for headers and footers.

Known issues:


  1.  
  2. The portfolio page has a dead link to one of the sites - that issue will be settled soon.
  3. The Chiropractor's site has been redesigned recently and I have to get a new screenshot up.
  4. Not all the pages are available in both languages - the translation is getting done ASAP but it does take time.

 

If there are other issues I'd like to hear about them - I do not want to produce mediocre work, and am looking forward to constructive criticism.

 

Thanks,

AlphaMare

Link to comment
Share on other sites

Hi and welcome.

 

First impression is that it's quite white thus makes it a bit boring. Somehow I think you can bring it to life by using some light-grey gradient background(s).

 

However, I like your logo and the name.

 

The code is pretty good but you don't have to do this:

<span style="font-weight: bold">Reliability and speed of access</span>

 

...when you can do this:

 

<strong>Reliability and speed of access</strong>

 

This...

<p> </p>

<p> </p>

<p> </p>

 

 

...is not a good practice to space content.

 

 

I am sure there is more but you are on the right track.

Link to comment
Share on other sites

Hi and welcome.

 

First impression is that it's quite white thus makes it a bit boring. Somehow I think you can bring it to life by using some light-grey gradient background(s).

 

However, I like your logo and the name.

 

The code is pretty good but you don't have to do this:

<span style="font-weight: bold">Reliability and speed of access</span>

 

...when you can do this:

 

<strong>Reliability and speed of access</strong>

 

This...

<p> </p>

<p> </p>

<p> </p>

 

 

...is not a good practice to space content.

 

 

I am sure there is more but you are on the right track.

 

 

Hi - I am sorry you don't like my white background - I like it a lot and find it makes the rest stand out. But I'll take it under consideration. If I were to go to a gradient it would probably be a colour, not grey...I do tend to rainbows and bright colours, maybe I have erred too far in the direction of restraint! :rolleyes:

 

I am not sure where you got the <span style="font-weight: bold">Reliability and speed of access</span> because I don't have that text anywhere on my site.

 

If you are referring to the fact that I have

 

<span class="strong">DigitalInsite</span> and <span class="strong1">“good enough”</span> and so on, that is because they are actual classes in the CSS, and define more properties than just the weight, as follows:

 

#tagline h2 .strong{ color: #333; font-family: "Century Gothic", Verdana, Geneva, sans-serif; font-size:22px; font-weight: bold; font-variant: small-caps;}

 

#tagline h2 .strong1{ color: #333; font-family: "Century Gothic", Verdana, Geneva, sans-serif; font-weight: bold;}

 

#tagline h2 .strong2{ color: #333; font: oblique bold small-caps "Century Gothic", Verdana, Geneva, sans-serif;}

 

I guess I should clean up my CSS and decide whether to go in shorthand or not! ;)

 

You are quite right -

<p> </p>

<p> </p>

<p> </p>

used in a chunk like that, is not good practice - but as a placeholder for later addition of text I find it useful. In this case I did not add the text and should have taken it out.

 

I'm glad you like my logo - I do too, even tough my marketing advisor tells me there are too many colors...LOL

 

Any comments on the general content of the site, the layout, the pages etc?

 

Thanks,

AlphaMare.

Link to comment
Share on other sites

I am not sure where you got the <span style="font-weight: bold">Reliability and speed of access</span> because I don't have that text anywhere on my site.

I found it on your Faqs page.

 

Any comments on the general content of the site, the layout, the pages etc?

You need paragraph spacing.

Edited by newseed
Link to comment
Share on other sites

  • 2 months later...

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Loading...
×
×
  • Create New...