Adam Posted May 9, 2009 Share Posted May 9, 2009 Here's a site I created quite a while ago, I've posted before but just curious to find out what the general opnion is? It's under going some maintenance so now's a good time if anyone has any ideas to improve it a bit. http://minimerc.co.uk Thanks Adam Quote Link to comment https://forums.phpfreaks.com/topic/157468-minimerccouk/ Share on other sites More sharing options...
Axeia Posted May 9, 2009 Share Posted May 9, 2009 I'd make an unordered list out of the links at the bottom. I'd use a ul for the upcoming events as well... mmh a <dl><dt><dd> might even be better. http://minimerc.co.uk/browse/ yet another list there Perhaps some kind of big orange border (5px or so) in the main menu below the current page. h1/h2 are only used on the frontpage? Think http://minimerc.co.uk/about.php lacking alt text was just a hasty "lets put some content up" thing? Form on http://minimerc.co.uk/contact.php can use some work.. like not using javascript to submit it just cause you want to to style the button and using labels.. and a proper table. Reworked version below (though the added th's probably mess up your layout slightly and need some CSS work. <table width="300" style="margin: 0px auto;"> <tbody> <col style="width: 30%;" /> <col style="width: 70%;" /> <tr> <th><label for="name">Your Name:</label></th> <td><input type="text" value="" class="contact_text" id="name" name="name"/></td> </tr> <tr> <th><label for="email">Your Email:</label></th> <td><input type="text" value="" class="contact_text" id="email" name="email"/></td> </tr> <tr> <th><label for="email_confirm">Confirm Email:</label></th> <td><input type="text" value="" class="contact_text" id="email_confirm" name="email_confirm"/></td> </tr> <tr> <th><label for="subject">Subject:</label></th> <td> <select class="contact_text" id="subject" name="subject"> <option value="General">General</option> <option value="Technical">Technical</option> <option value="Orders">Orders</option> </select><input type="hidden" value="true" name="valid"/> </td> </tr> <tr> <th valign="top"><label for="message">Message:</label></th> <td><textarea cols="23" rows="5" class="contact_text" id="message" name="message"/></td> </tr> <tr> <th><label for="email_secure"><img src="images/captcha.php"/></label></th> <td>Please enter the security number:<br/><input type="text" class="contact_text" id="email_secure" name="email_secure"/></td> </tr> <tr> <td style="text-align: right; padding-top: 10px;" colspan="2"> <input type="submit" value="Next" style="width: 70px; height: 25px; background: #cecece url(/images/nextbutton.gif); color: transparent; font-size: 0.1px; display: block; margin: 0 auto;" /> </td> </tr> </tbody> Tables are a sort of inbetween case for using tables.. I'm not convinced they should be used for them, however I'm not convinced they should not be used for them either. But if you do then do it properly. For something like that I'd simply float the form controls next to the static width labels. Overal it looks quite nice, strangely enough even though still having sideroom on 800x600 it doesn't feel too cramped. If you're going to add a lot of content you might want to consider dropping 800x600 and go for a 1024x768 minimum Quote Link to comment https://forums.phpfreaks.com/topic/157468-minimerccouk/#findComment-830228 Share on other sites More sharing options...
gregor171 Posted May 16, 2009 Share Posted May 16, 2009 NIce and simple. Compliments for coding it. Some SEO optimization should be added. - liks with titles - h tags A bit of reworking on html and css. Using DIV is ok. I have a strange position of shopping chart from linux and also * Contact » Updates navigation is not in line. I thing that titles of models and brands are a bit too little. Quote Link to comment https://forums.phpfreaks.com/topic/157468-minimerccouk/#findComment-835383 Share on other sites More sharing options...
Masna Posted May 16, 2009 Share Posted May 16, 2009 Redo the logo and the menu. The logo is very sketchy and has no flow. And the diagonal lines need to go. And, the menu is just plain boring (and the colors are rather bothersome). Quote Link to comment https://forums.phpfreaks.com/topic/157468-minimerccouk/#findComment-835519 Share on other sites More sharing options...
Adam Posted May 18, 2009 Author Share Posted May 18, 2009 Great, thanks for the feedback. Once I've completed some of the other upgrades I'm currently making I'll get to work on some of your points - especially the SEO stuff. And the diagonal lines need to go ...What's wrong with the diagonal lines by the way? Quote Link to comment https://forums.phpfreaks.com/topic/157468-minimerccouk/#findComment-836268 Share on other sites More sharing options...
Masna Posted May 18, 2009 Share Posted May 18, 2009 ...What's wrong with the diagonal lines by the way? I've found that when presented in a situation like this, it's always best to ask yourself... What's not wrong with diagonal lines? Quote Link to comment https://forums.phpfreaks.com/topic/157468-minimerccouk/#findComment-836271 Share on other sites More sharing options...
Adam Posted May 18, 2009 Author Share Posted May 18, 2009 I don't know? I like them... Quote Link to comment https://forums.phpfreaks.com/topic/157468-minimerccouk/#findComment-836272 Share on other sites More sharing options...
Masna Posted May 18, 2009 Share Posted May 18, 2009 I don't know? I like them... Actually, no, you don't. Quote Link to comment https://forums.phpfreaks.com/topic/157468-minimerccouk/#findComment-836273 Share on other sites More sharing options...
Axeia Posted May 18, 2009 Share Posted May 18, 2009 On some monitors small diagonal lines look like they're moving all the time.. so yeah avoiding them (or keeping them at ~5px+ sizes) is usually advisable. They're fine on my monitor(s) though. Quote Link to comment https://forums.phpfreaks.com/topic/157468-minimerccouk/#findComment-836308 Share on other sites More sharing options...
DarkSuperHero Posted May 18, 2009 Share Posted May 18, 2009 Im not much of a fan of the long stretch of content running along the right handside....Makes scrolling a have to....and well theres not enough content in the main content area to make me want to scroll...chances are I would have never made it to bottom of page.... http://minimerc.co.uk/browse <- on this page how about making the box my mouse is over change bg color ? indicating which box im hovering over? just a couple thoughts... Cheers! Quote Link to comment https://forums.phpfreaks.com/topic/157468-minimerccouk/#findComment-836652 Share on other sites More sharing options...
Adam Posted May 18, 2009 Author Share Posted May 18, 2009 Yeah that is actually one of the changes I'm planning to make, re-do the side bar and homepage. The roll over effect on the browse page is a good idea as well. Cheers! Quote Link to comment https://forums.phpfreaks.com/topic/157468-minimerccouk/#findComment-836655 Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.