I was hired to do this website in about a week and the people that hired me love it. I feel that there are a few things that can definitely be fixed (and a few that can't). I'm not in love with the font, but it's what the people that paid me wanted to have. But any suggestions, tips are welcomed. You can really tear me apart, I'd really appreciate it.

 

I know I have some compatibility problems, but let me know.

 

It's built using a custom WordPress theme I made, for a simple beauty salon, that really embraces social media (They have a lot of friends and followers)

 

I thank you guys in advance.

 

 

P.S. It's not showing up too high up on Google. Any tip's on that? Should I include hidden text behind the logo stating the title?

 

http://bachisalon.com

I apologise in advance for perhaps coming across blunt, I'm honestly not trying to be an asshole. Simply that it's 1AM and rather than spending effort thinking about how I might come across I'm just going to type what I think.

 

• The fixed position twitter link gets hidden behind content at my resolution, which isn't all that small (1440x900)
  
• Unless I am on the gallery page, where it is infront of the content
  
• Also, it says 'latest tweet' but shows me a button to follow the account.
  
• Text is poor. Just making it bigger and in caps doesn't do you any favours.
  
• Shadows are too harsh
  
• Footer is too close to content and too dark, making it look like you've forgotten to delete or style it.
  
• Buttons that aren't buttons but boxes with only text clickable make me want to kill babies
  
• To little padding on content pages
  
• Contact information just looks like a wall of text. Make it clear where the label ends and the actual information begins.
  
• Social buttons don't match the overall style at-all
  
• Why do the gallery images have grey borders and nothing else does?
  
• Header text is far too big and has far too little padding.
  

 

Actually most of these points can be summed up by saying that the site is generally failing one of the three things I consider most important in good web design,

 

Relative sizing. Text (Or any object really) should be appropriately sized and padded to not overwhelm either the box that contains it or the next piece of equally important information. Header text doesn't have to be double the size of body text to stand out, just bolding it can be enough.

 

The other two, if you are interested, are Subtlety and Minimalism. It's not terrible on those fronts. I mentioned the Shadows and there doesn't seem to be a great deal of useless information or design features so you are fine on the Minimalism front.

Edited July 7, 201214 yr by Glowbridge

Thanks, I need someone that can give me effective feedback. I will take it into consideration and have already started to updated the site. Thanks

 

 

EDIT:

 

You said this:

 

The fixed position twitter link gets hidden behind content at my resolution, which isn't all that small (1440x900)

Unless I am on the gallery page, where it is infront of the content

 

Do you have any advice on how to handle this?

 

EDIT 2:

 

I just tested it on 1440 x 900 and it looks fine. Thoughts?

Edited July 7, 201214 yr by EGamerHDK

Don't make the twitter link fixed with the background. Instead, you can float it beside the content.

 

Make the twitter link as an object, stack it before the content, float left for both twitter link and content.

 

There are other methods though.

Don't make the twitter link fixed with the background. Instead, you can float it beside the content.

 

Make the twitter link as an object, stack it before the content, float left for both twitter link and content.

 

There are other methods though.

 

It's fixed to the lower left "leaf" image. I couldn't think up of any other way to do it. Possibly making it dissapear with screen resolutions lower than 1200px in width altogether.

In my opinion I think that you should make the header text different from the paragraph text. Maybe change the text color on the header to make it contrast better on the page so it doesn't look too plain. That's just me though!

In my opinion I think that you should make the header text different from the paragraph text. Maybe change the text color on the header to make it contrast better on the page so it doesn't look too plain. That's just me though!

 

Thank you. That actually helps a lot.

I am glad I helped! This is my first review...>_>

I mean, don't fixed it with the leaf. Like.

 

<body>

<!-- You make your twitter widget as a new object, and stack it on top of your content -->

<div id="twitter"> your twitter stuff </div>

<div id="content"> your content </div>

</body>

<style>

/* You float both object, so the twitter link will be beside content, 
and push the twitter link down to any pixel you want with margin. 
Position fixed so it will not go anywhere */

#twitter {
float: left;
position: fixed;
margin-top: any number you want;
}

#content {
float: left;
}

</style>

Edited July 8, 201214 yr by marcelc

Hmm... Yeah, it looks like I might have to rethink how it works on different sizes.