Welcome Guest!
Please login
If you do not have an account yet on The Web Squeeze forums, please Register! It’s FREE and there are many benefits:
- Receive Fast Advice
- Learn Programming Languages
- Get Professional Website Reviews
- Quick Troubleshooting Assistance
|
|
Be Harsh
This is a discussion on Be Harsh, within the Website Critique & Squeeze section. This forum and the thread "Be Harsh" are both part of the The Squeeze Center category.
![]() ![]() |
Jun 10 2008, 11:40 AM
Post
#1
|
|
![]() Rapid Squeezer ![]() ![]() ![]() ![]() Group: Advisors Posts: 427 Joined: 14-February 08 From: Stockport Member No.: 75 |
Right here we go (nervous shaking of the hands as i type :S)
Site is for an electrician based in Munich, I have run out of ideas so want to see what you all think. Site is valid xHTML strict apart from the contact page due to the contact form. This also doesn't work at the moment, trying to figure that out. The current page highlight doesn't work as well, if anyone wants to take a look at my code it would be much appreciated Got a random header displayed through php, is this idea with the quote good or irritating? Be as honest as you can please Craig http://www.craigmoran.co.uk/pg -------------------- |
|
|
Jun 10 2008, 11:43 AM
Post
#2
|
|
![]() Rapid Squeezer ![]() ![]() ![]() ![]() Group: Advisors Posts: 427 Joined: 14-February 08 From: Stockport Member No.: 75 |
Also only done one version of the site, the flag buttons are not working as of yet
-------------------- |
|
|
Jun 10 2008, 12:26 PM
Post
#3
|
|
![]() Don deluzione ![]() ![]() ![]() ![]() ![]() Group: Team Leaders Posts: 740 Joined: 13-February 08 From: 10km from nowhere and just south of nothing Member No.: 7 |
I'd put some spark in there.. (get it? spark
I mean, apart from reading "Electro", how do I know what he does? I'd put in an electric socket, a power tower or a windmill.. I don't mind the header.. -------------------- |
|
|
Jun 10 2008, 12:46 PM
Post
#4
|
|
![]() Rapid Squeezer ![]() ![]() ![]() ![]() Group: Advisors Posts: 427 Joined: 14-February 08 From: Stockport Member No.: 75 |
I'd put some spark in there.. (get it? spark I mean, apart from reading "Electro", how do I know what he does? I'd put in an electric socket, a power tower or a windmill.. Haha very good, and all that's going to be on the home page, its for my uncle, not a proper client and he's writing the homepage description, it needs to be in german so should be fun HTMLing all the umlouts lol. I don't mind the header.. Do you think it needs livening up a bit, or leave it how it is? Cheers for the feedback -------------------- |
|
|
Jun 10 2008, 03:11 PM
Post
#5
|
|
![]() Squeezing ![]() ![]() ![]() Group: Members Posts: 67 Joined: 14-February 08 Member No.: 86 |
Your current page highlight doesn't work because #nav-main a:link has a higher CSS specificity than #active. You should change your css to #nav-main a#active or the #nav-main id to a class.
|
|
|
Jun 10 2008, 03:38 PM
Post
#6
|
|
![]() Rapid Squeezer ![]() ![]() ![]() ![]() Group: Advisors Posts: 427 Joined: 14-February 08 From: Stockport Member No.: 75 |
Your current page highlight doesn't work because #nav-main a:link has a higher CSS specificity than #active. You should change your css to #nav-main a#active or the #nav-main id to a class. -------------------- |
|
|
Jun 10 2008, 04:32 PM
Post
#7
|
|
![]() Squeezing ![]() ![]() ![]() Group: Members Posts: 69 Joined: 14-February 08 From: Dublin, Ireland Member No.: 107 |
Hi Craig,
I am an electrician as well. I used to work for a company and I did their website for them for free. They were s$*th€&ds, and thanks be to god I don't work for them anymore. Anyway, I did the following site for them. http://www.microelec.ie It was when I was starting out and is totally tables no css (so nobody needs to give any critique). I got no thanks for it either. If it gives you some idea's, well then some good has come out of it. Only for content that is!! I dont think anyone will be copying the design!!!!!! This post has been edited by thewal: Jun 10 2008, 04:35 PM -------------------- Finally squeeezin!!!!
|
|
|
Jun 11 2008, 02:21 AM
Post
#8
|
|
![]() Rapid Squeezer ![]() ![]() ![]() ![]() Group: Advisors Posts: 427 Joined: 14-February 08 From: Stockport Member No.: 75 |
Thanks for this, I'll show my uncle see if he can get any ideas
Anyone else any thoughts about the site? -------------------- |
|
|
Jun 11 2008, 02:54 AM
Post
#9
|
|
|
Master of the Universe ![]() ![]() ![]() ![]() ![]() Group: Mentor Posts: 984 Joined: 15-February 08 From: London, England Member No.: 141 |
Here is the way I see it.
+ It's Clean + Contact details are clear and concise + The company logo is nicely positioned + Nice Layout / Use of Colors - The 'contact page' link in the left column doesn't look right. Perhaps replace it with an image - The flags just look... Meh. Maybe have 'Languages: Flag Flag' - Contact form is pretty much unstyled - What the hell are the favorite insect's about? - No text whatsoever on why I should contact you. The site is OK. I think you have gone a bit astray with the 'favorite insects'. It is one thing to stop spam and another to impact the user submitting the form. You should look at other methods of spam protection. |
|
|
Jun 11 2008, 05:42 AM
Post
#10
|
|
|
Rapid Squeezer ![]() ![]() ![]() ![]() Group: Mentor Posts: 414 Joined: 15-February 08 From: UK Member No.: 143 |
I assume the "favourite insects" thing is temporary.
I'd like to see at least one image to suggest what the site is about -- perhaps even a photo of Mr. Gibson working. The logo doesn't stand out at all, although that may be outside your remit (I know, I'm in no position to criticise overly-plain logos The flags are too small, or at least need a white background (they look bad on the dark red of the navbar). You have some errors in your CSS, including two parse errors. The error on line 36 is unimportant, and should disappear when validating against CSS 3. For the contact form errors, I believe your main problem is that you've allowed inline elements without a block container. <form> is weird like this; I've never bothered to find out why. Simply put a <div> inside your <form>, like this: CODE <form> <div> ...all your other code... </div> </form> That should work. |
|
|
Jun 11 2008, 07:17 AM
Post
#11
|
|
![]() Rapid Squeezer ![]() ![]() ![]() ![]() Group: Advisors Posts: 427 Joined: 14-February 08 From: Stockport Member No.: 75 |
Here is the way I see it. + It's Clean + Contact details are clear and concise + The company logo is nicely positioned + Nice Layout / Use of Colors - The 'contact page' link in the left column doesn't look right. Perhaps replace it with an image - The flags just look... Meh. Maybe have 'Languages: Flag Flag' - Contact form is pretty much unstyled - What the hell are the favorite insect's about? - No text whatsoever on why I should contact you. The site is OK. I think you have gone a bit astray with the 'favorite insects'. It is one thing to stop spam and another to impact the user submitting the form. You should look at other methods of spam protection. Thanks for the feedback The contact form is the example one Luke did lol, that why the insects are there Thanks for the comments -------------------- |
|
|
Jun 11 2008, 07:22 AM
Post
#12
|
|
![]() Rapid Squeezer ![]() ![]() ![]() ![]() Group: Advisors Posts: 427 Joined: 14-February 08 From: Stockport Member No.: 75 |
I assume the "favourite insects" thing is temporary. I'd like to see at least one image to suggest what the site is about -- perhaps even a photo of Mr. Gibson working. The logo doesn't stand out at all, although that may be outside your remit (I know, I'm in no position to criticise overly-plain logos The flags are too small, or at least need a white background (they look bad on the dark red of the navbar). You have some errors in your CSS, including two parse errors. The error on line 36 is unimportant, and should disappear when validating against CSS 3. For the contact form errors, I believe your main problem is that you've allowed inline elements without a block container. <form> is weird like this; I've never bothered to find out why. Simply put a <div> inside your <form>, like this: CODE <form> <div> ...all your other code... </div> </form> That should work. I did what you said with the div's inside the form but now there are 20 errors, any more ideas? When i get on my laptop I'll add the background to the flags, i agree with the lack on contrast there. As for the logo, his actual logo is that but with a small lightbulb in place of the "I", so i will try and get that into the site. Thanks for the feedback, I'll have a look at what was suggested Oh yeh and if anyone has any ideas of the non validating page? -------------------- |
|
|
Jun 11 2008, 07:30 AM
Post
#13
|
|
|
Rapid Squeezer ![]() ![]() ![]() ![]() Group: Mentor Posts: 414 Joined: 15-February 08 From: UK Member No.: 143 |
I did what you said with the div's inside the form but now there are 20 errors, any more ideas? When i get on my laptop I'll add the background to the flags, i agree with the lack on contrast there. I like to call this effect an error cascade. All you need to do now is self-close your <input>s: CODE Wrong: <input .... >
Correct: <input .... /> |
|
|
Jun 11 2008, 07:48 AM
Post
#14
|
|
![]() Rapid Squeezer ![]() ![]() ![]() ![]() Group: Team Leaders Posts: 497 Joined: 13-February 08 From: Catching the squeezed drips downunder. Member No.: 13 |
QUOTE (unitedcraig) The contact form is the example one Luke did lol, that why the insects are there , I will also look into styling the contact form. Hi Craig, what problems are you having with the form? I can probably help you nut that out. QUOTE (mikehopely) I like to call this effect an error cascade. Fixing one error appears to create new errors, but actually they were there all along. All you need to do now is self-close your <input>s: This would be my fault due to a hastily typed form for example purposes however I have rendered myself liability free QUOTE (Rakuli in tutorial post) Things to note in this form action="emailMe.php" needs to be altered so it points to the location of the PHP script or can be changed to action="<?php echo $_SERVER['php_self']; ?>" if you want a recursive form ie. Points to the same page method="post" can be changed to method="get" if you wish This form is very simple without much thought for semantics, accessibility and sense As for the critique on the site my opinion is along the lines of those above. I would only know it is a site for an electrician after reading it ti find how, there's no "spark" that suggests this. **sorry, bad puns are no fun** Perhaps a header with some coils of wire or lines of insulated circuits would lend more of a suggestion to the pupose of the site (you can probably find some free/cheap royalty-free images that contain this). The layout is uncluttered which is good and the "call to action" is made readily available which is what an electrician would really care about; getting calls. -------------------- Bright Idea? -- Don't Let it disappear
|
|
|
Jun 11 2008, 08:19 AM
Post
#15
|
|
![]() Rapid Squeezer ![]() ![]() ![]() ![]() Group: Advisors Posts: 427 Joined: 14-February 08 From: Stockport Member No.: 75 |
Its worked lol
I will take a proper read through your ideas tomorrow afternoon, but due to a maths exam that i need to get an A in to get into Uni, i won't b able to change anything until then. Thanks everyone for your feedback i really appreciate it, and Luke I wil probably end up mithering you about how i can't get it to work lol Craig -------------------- |
|
|
Jun 11 2008, 02:13 PM
Post
#16
|
|
![]() Squeezing ![]() ![]() ![]() Group: Members Posts: 69 Joined: 14-February 08 From: Dublin, Ireland Member No.: 107 |
Rakuli said that you could get some royality free images. If you want me get pics for you on anything electrical, I can do that no hastle. Let me know what you need and I can e-mail to you
-------------------- Finally squeeezin!!!!
|
|
|
Jun 12 2008, 02:02 PM
Post
#17
|
|
![]() Rapid Squeezer ![]() ![]() ![]() ![]() Group: Members Posts: 314 Joined: 14-February 08 From: Lemon Land Member No.: 68 |
do you need js to go to the top of the page?
-------------------- |
|
|
Jun 12 2008, 04:36 PM
Post
#18
|
|
![]() Rapid Squeezer ![]() ![]() ![]() ![]() Group: Advisors Posts: 427 Joined: 14-February 08 From: Stockport Member No.: 75 |
do you need js to go to the top of the page? I did have an #top thing, but it messed up in internet explorer so i used the JS method. Is it that much of an issue, it's only to scroll up to the top of the page, nothing fancy like the nav bar. -------------------- |
|
|
If you found The Web Squeeze to be helpful, please donate so we can keep this site FREE, FRESH, and fortified with Web Design & Development info!
![]() ![]() |
1 User(s) are reading this topic (1 Guests and 0 Anonymous Users)
0 Members:







Jun 10 2008, 11:40 AM










