Afficher un message
Vieux 04/09/2007, 16h19   #3
John Hosking
Aucun Avatar
 
Messages: n/a
Hébergeur:
Par défaut Re: What do you think about my new site?

Lirdn wrote:
> I just put on line my new site called www.woman-2-woman.net I would
> like to know what do you think about my site any ideas about
> imporvements


Caveat: I'm male, and so apparently not in your target demographic.

The offset of the Woman-2-Woman.net heading relative to the rest of the
page looks broken, like something was supposed to be centered but isn't.
And yes, kill the blinking.

That heading is coded as <div id='navigation'>Woman-2-Woman.net</div>,
when it appears much more like it ought to be <h1>Woman-2-Woman.net</h1>
or so. And what's "navigation" about that element?

It's always a real turn-off to me when a website makes ads a prominent
feature. Can't you move the Ads by Google down somewhere (if not off the
page completely ;-) )? And why would I ask to view ads about anything?

The colors don't do it for me, but mbstevens appears to like them...

Photos would be a nice touch, if they were relevant to your page. Even
simple BG images might , e.g., a stethoscope and thermometer behind
"Women's Health", some luggage or an airplane behind "Women Travellers".

Do get rid of all the extraneous space at the bottom of the page and at
the bottom of the table. It's bad enough you have a goofy little
sitemeter thingy, but it's easier to ignore if it's at the bottom of the
natural page than if I see a vertical scrollbar and then scroll scroll
scroll down to see what's there and it's... CRAP!

You're using CSS embedded and external stylesheets already, so there's
no excuse for this to generate vertical spacing:
<p>&nbsp;</p>
<p>&nbsp;</p>

You're using an external stylesheet, so why the embedded styles?

You don't need to "comment out" styling, as in
<style type="text/css">
<!--
body {
background-image: url();

If you've got a background image for the page, bring it on. Otherwise,
delete that line.

Your HTML does not validate. Go to http://validator.w3.org/ and see.

If you're not using XHTML for a reason (odds are you aren't) then go
with HTML 4.01 strict. Change your doctype to
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN"
"http://www.w3.org/TR/html4/strict.dtd">

Using px for text, as in font-size: 24px; keeps users of IE <6 from
changing the text size. And yes, your a:hover size should be no
different from the unhovered size.

You need to get some proofreading . Violence against women is
horrible (sincerely, it sickens me), so don't make things worse by
spelling "torture" as "Torcher". ;-) You might also want to indicate
that your links go offsite, not to your own content.

Your markup contains

<p><a href="file:///C|/Documents and Settings/Lisa_Avi_Ilanit/My
Documents/life_site/Woman/Violence_Mutilation_Against_Women/Violence_Against_Women.html"></a></p>

which is an empty link (meaning visitors will almost certainly never
find it) to a file on your local PC (meaning even if they could click
the link, visitors would never get the file).

Your CSS includes a bunch of position:absolute; and z-index and #Layer7
junk, which is even more ridiculous since the pages are (mis)using
tables for layout. In general, your code (markup and CSS) is way too
complicated for the simple effect you're aiming for. If your page is to
get any kind of traffic at all, the maintenance of the site as it's now
constructed will kill it (and you). I suggest you start over, with
simple, basic markup, *then* styled for presentation.

And for critiques, you can post in alt.html.critique.

HTH. GL.

--
John
Pondering the value of the UIP: http://improve-usenet.org/
  Réponse avec citation
 
Page generated in 0,07339 seconds with 9 queries