Register

If this is your first visit, please click the Sign Up now button to begin the process of creating your account so you can begin posting on our forums! The Sign Up process will only take up about a minute of two of your time.

Page 1 of 2 1 2 LastLast
Results 1 to 10 of 15
  1. #1
    Junior Member
    Join Date
    Jul 2009
    Posts
    6
    Member #
    19447

    Current Rating

    Visual Appeal:
    4.0 out of 5

    Accessibility:
    5.0 out of 5

    Website Speed:
    4.0 out of 5

    Compatibility:
    5.0 out of 5

    Overall Rating:
    3.0 out of 5

    Rate This Site


    Advertisement (login to hide this!)


    You must be logged in to view this site

    Hi all,

    I am new to this forum and im hoping I can get some good tips on my web site.

    I am a local pet sitter and my web site is very important for my business.

    http://www.nymuddypaws.com

    I am also having problems with my forms. When I fill them out from my computer, sometimes I get the e-mail sometimes I dont. So if you can please fill out my form so i can test it out.

    This is the php code from my contact e-mail form.

    Code:
    <?php
    $to = 'info@nymuddypaws.com';
    $subject = 'Muddy Paws New Customer Request';
    
    foreach($_POST['check']  as  $value)  {
    
    $check_msg .= "Checked: $value\n";
    
    } 
    
    $message = 'From: ' . $_REQUEST['contactname'] . "\n\n" . 
    'Type of Pet : ' . $check_msg . "\n\n" . 
    'Pet Name: ' . $_REQUEST['contactpetname'] . "\n\n" . 
    'Address : ' . $_REQUEST['contactaddress'] . "\n\n" .
    'Phone :' . $_REQUEST['contactphone'] . "\n\n" . 
    'Email : ' . $_REQUEST['contactemail'] . "\n\n" . 
    'Call Time : ' . $_REQUEST['contactcalltime'] . "\n\n" . 
    'Subject : ' . $_REQUEST['contactsubject'] . "\n\n" . 
    'Message : ' . $_REQUEST['contactmessage'] . "\n\n" ;
    $email = $_REQUEST['contactemail']; 
    $headers = 'From: ' . $email . "\r\n" .
                'Reply-To: ' . $email . "\r\n" .
              'X-Mailer: PHP/' . phpversion();
     
    mail ($to, $subject, $message, $headers);
    header("Location: thankyou.html");
    ?>

  2.  

  3. #2
    Junior Member
    Join Date
    Aug 2009
    Posts
    6
    Member #
    19607
    Hi, Dennis. The best constructive criticism I can give you is regarding the photos of the dogs. For whatever reason, you chose to make most of your photos short and wide. This has the effect of cutting off the top and bottom of the animal's face, while including large sections of nothing (grass, sidewalk, floor, etc.) to the left and right. I'm sure you were going for a particular style, but it's not really working. crop those photos better and let those wonderful doggy faces put a smile on your visitors' faces.

  4. #3
    Junior Member
    Join Date
    Aug 2009
    Posts
    6
    Member #
    19607
    Here is the code from one of my contact pages. I've modified it somewhat (to use your e-ddress, etc.) but there's still proprietary css references and an image or two that you'd have to tweak. Maybe put this on a hidden (non-linked) page of your site and enter the direct url to test it out, then modify to include the additional fields you need.

    Hope this helps...
    Code:
    <?php
    $qs =  $_SERVER['QUERY_STRING'];
    $parts = explode("=", $qs);
    $contact = $parts[1];
    $send2 = "info@nymuddypaws.com";
    $contact = "Muddy Paws Pet Care";
    $formfooter = "<h3><br><br>Or call us at 619-555-1518</h3>";
    echo'<img id="contactTop" class="ctr" alt="" src="img/yourimage.png" /><div class="inner2col2">';
    }
    $guest_email = $_POST['Email'];
    $msg_ = $_POST['Message'];
    $msg = str_replace("\r", '<br />', $msg_);
    //BEGIN CONFIRMATION MESSAGE
    $confirm = "
    <head><title>Thank You</title>
    <link rel='stylesheet' type='text/css' href='hth.css'>
    </head><body>
    <table width='50%' border='0' align='center'>
    <tr><td align='center'>
    <h3>Your message:</h3></td></tr>
    <tr><td align='left'><span class='strong'>&quot;$msg&quot;</span> has been sent to $contact.<br><br>
    A copy of the message has been sent to $guest_email.<br><br>
    Thank you for your correspondence. Someone will reply as soon as possible.</td></tr></table>";
    
    //END CONFIRMATION MESSAGE
    
    if($_POST['submitform'])
    {
        $Name = $_POST['Name'];
        $Email = $_POST['Email'];
        $msg1txt = $_POST['Message'];
        $required = $_POST['required'];
        $browser = $HTTP_USER_AGENT;
        $ip = $_SERVER['REMOTE_ADDR'];
    
    //BEGIN FORM VALIDATION
        $validate = explode(",",$required);
        while(list($check) = each($validate))
        {
            if(!$$validate[$check]) {
            $error .= "You did not fill in the $validate[$check] field.<br>";
            }
        }
        if ((!ereg(".+\@.+\..+", $Email)) || (!ereg("^[a-zA-Z0-9_@.-]+$", $Email))){
        $error .= "&quot;$Email&quot; is not a valid e-mail address.";
        }
        if($error)
        {
        echo '<head><title>Oops!</title><link rel="stylesheet" type="text/css" href="$cssfile"></head><body><table width="40%" border="0" align="center"><tr><td align="center"><h1><br><br></h1><h1><br><br></h1></td></tr><tr><td>';
        echo $error;
        echo '<br><a href="#" onClick="history.go(-1)">Please try again.</a>';
        echo '</td></tr></table>';
        }
        else
    //FORM VALIDATION PASSES
        {
    //BEGIN INCOMING MESSAGE
    $msg1subject = "Message from $Email re: $contact";
    $msg1txt = "
    Name: $Name:
    E-mail: $Email
    
    Message: $msg1txt
    
    -----------------------------
    Browser: $browser
    IP: $ip
    ";
    //END INCOMING MESSAGE
    
    //BEGIN OUTGOING MESSAGE (to message originator)
    $msg2subject = "Confirmed - your message to $contact";
    $msg2txt = "Your message was sent to $contact:
    -----------------------------
    From: $Name:
    E-mail: $Email
    
    Message: $msg1txt
    
    -----------------------------
    ";
    //END OUTGOING MESSAGE
    
    
    mail($send2,"$msg1subject","$msg1txt","From: $Name <$Email>");
    mail($Email,"$msg2subject","$msg2txt","From: <$send2>");
    echo "$confirm";
        }
    }
    else{
    echo '
    <head><title>Contact $contact</title>
    <link rel="stylesheet" type="text/css" href="hth.css"></head><body>
    <img class="'.$linkimg_class.'" src="'.$linkimg_src.'">
    <form name="contactform" action="'.$PHP_SELF.'" method="post">
    <table width="50%" border="0" align="center">
    <tr><td colspan="2" align="center">
    <h3>Email '.$contact.'</h3>
    <input type="hidden" name="required" value="Name,Email"></td></tr>
        <tr>
          <td colspan="2" align="center">
            <p></p>
          </td>
        </tr>
        <tr>
          <td valign="top" align="right"><b>Name:</b></td>
          <td valign="top"><input name="Name" size="30">
          </td>
        </tr>
        <tr>
          <td valign="top" align="right"><b>E-ddress:</b></td>
          <td valign="top">
            <input name="Email" size="30">
          </td>
        </tr>
        <tr>
          <td valign="top" align="right"><b>Message:</b></td>
          <td valign="top">
            <textarea name="Message" rows="10" cols="24"></textarea>
          </td>
        </tr>
        <tr>
          <td colspan="2" align="center"><br>
            <input type="submit" value="Send" name="submitform">
            <input type="reset" value="Reset" name="reset">
          </td>
        </tr>
      </table>
    </form>';
    echo $formfooter;
    }
    ?>
                    </div> <!-- inner -->
                    <div class="clearing">&nbsp;</div>
                </div> <!-- content -->
            </div> <!-- main -->
        </div> <!-- wrapper -->
    </body>
    </html>

  5. #4
    Member Trendog's Avatar
    Join Date
    Aug 2009
    Posts
    51
    Member #
    19603
    I agree with Donny Bahama on photos .
    I like the fonts and text size, I think the site is friendly and nice to visit.


  6. #5
    Junior Member
    Join Date
    Jul 2009
    Posts
    6
    Member #
    19447
    Thanks everyone for your input. Right now I am removing the wierd photo boxes, and I am making my front page more concise. I want to have a slideshow like hulu.com has on the front page. Except with recent photos of my dogs.

    Thanks again

  7. #6
    Senior Member aeroweb99's Avatar
    Join Date
    Feb 2008
    Location
    Port Huron, Michigan
    Posts
    1,037
    Member #
    16468
    Liked
    1 times
    Let's get into the content and forget about the visual for now.

    There are two things most people want to know about pet sitting.
    1) What do you do
    2) How much does it cost

    So I would break down the home page to those 2 categories. First... you need to make the home page 2 columns. This is more important than you may think. One reason is to section things off a little. The other reason and most important, is because users hate reading text that spans the whole site width. I didn't want to read it and I'm doing a critique! Shorter widths for your text will lure users to read the info you have spent a lot of time writing. It's a fact that it is easier to read that way, eg; newspaper print.

    You need to put on the home page, a bulleted list of your services. I didn't know what you did til I went to the rates page. I just assumed you did dog walking. In fact I would call that page "Services and Rates". You can't assume people are going to go thru your whole site.

    The home page should be an overall synopsis of what you do, then lead the user to the info from there.

    The "About" page needs to be about your biz, not the services. Why pick you? What are your credentials and experience? All that stuff you have on that page should be on the services page or a "what is?" page.

    Visually, the nav looks bad as well as the photos which was mentioned.

    This post is critical, but don't take it personal. The site needs work.

  8. #7
    Junior Member
    Join Date
    Jul 2009
    Posts
    6
    Member #
    19447
    Thank you aero, please no one hesitate I really appreciate constructive criticism. I built this web site totally on my own, with minimal experience. So anything that will make me stand out and look better is appreciated.

  9. #8
    Member
    Join Date
    Jul 2007
    Posts
    92
    Member #
    15534
    Quote Originally Posted by aeroweb99
    Let's get into the content and forget about the visual for now.

    There are two things most people want to know about pet sitting.
    1) What do you do
    2) How much does it cost

    So I would break down the home page to those 2 categories. First... you need to make the home page 2 columns. This is more important than you may think. One reason is to section things off a little. The other reason and most important, is because users hate reading text that spans the whole site width. I didn't want to read it and I'm doing a critique! Shorter widths for your text will lure users to read the info you have spent a lot of time writing. It's a fact that it is easier to read that way, eg; newspaper print.

    You need to put on the home page, a bulleted list of your services. I didn't know what you did til I went to the rates page. I just assumed you did dog walking. In fact I would call that page "Services and Rates". You can't assume people are going to go thru your whole site.

    The home page should be an overall synopsis of what you do, then lead the user to the info from there.

    The "About" page needs to be about your biz, not the services. Why pick you? What are your credentials and experience? All that stuff you have on that page should be on the services page or a "what is?" page.

    Visually, the nav looks bad as well as the photos which was mentioned.

    This post is critical, but don't take it personal. The site needs work.
    Aeroweb has some good points, I would second these. Also, its okay to have some code errors using a validation service (W3C Validator) but you have quite a few major ones... Also, if you are going to declare your doctype as XHTML, you need to make sure there are no caps in your code.

  10. #9
    Junior Member
    Join Date
    Jul 2009
    Posts
    6
    Member #
    19447
    Thanks, I just went over to w3c for the first time actually....
    I managed to cut down my errors to 15 from like 30. But my dreamweaver template seems to causing this strange "head" and "meta" closing error. Should I bother trying to fix it?

  11. #10
    Senior Member
    Join Date
    Nov 2008
    Posts
    415
    Member #
    17793
    Liked
    1 times
    I guess that would be better, though I find it quite well


Page 1 of 2 1 2 LastLast

Remove Ads

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •  
All times are GMT -6. The time now is 08:58 PM.
Powered by vBulletin® Version 4.2.3
Copyright © 2019 vBulletin Solutions, Inc. All rights reserved.
vBulletin Skin By: PurevB.com