Davidx000000

Thanks for the input and giving me something to work on.

Hi,
 
In regards to your post I think looking at your website with fresh eyes the first impression I got was a paypal kind of feel in terms of the color scheme, buttons, fonts etc... which I think kind of gives it a feel of familiarity (but that's just my opinion)
 
I like the fact you got today's date in the corner it gives the impression that its fresh and the content is constantly maintained which is something I intend to do for my website www.guildbathrooms.com.
 
Personally I like the layout I think the content flows well and I don't feel overloaded with information but my only dislike would be the light greeny/blue color used for the border on the navigation section I would personally use a bolder color but thats just my personal choice.
 
Hope this helps
 
David

Thanks BrandableWeb to be honest I had noticed both them points myself, thanks for taking the time to look and for the helpfull feedback

The color scheme is perfect, fits the bathroom concept perfectly. I personally like the scrolling bar, gives you the ability to see lots of examples without having to leave the home page.
 
Two things that I wll comment on. First, the text in the menu bar seems too small or at least not apparent enough. The Town names overshadow them. I would make sure they stand out more so your clients know to click on them.
 
Second ( a very silly thing) the logo at the top seems like it could move over just a teeny bit to be better centered. Again not that big a deal and it could just be me.

Thanks for the feedback , its a first impression i really need as i have stared at this for the last month and lost all perspective so any input is helpful.
 
Thanks for taking the time to look I appreciate the comment

I think it's a big improvement on the old site.
 
I like the colour scheme.
 
I think the text may be a little small.
 
At first sight I thought the place names across the top were menu items and wanted to click on them but I really don't know whether that's a problem.
 
I think you could make the text a little smaller in the box that says "Please enter a contact numbe" so you can fit the whole of "number" in. Or make the box wider, of course.
 
For some reason, I don't like the scrolling slideshow. It keeps drawing my eye and distracting me when I'm trying to look at the rest of the page.
 
Take all this with a pinch of salt, because I'm not a designer.

1. Although setting the form action to nothing will theoretically work I have found that not all servers will accept this. Setting the action to:
 
 
<?php echo htmlentities($_SERVER['PHP_SELF']); ?> or
 
 
<?php echo htmlspecialchars($_SERVER['PHP_SELF']); ?> Will solve the issue.
 
Note: Most modern servers will default to one of these methods automatically if the action is set to
 
<?php echo $_SERVER['PHP_SELF']; ?>  
2. You are not setting any headers to the mail function.
 
Many servers will not allow a mail to send if it is not from an email on that domain:
 
You have a $from variable:
$from = "mail.guildbathrooms.com"; $headers = "From: $from"; Try
 
 
<?php if (isset($_POST['callBackSubmit'])) { $to = "xxxxxxx@livexxxx.co.uk"; $subject = "Call me back, Guild Bathrooms"; $message = $_POST['callBackField']; $from = "mail.guildbathrooms.com"; $headers = "From: $from"; mail($to,$subject,$message;$headers); echo "Mail Sent."; } ?>

Hey all got a bit of a complicated question that I would appreciate any help to.
 
Im trying to build a search system where the user ticks a choice of checkboxes to choose what content gets displayed once the form has been submitted.
 
I realise you would give each checkbox a value then post that value over but im a little bit unsure how to go about the php / mysqli.
 
I know im being a little vague but any help would be appreciated.
 
Thanks for reading

if you want it to echo in a certain area then use AJAX not only it is dynamic it also sets where this echo procedure will output for example is you wanted it to output in the center you just put a div id there and it will simply find that id and put the code there. I used a a guy on youtube which had tutorials of how to use AJAX.
 
heres the link if your interested, I got to number ten and then I understood it.
 

You don't provide a name for each <option> you only provide a name for the select tag like below. The option values will be pulled as needed
 
<select class="indexSearchLocationList" name="locationList"> <option value="a" >Anywhere in London </option> <option value="b" >Barking and Dagenham </option> <option value="c">Barnet </option>

$(function(){ // This is just a jQuery shorthand for: onPage load, do everything inside here...

// This adds an onchange event handler to the select element
// with the data-role value of 'initial-selector'
$("select[data-role='initial-selector']").on('change', function(){
// Gets the selected index of the initial-selector select element
// the selected index is an integer value 0 -> number of options-1
var index = this.selectedIndex;
// Hide all of the other selects using a simple js-hide class
// this line hides all select elements with that class
$("select.hide-js").hide();
// Now show the appropriate select element using the index
// we got previously – uses the data-role property again
// If the index is 1, then this line chooses selector-1
// if the index is 2, then this line chooses selector-2, and so on...
$("select[data-role='selector-" + index + "']").show();
});

// onLoad, just hide the select boxes initially – this is better than
// CSS as it allows for non-js users to still use the form at least
$("select.hide-js").hide();

}); // End of the initial function (jQuery shorthand onPage load)

I'm impressed - if this is your first attempt at a web project then you've taken a great stride in the right direction. You've used conventional elements (divs, spans etc), declared the doctype and complied with it, CSS looks good too, nothing to really 'criticise' as it were, so I'll move onto design.
 
I would add a background to the bottom footer containing all the links (which, again, is a good addition) and there's a huge space at the bottom of the web page.
 
Under the 'Areas we cover' drop-down tab, do you plan to make those links to the individual areas? If not, it would be unusual to have this kind of information in a navigation drop-down tab.
 
On a personal note, I would add a 'margin-top' to the #wrap div, to separate it from the #holder1 block, since there's a gap between #pictureholder and the grey #textholder bar.
 
Anything else I would say would be focussed on the graphical side of things, but since you haven't claimed to be proficient in graphics design then it wouldn't be fair to critique it. That being said, it's a fantastic effort graphically speaking, on the whole it's generally a pleasing website to view.
 
Keep it up.

you could use sub domains ie kitchen.businessname.com, bathroom.businessname.com etc... then with your main website (www.businessname.com) have that as a landing page so the user can navigate to the required section