Jump to content

Archived

This topic is now archived and is closed to further replies.

mrsC2003

Open for review

Recommended Posts

At last I think I am ready for some comments. I was not looking to change all that much, but some colors and get rid of a few boxes, redo some graphics and Icons, and give it a kind of old news paper type fell without having to use a background graphic to slow down things. I have added some contributions, most of them are backbend stuff for the admin.

 

I know this is not the flash up to date look that most are going for, so please keep that in mind when you visit.

 

Thank you

 

Kel

 

Kel's Corner Store


I live in my own little world. But it's OK. They know me here.

Share this post


Link to post
Share on other sites

I would like to say sorry to those who click on my link and get DNS error page. I am very upset that my server company sold out to another this past week and I was promised that my site would not be effected.....but it seems that this is not the truth. The real shame is I am paid up for another 6 months on my account.

 

Please don't give up on checking out my site because I really want to know what you all think and hear your ideas

 

Kel


I live in my own little world. But it's OK. They know me here.

Share this post


Link to post
Share on other sites

It is up and running now.

 

You might want to do something with your infobox headers. I am guessing that you are using a transparent gif for the corners. If so you should use black or just edit them out . Otherwise from what I saw it looked pretty good . I would think about getting some form of SSL. If just for the customers Info makes Folks feel good when they are protected


Listen Everyone!

Just be Calm, Cooperate and this will all be over in a minute

Share this post


Link to post
Share on other sites

Thank you for for your comments.

 

The headers have no transparent .gif's I want them to be like that. The images are really there just made to blend in.

 

I don't collect money on my site.....I direct to paypal so I don't need a SSL.

 

Kel


I live in my own little world. But it's OK. They know me here.

Share this post


Link to post
Share on other sites

Hi Kel, indeed a different, but nice look.

 

Suggested improvements:

- Resize paypal box a little so the width is the same as for the other boxes in the righthand column.

- Given the centered box image on top, I'd put the shopping cart contents also in the top header area.

 

Carine


KEEP CALM AND CARRY ON

I do not use the responsive bootstrap version since i coded my responsive version earlier, but i have bought every 28d of code package to support burts effort and keep this forum alive (albeit more like on life support).

So if you are still here ? What are you waiting for ?!

 

Find the most frequent unique errors to fix:

grep "PHP" php_error_log.txt | sed "s/^.* PHP/PHP/g" |grep "line" |sort | uniq -c | sort -r > counterrors.txt

Share this post


Link to post
Share on other sites

Thank you Carine

 

I was thinking about changing that paypal box, I just am going to give it a few days till this new server company stops messing around. I have had to redo the same changes 3 days in a row, and each morning I got up to the same site from the week before when they got it from the old server. But I will make sure to do it. And I will see how the shopping cart looks in the header. I am still thinking about some kind of top that would make it look like a news paper.

 

Thanks

 

Kel


I live in my own little world. But it's OK. They know me here.

Share this post


Link to post
Share on other sites

I haven't had a chance to check out your site, but I just thought I'd share my opinion about the SSL that Terry was talking about. Even if you don't collect money, but you do collect customer information like address, phone number and email address... I'd want to be protected if I was signing up on your site....

 

Just a thought....


Joel Gallant

Share this post


Link to post
Share on other sites

hmmm, now its your database....

 

 

Warning: mysql_connect(): Can't connect to local MySQL server through socket '/var/tmp/mysql.sock' (2) in /home/kels2/public_html/includes/functions/database.php on line 19

Unable to connect to database server!

Share this post


Link to post
Share on other sites

the error

 

"Warning: mysql_connect(): Can't connect to local MySQL server through socket '/var/tmp/mysql.sock' (2) in /home/kels2/public_html/includes/functions/database.php on line 19

Unable to connect to database server! "

 

Is my server messing around with my files. They are doing some kind of work to their server's sql so it makes this error. To say I am getting a bit upset is an understatement.

 

I am so sorry it was not their for you to view

 

As for the SSL thing, the only part that is not a matter of public record is the email address. My .com emaill is at much greater risk then that of any one who make an account.

 

But I will keep the SSL thing in mind.

 

Kel


I live in my own little world. But it's OK. They know me here.

Share this post


Link to post
Share on other sites

If your server has one you can just share it. Check and see if they have one. If so all you need to do is make a couple of small changes in your configure files and you are set.

 

If you ever think about getting a stable server. The one I use is rock solid and is under $10 monthly. And have a 99% uptime . with SSl, If you wish to know more just PM me.

 

Plus they give you 3 Databases for that price.. So you can have a backup


Listen Everyone!

Just be Calm, Cooperate and this will all be over in a minute

Share this post


Link to post
Share on other sites

Yes they have one, at least they list it in the cpanel. I emailed them about it and await a reply.

 

Still love that background of yours Terry

 

Kel


I live in my own little world. But it's OK. They know me here.

Share this post


Link to post
Share on other sites

Well it has been a few days and it seems my server has at last got everything in order and is not messing my site up any more. Thanks to all who have not given up on checking me out.

 

I have gotten family products installed now and I am busy making familys.

 

Once again thank you all for your understanding.

 

Kel


I live in my own little world. But it's OK. They know me here.

Share this post


Link to post
Share on other sites

I got a SSL certificate and await my server company. RegisterFly.com has them right now for one year fee of $24.99 if anyone is looking for one that will not break the bank.

 

Kel


I live in my own little world. But it's OK. They know me here.

Share this post


Link to post
Share on other sites

While I am not a coder or graphic artist by any means, I do seem to do a decent job proof reading (future employment opportunity!?).

 

You can have the prettiest site on the net, but if you have articles and text with words spelled incorrectly and gramatical errors, potential customers will head for the door.

 

In the first ten seconds I was on your site, I began to read the article regarding Paypal and the change to that service.

 

"This is great news" says many business across the country."

 

"This is great news," stated many businesses (or business owners) across the country.

 

ALWAYS proof read before posting to a business website.

 

Aesthetics: I really like the absence of the top right corner graphics on your boxes. I think that looks nice and artistic. Paypal box on right menu definately needs to be resized to match the width of the other boxes though.


Dann Out!

Share this post


Link to post
Share on other sites

Thank you Dan for your corretion of my gramer, and here it is my spelling I worry about the most. (spell check is a heaven sent for dislexic me). And thank you for your comments about my boxes. As for that pay pal box...it seems to have a mind of it'. I have tried and tried to get it to be the same as the others but I have not a clue as to why it keeps changing sizes according to the review box. It most likely is a dumb error in the code that I have looked at so often I have become blind to it. Here I will post it to see if any one can see it.

 

Thanks

 

Kel

 

?>

<!-- Card Info Box //--><table width="100%"  border="1" cellspacing="0" cellpadding="0">

          <tr>

            <td>

<?php

  $info_box_contents = array();

  $info_box_contents[] = array('align' => 'left',

                              'text'  => BOX_HEADING_CARD

                              );

  new infoBoxHeading($info_box_contents, false, false);

 

  $info_box_contents = array();

  $info_box_contents[] = array('align' => 'center',

//                              'text'  => tep_image(DIR_WS_IMAGES . 'cards.gif') .

//elari chanegd to provide a link to your payment acount

                              'text'  => '<table width="100%"  border="0" cellspacing="0" cellpadding="0">

          <tr>

            <td>

<table width="100%"  align="center" >

  <tr>

    <td class="infoBoxContents"><div align="center"><a href="https://www.paypal.com/us/verified/pal=customer_service%40kels-site.com"><b>We

        Are Verified<b></a> </div></td>

  </tr>

  <tr>

    <td class="smalltext" ><div align="center">American Express, Master Card,

        Visa, E-Check, Discovery</div></td>

  </tr>

  <tr>

    <td class="infoBoxContents" ><div align="center"><strong>Our Store Is </strong></div></td>

  </tr>

  <tr>

    <td height="56" ><div align="center">

            <table width="100%" border="0">

              <tr>

                <td class="footer"><div align="center"><a href="http://www.freessl.com/starterssl/starterssl.html" target="_blank"><img src="/images/secured_by_starterssl.gif" alt="Secured By StarterSSL 128 bit Encryption" width="113" height="54" border="0"></a></div></td>

              </tr>

            </table>

          </div></td>

  </tr>

</table>'

);                             

new infoBox($info_box_contents);

         

?>


I live in my own little world. But it's OK. They know me here.

Share this post


Link to post
Share on other sites

Is it just me or am I seeing only two </TABLE> closing tages and three <TABLE> opening tags. I would not think this would cause your problems, but then again...

 

I rellay enjoyed how you included the "free gift" part of your site. It sets you apart. However, one suggestion. Do an "if else" statement in your code to allow for the actual "invisible" category your created for the freebies to not either show period, or not have an arrow accompanied before it. Otherwise, this little arrow shows up after you have enough to get a freebie without text. Most will probably NEVER notice it, but details are what set us apart. :-)

 

I particularly enjoy the design of your site color-wise, though it does venture very close to a stock layout of osC. The colors make me think of the country decor I actually like, without all the clutter, knicknacks and plaid! (GASP!) Clean, original and nicely done. The newspaper look also works for me, with the two columns in the center. Usually, this is frowned upon, but you have managed to make it work very nicely. Congratulations.

 

Good use of a USP (Unique Sales Proposition) immediately top and center! "We believe shopping should be relaxing and crowd free." Assuming you follow throgh with this to the end, and incorporate the ease and relaxing issues of shopping with you, it is a grand and individual USP. Whether you consciously created that phrase as a USP or not is irrelevant, what you do with it is.

 

I seem to be recommending the box config contribtuion alot lately, but it helps dinstinguish the sites more from osC. You can find it here:

 

http://www.oscommerce.com/community/contri...1/search,custom

 

May you and George have the best of luck with your ventures!

 

Ruhl

Share this post


Link to post
Share on other sites

Hi Kel,

 

I'm a bit late getting round to looking at your store but I thought I'd add my 2 cents/pence...

 

I really like the newspaper idea/ theme but you should know that the narrow columns (Kel's Offers Quantity Buying At Even Greater Savings) can be pretty difficult to read (esp at 800x600)

 

Also on most sites I'd expect the logo to be a link to the home page, although the breadcrumb trail obviously does the job nicely.

 

Nice site, very cute graphics!

Anna

Share this post


Link to post
Share on other sites

Thank You Ruhl and Anna very much for your help and your comments.

 

Ruhl

I down loaded the contribution you suggested and I will give it a go once I get done updating the paypal module. And as for the USP (Unique Sales Proposition) , well that is my brain child, and to the best of my memory, I have never seen or heard it used before, although I have been around since the days of 8 tracks and 45 records .... :lol: , my memory is not what it use to be. Everything has a county flavor and down to earth because I grew up in farm country and we are never in a rush to do anything but get the cows milked and get the crops in before it rains, and we love peace and quiet. The logo is really our business card that I designed, and people liked it so well that I decided to use it on the site. We do fundraising for our community locally so I needed a card as well as a flier.

 

The if statement you talk about is something I don't understand. I have very little know how when it comes to php and sql and java. I know I did use Category Box Enhancement v 1.1 to make it have the little graphics beside the cat names. But I will try to find the file/s that control that and see what I can do to mess it up. Also the free gift contribution. So I will have a lot of hunting I would guess.

 

Anna

I will work on making the column to see what I can do to correct that issue, and toss in a link on the logo. You are right the majority of sites always have the logo linked to the index page for the site.

 

Kel


I live in my own little world. But it's OK. They know me here.

Share this post


Link to post
Share on other sites

Hello Kel:

 

Forgive me, I use a heavily modified edition of osC and so my files will most definitely differ from yours. However, the material you will be looking for is in the file catalog/includes/boxes/categories.php.

 

You will want to look for something like the following:

 

 ?function tep_show_category($counter) {
? ?global $foo, $categories_string, $id;

? ?for ($a=0; $a<$foo[$counter]['level']; $a++) {
? ? ?$categories_string .= "  "; ?//This could possibly be "pointer_blue_light.gif"
? ?}

? ?$categories_string .= '<a href="';

? ?if ($foo[$counter]['parent'] == 0) {
? ? ?$cPath_new = 'cPath=' . $counter;
? ?} else {
? ? ?$cPath_new = 'cPath=' . $foo[$counter]['path'];
? ?}

 

You can change the above to something resembling the following:

 

  function tep_show_category($counter) {
   global $foo, $categories_string, $id;

   for ($a=0; $a<$foo[$counter]['level']; $a++) {
     if ($categoryid != 62) {  //I noticed your free cat # is 62
     $categories_string .= "  ";  //This could possibly be "pointer_blue_light.gif"
     }
   }

   $categories_string .= '<a href="';

   if ($foo[$counter]['parent'] == 0) {
     $cPath_new = 'cPath=' . $counter;
   } else {
     $cPath_new = 'cPath=' . $foo[$counter]['path'];
   }

 

What this basically does is allow you to have all categories that are NOT category # 62 (free one) have the "pointer blue light" arrow. Of course, you can extend this "if" statement to include the entire category listing procedure so that no text will show up if you gave the category a name.

 

Remember, back-up your files before experimenting!

 

Ruhl

Share this post


Link to post
Share on other sites

gee thanks so much .... you are a doll........ :)

 

Kel


I live in my own little world. But it's OK. They know me here.

Share this post


Link to post
Share on other sites

It is definitely no problem. I just like to remind everyone I too make mistakes...ok, ALOT of mistakes, so I am sure that if you played around with the code you could do it cleaner and more efficiently.

 

Again, best of luck with your site!

 

Ruhl

Share this post


Link to post
Share on other sites

As I've stated in my first post in this thread, I am no coder. It would seem to me that you can work around the paypal box issue by giving the table a finite pixel number instead of just 100%. This may pull the box out to the right size, with some guessing and re-guessing.

 

I am horrid with tables, so even if I was looking at a table with borders (which I have to do when I am tinkering with them to keep m bearings), I would have troubles.

 

Just a thought


Dann Out!

Share this post


Link to post
Share on other sites

Thank you both once again for your help. And Dann thank you for the email.

 

Kel


I live in my own little world. But it's OK. They know me here.

Share this post


Link to post
Share on other sites

Ruhl, I am sorry to say my code looks nothing like your post.

 

my catalog/includes/boxes/categories.php.

 

function tep_show_category($counter) {

 

// BoF - Contribution Category Box Enhancement 1.1

    global $tree, $categories_string, $cPath_array, $cat_name;

 

    for ($i=0; $i<$tree[$counter]['level']; $i++) {

      $categories_string .= "  ";

    }

    $cPath_new = 'cPath=' . $tree[$counter]['path'];

    $categories_string .= '<a href="';

 

    if ($tree[$counter]['parent'] == 0) {

      $cPath_new = 'cPath=' . $counter;

    } else {

      $cPath_new = 'cPath=' . $tree[$counter]['path'];

    }

 

  $categories_string .= tep_href_link(FILENAME_DEFAULT, $cPath_new) . '">';

 

    if (tep_has_category_subcategories($counter)) {

      $categories_string .= tep_image(DIR_WS_IMAGES . 'pointer_blue.gif', '');

    }

    else {

      $categories_string .= tep_image(DIR_WS_IMAGES . 'pointer_blue_light.gif', '');

    }

 

    if (isset($cPath_array) && in_array($counter, $cPath_array)) {

      $categories_string .= '<b>';

    }

   

    if ($cat_name == $tree[$counter]['name']) {

      $categories_string .= '<span class="errorText">';

    }

 

// display category name

    $categories_string .= $tree[$counter]['name'];

 

  if ($cat_name == $tree[$counter]['name']) {

  $categories_string .= '</span>';

    }

 

    if (isset($cPath_array) && in_array($counter, $cPath_array)) {

      $categories_string .= '</b>';

    }

 

    if (tep_has_category_subcategories($counter)) {

      $categories_string .= '->';

    }

 

    $categories_string .= '</a>';

 

    if (SHOW_COUNTS == 'true') {

      $products_in_category = tep_count_products_in_category($counter);

      if ($products_in_category > 0) {

        $categories_string .= ' (' . $products_in_category . ')';

      }

    }

 

    $categories_string .= '<br>';

 

    if ($tree[$counter]['next_id'] != false) {

      tep_show_category($tree[$counter]['next_id']);

    }

  }

?>

<!-- categories //-->

          <tr>

            <td>

<?php

 

// BoF - Contribution Category Box Enhancement 1.1

if (isset($cPath_array)) {

  for ($i=0, $n=sizeof($cPath_array); $i<$n; $i++) {

    $categories_query = tep_db_query("select categories_name from " . TABLE_CATEGORIES_DESCRIPTION . " where categories_id = '" . (int)$cPath_array[$i] . "' and language_id = '" . (int)$languages_id . "'");

    if (tep_db_num_rows($categories_query) > 0)

    $categories = tep_db_fetch_array($categories_query);

  }

$cat_name = $categories['categories_name'];

}

// EoF Category Box Enhancement

// display category name

 

  $info_box_contents = array();

  $info_box_contents[] = array('text' => BOX_HEADING_CATEGORIES);

 

  new infoBoxHeading($info_box_contents, true, false);

 

  $categories_string = '';

  $tree = array();

 

  $categories_query = tep_db_query("select c.categories_id, cd.categories_name, c.parent_id from " . TABLE_CATEGORIES . " c, " . TABLE_CATEGORIES_DESCRIPTION . " cd where c.parent_id = '0' and c.categories_id = cd.categories_id and cd.language_id='" . (int)$languages_id ."' order by sort_order, cd.categories_name");

  while ($categories = tep_db_fetch_array($categories_query))  {

    $tree[$categories['categories_id']] = array('name' => $categories['categories_name'],

                                                'parent' => $categories['parent_id'],

                                                'level' => 0,

                                                'path' => $categories['categories_id'],

                                                'next_id' => false);

 

    if (isset($parent_id)) {

      $tree[$parent_id]['next_id'] = $categories['categories_id'];

    }

 

    $parent_id = $categories['categories_id'];

 

    if (!isset($first_element)) {

      $first_element = $categories['categories_id'];

    }

  }

 

  //------------------------

  if (tep_not_null($cPath)) {

    $new_path = '';

    reset($cPath_array);

    while (list($key, $value) = each($cPath_array)) {

      unset($parent_id);

      unset($first_id);

      $categories_query = tep_db_query("select c.categories_id, cd.categories_name, c.parent_id from " . TABLE_CATEGORIES . " c, " . TABLE_CATEGORIES_DESCRIPTION . " cd where c.parent_id = '" . (int)$value . "' and c.categories_id = cd.categories_id and cd.language_id='" . (int)$languages_id ."' order by sort_order, cd.categories_name");

      if (tep_db_num_rows($categories_query)) {

        $new_path .= $value;

        while ($row = tep_db_fetch_array($categories_query)) {

          $tree[$row['categories_id']] = array('name' => $row['categories_name'],

                                              'parent' => $row['parent_id'],

                                              'level' => $key+1,

                                              'path' => $new_path . '_' . $row['categories_id'],

                                              'next_id' => false);

 

          if (isset($parent_id)) {

            $tree[$parent_id]['next_id'] = $row['categories_id'];

          }

 

          $parent_id = $row['categories_id'];

 

          if (!isset($first_id)) {

            $first_id = $row['categories_id'];

          }

 

          $last_id = $row['categories_id'];

        }

        $tree[$last_id]['next_id'] = $tree[$value]['next_id'];

        $tree[$value]['next_id'] = $first_id;

        $new_path .= '_';

      } else {

        break;

      }

    }

  }

  tep_show_category($first_element);

 

  $info_box_contents = array();

  $info_box_contents[] = array('text' => $categories_string);

 

  new infoBox($info_box_contents);

?>

            </td>

          </tr>

 

It was suppose to remove them > marks at the end of the cat name but never did. But any how that is the code.

 

Kel


I live in my own little world. But it's OK. They know me here.

Share this post


Link to post
Share on other sites

oh sorry forgot to mention normally I have free gifts set to 99, but must of missed setting it in my test cart or my fingers took over and used their own mind but once again..... :lol:

 

Kel


I live in my own little world. But it's OK. They know me here.

Share this post


Link to post
Share on other sites

×