Jump to content

Recommended Posts

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

 

Link to comment
https://forums.phpfreaks.com/topic/157468-minimerccouk/#findComment-830228
Share on other sites

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.

 

Link to comment
https://forums.phpfreaks.com/topic/157468-minimerccouk/#findComment-835383
Share on other sites

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?

Link to comment
https://forums.phpfreaks.com/topic/157468-minimerccouk/#findComment-836268
Share on other sites

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!

Link to comment
https://forums.phpfreaks.com/topic/157468-minimerccouk/#findComment-836652
Share on other sites

This thread is more than a year old. Please don't revive it unless you have something important to add.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.