Sensual lady

Firstly thank you to everyone who posted on my last effort.
I have done a bit of updating and I now need more feedback I have put two temp pages online and would like feedback on the layout and design.
The centre images are linked to the other page I would like you to look at, any feedback good or bad would be appreciated.
thanks in advance

Pat.

IT CONTAINS MODERATE ADULT MATERIAL

Ok well done for using mostly CSS.

However attributes like:

HTML: Select all

align="center"

should also be in CSS, my method is

Code: Select all

classname {
      margin: 0px auto;
}

Where class name is the class of the object you are trying to center.
Also for search engines and text browsers your site isn't friendly at all.

For example your header graphic would show as alt text in a text browser when really, you want it to be a <h1> or <h2>.

To do this you want to add a background image to an h1 tag using CSS. For example your Sensual Lady header would look like this:

html

HTML: Select all

<h1><span class="nodisplay">Sensual Lady - Your outlet for sophisticated lingerie</span><h1>

css

Code: Select all

.nodisplay
       {    display: none
       }

h1    {   background: url(images/sensual.gif) no-repeat #fff;
            width: 364px;
            height: 87px;
            margin: 0px auto;
       }

Let me talk you through this...

The XHTML
Well its an h1 tag, I've added "Your outlet for sophisticated lingerie" to your title because an h1 tag should be keyword rich. By adding this you are adding keywords. By all means replace this with your own keyword rich tag line. The text part is wrapped in a <span> tag. This will become clear when I explain the CSS.

The CSS
The nodisplay class, uses display: none; to hide the text from any browser that supports CSS. If the browser does not support css it will show the text we wrapped in the span tag earlier.

The h1 tag. We give the h1 a background of your image, we specify the hight and the width to match the image so that it displays the whole image, and we use margin:0px auto; to center the h1 tag.

Done. Now it is search engine friendly, friendly to all users of your site even text browsers, and looks great ot people with CSS enabled browsers

Any questions please ask. I think I've explained it ok.

As i said all your alignment should be done using CSS.

Hope this helps.

Mike

Thanks for that, as you may have guessed im learning,
I am going to try and implement what you have said, ill let you know how i get on.

cheers.

I'm liking it more now! I like the background and the fact that you dropped the white background. I also like that you got rid of lots of the clutter that was originally there.

I would suggest you add a hover instance on all your links. It's just nice and expected in my opinion.

I like the new header much better! On the view basket graphic the heart and lady seem a little squished.. Any chance you could resolve that?

I'd be very curious what Deigo would think of all the keywords at the bottom of the homepage. I think that is borderline if not crossing the line of keyword stuffing. But I might be wrong.

Overall.. much improved.

I was talking to my wife about the bottom paragraph, we agreed, something has to change.
the logo has been designed with ideas and help from youself (thanks), I agree the lady on the heart needs a bit of work and will possibly change, as for the hover should it be in the same colour(alter size) or a different colour altogether.

pat

I am quite fond of a dotted bottom border

Code: Select all

a {
border-bottom: 1px dotted #xxx; 
}

for links and then when they are hovered that border goes solid

Code: Select all

a:hover {
border-bottom: 1px solid #xxx;
}

That might work. It would look quite nice in pink actually.

Pete.

I had a little play around with that.
I couldnt get all links to behave the same. I will have to check.
but it looked good.
thanks

Hey Pat:
I think the logo is very good.. you don't need to change that. Just the view cart button... the perspective is off.. it's too tall versus it's width.
Glad I could be of inspiration... I like your silhouette better!

I think you have enough going on with the site and at the risk of falling back into the "old" look of busy, I'd just do a simple hover over color change. Possibly #333333 or #666666.

Glad you think the bottom paragraph needs to change. You are sort of all over the place in that with keywords. Google is going to frown upon all those tricks. Just be careful ok..

Once again thanks to all for your help.
I have changed a few things already and am working on the CSS suggested by Mike.

My site will raise its ugly head again when it is complete.

Cheers all

BTW I am trying to add rep points to those that helped me but it comes up with error on page

rep points system is broken atm.