ready for my first round of feedback on new photography site

webbydeb · September 26, 2008

I like it!

In the product info page, I would just put the attributes before the text info. You have to really scroll to make the selections, and they may confuse some...(like old people, blondes, you know ;) ;) )

But at this point, I'd leave the damn thing alone. don't want something screwing up while someone's trying to give you money!

webbydeb · September 26, 2008

BTW: That details button looks much better on top a black background! Good move.

lindsayanng · September 26, 2008

I like it!

In the product info page, I would just put the attributes before the text info. You have to really scroll to make the selections, and they may confuse some...(like old people, blondes, you know ;) ;) )

But at this point, I'd leave the damn thing alone. don't want something screwing up while someone's trying to give you money!

That is what i was TRYING to do, but everytime i made a change i broke the tables.. I tried just interchanging the description code with the attributes table, but it didnt seem to work AT ALL

webbydeb · September 26, 2008

That is what i was TRYING to do, but everytime i made a change i broke the tables.. I tried just interchanging the description code with the attributes table, but it didnt seem to work AT ALL

Oh okay, can you post the offending table here? I'll see if I can fix it for you before the REAL coders show up. Then I can look like I know something... :blush:

lindsayanng · September 26, 2008

here is the table as it is now:

		  <table border="0" cellspacing="0" cellpadding="2" align="right">
		<tr>
		  <td align="center" class="smallText">
<script language="javascript"><!--
document.write('<?php echo '<a href="java script:popupWindow(\\\'' . tep_href_link(FILENAME_POPUP_IMAGE, 'pID=' . $product_info['products_id']) . '\\\')">' . tep_image(DIR_WS_IMAGES . $product_info['products_image'], addslashes($product_info['products_name']), SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT, 'hspace="5" vspace="5"') . '<br>' . TEXT_CLICK_TO_ENLARGE . '</a>'; ?>');
//--></script>
<noscript>
<?php echo '<a href="' . tep_href_link(DIR_WS_IMAGES . $product_info['products_image']) . '" target="_blank">' . tep_image(DIR_WS_IMAGES . $product_info['products_image'], $product_info['products_name'], SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT, 'hspace="5" vspace="5"') . '<br>' . TEXT_CLICK_TO_ENLARGE . '</a>'; ?>
</noscript>
		  </td>
		</tr>
	  </table>
<?php
}
?>
		   <p><?php echo stripslashes($product_info['products_description']); ?> <h1>Please Note that we are in the process of adding images to this gallery. Digital Files are not yet available!</h1> </p> 
<?php
$products_attributes_query = tep_db_query("select count(*) as total from " . TABLE_PRODUCTS_OPTIONS . " popt, " . TABLE_PRODUCTS_ATTRIBUTES . " patrib where patrib.products_id='" . (int)$HTTP_GET_VARS['products_id'] . "' and patrib.options_id = popt.products_options_id and popt.language_id = '" . (int)$languages_id . "'");
$products_attributes = tep_db_fetch_array($products_attributes_query);
if ($products_attributes['total'] > 0) {
?>
	  <table border="0" cellspacing="0" cellpadding="2">
		<tr>

		  <td class="main" align="center"><?php echo TEXT_PRODUCT_OPTION; ?></td>
	  <td class="main" align="center"><?php /*  uncomment if you want to show the model number echo TEXT_PRODUCT_MODEL; */ ?></td>
	  <td class="main" align="center"><?php echo TEXT_PRODUCT_PRICE; ?></td>
	  <td class="main" align="center"><?php echo TEXT_PRODUCT_QUANTITY; ?></td>

 </tr>

this is the line for the descriptions:

<p><?php echo stripslashes($product_info['products_description']); ?> <h1>Please Note that we are in the process of adding images to this gallery. Digital Files are not yet available!</h1> </p>

and the list of <td> are all of the sections for the attributes.. trid moving everything around and it really got a little nutty.

webbydeb · September 26, 2008

Okay, I'm running out, but clearly the attributes are in a different sub-table. That's why it wasn't working. I think it's a simple matter to re-construct the table, but I can't see the whole thing.

For instance:

<table> *the master table*

<table> *the attributes table

<tr><td>here sits the php and the attributes</td></tr>

</table>

<table> *the thumbnail and add to cart table*

<table> *the attributes table

<tr><td>here sits the thumbnail</td></tr>

</table><br><br>

<table> *the descriptions table

<tr><td>here sits the general/product description</td></tr>

</table>

</td></tr>

</table>

lindsayanng · September 26, 2008

you rock.. I will work on this LATE tonight.. when i know all possible customers are sleeping

webbydeb · September 26, 2008

make sure the product description table is set to be as wide as the attribute and thumbnail table together...

So, attribute and thumbnail tables should be set to 250px each, and description table to 500px.

Backup and good luck! Try getting the debate in....I'm turning it on in my club tonight. LOL!

lindsayanng · September 26, 2008

i thought it had gotten canceled.. shows what i know...

webbydeb · September 26, 2008

No, Obama :wub: was going to be there anyway, and apparantly John McCain is not as important in DC as he previously thought....so now he's bringing his toys back and going to play like he was supposed to.

No one can tell my political leanings, can they!

:o

:D

lindsayanng · September 26, 2008

well, i am just dying to see Palin in ANY type of debate.. I love politics, but really, i dont NEED to see a debate to know how i am viting for.. I hope that McCain shoots himself in the foot.. but there are people who, no matter how many bad things surface about "their" candidate, they still supprt..

Do you know how many peopl ei have talked to who think Obama is Muslem???

lindsayanng · October 2, 2008

If anyone wants to take a look now, i am pretty much there.. a few things that need to be added, but more feedback is appreciated.

gretchenwf · October 5, 2008

Overall I like the look and feel of the site. I do have some comments though.

- I see the reviews box in both the left column and the right column. Also, it just says no reviews. Maybe you would want to consider only showing that once there are some reviews to show.

- On the top of the right column it just says 0 items. I am assuming that means in my cart, but there is nothing to indicate that. You should make that into an info box with a header just like galleries and reviews.

- The free shipping banner in the right column is a link, but when I click on it it just refreshes the page. Maybe it should link to your shipping information page.

- Your shipping & returns page still says to add your info.

- On the homepage I feel like the main images (the things that say Weddings, Portraits, etc.) should be clickable. It even says that your portraits are just a click away (I know that is not what that is referring to, but still... (: )

- You still have a few icky category images (by icky, I mean original osCommerce). Looks like all of them in the Corporate Events category.

- The image names look like they are whatever his camera named them. I think it would be nicer if they were given meaningful names.

- I get a little confused when clicking around because I sometimes get taken back to the non-osCommerce part of the site. For example, when I click on Bio, all the menus change. I know this is a big project, but you may want to consider merging the two sites so that they have the same menus. Navigation would be much more logical that way. (also - the bio page doesn't have a bio just a picture)

- When I click on the Weddings category, I see a broken image link for the main category image.

- On the product detail page it took me a bit to find the top 'Add to Cart' link. I thought I had to scroll all the way down. It might stand out more at the top if you move it to the white space below the picture.

- I know I read somewhere that you don't like when websites go to the cart after you add a product, but I am so used to that being the normal behavior for a site (or at least have something that says, "Product added to cart") that I wasn't sure if the site did what I wanted it to. It took me a sec to see that the cart contents in the right column had changed. You might want to make that box more obvious on the page? Or maybe add a checkout link to that box? The reason I like when sites take you to your cart after adding a product is because it is in the logical flow of buying something. I add something to the cart and then proceed to checkout. It is easier to emphasize the go to checkout process in the shopping cart. Again after I added an image to my cart I had to think about how to get to checkout which might lose you some customers.

- On your checkout_shipping.php page, the header image is a postal guy, but he is really blown up and pixelated. It looks not great. It also spaces out that row so there is a ton of white space around the page title.

- On the checkout_payment.php page at the bottom is requires you to agree to the conditions. However, it is not obvious that 'conditions of use' is a link.

- The breadcrumbs and links at the very top of the page seem a little out of place to me. I do like that they are at the top of the page, but I think it might look better if they had a slight background color. I'm not sure how that would actually look, but it seems like it could be cool, especially if you added a small gradient below it.

- The black bar at the footer just has the date in it. What purpose does that serve?

Ok, that's as far as I could go because it looks like you got all your payments working.

Again, I do like the look of the site. It is clean and fresh. It looks like you've put a lot of work into it.

Gretchen

lindsayanng · October 5, 2008

Thanks for the feedback.. i think there is something weird going on with my shopping cart box header.> You said you just the the 0, but there IS an image there that looks just the like the rest, and has a cart image which links to the shopping cart.. Were you using IE?? that is the only time i noticed that it was showing! its sooo weird!

The wedding category.. WHERE are you seeing a broken image?> I just checked and there was no broken image. I saw the WEDDINGS text at the top, and two images of two different weddings underneath, If you click on the subcategories, you are redirected to put a password in.

Did you really have a hard time seeing the ADD TO CART that was directly above the image? I will move it, but its bright red and RIGHT there.

Yea, i know that there were a few STOCK OSCOMMERCE guys that i missed. I am taking care of that RIGHT NOW. We dont have much for the CORPORATE images, but will have some tonight.. and the postal guy was removed as of now.

I will make a SPAN CLASS to make sure that the TERMS AND CONDITIONS are a link. A different color than the rest of the text is a good idea.

I am going to move the breadcrumbs below the header image again.. people dont like it at the top either

I like having the date. It might not serve a HUGE purpose, but why not?

As for the html pages looking different than the php, i did that because the photographer likes to be able to edit and change his html pages in dreamweaver design mode, so i cant make them PHP pages, which means i can not add the column navigation to them (it is php based) I CAN make the bio page into a php page and flesh it out.. i have beenwaiting for him to create his own bio, but i just might do it myself. Make it llike a resume type of thing.

Thank you VERY VERY much for the feedback.. It really is MUCH MUCH appreciated.

rvsupply · October 5, 2008

On the main page you might want to consider resampling the larger images. On the catalog side, the categories' layout style do not seem to fit the rest of the theme.

gretchenwf · October 5, 2008

Thanks for the feedback.. i think there is something weird going on with my shopping cart box header.> You said you just the the 0, but there IS an image there that looks just the like the rest, and has a cart image which links to the shopping cart.. Were you using IE?? that is the only time i noticed that it was showing! its sooo weird!

The wedding category.. WHERE are you seeing a broken image?> I just checked and there was no broken image. I saw the WEDDINGS text at the top, and two images of two different weddings underneath, If you click on the subcategories, you are redirected to put a password in.

Yes I was using IE when I saw those. I just took a look in Firefox and I see both the shopping cart and weddings text. Here is what it looks like in IE.

Did you really have a hard time seeing the ADD TO CART that was directly above the image? I will move it, but its bright red and RIGHT there.

Now that I know it is there, I see it right away. But the first time I went to the product page, I totally missed it.

Yea, i know that there were a few STOCK OSCOMMERCE guys that i missed. I am taking care of that RIGHT NOW. We dont have much for the CORPORATE images, but will have some tonight.. and the postal guy was removed as of now.

I will make a SPAN CLASS to make sure that the TERMS AND CONDITIONS are a link. A different color than the rest of the text is a good idea.

I am going to move the breadcrumbs below the header image again.. people dont like it at the top either

I like having the date. It might not serve a HUGE purpose, but why not?

As for the html pages looking different than the php, i did that because the photographer likes to be able to edit and change his html pages in dreamweaver design mode, so i cant make them PHP pages, which means i can not add the column navigation to them (it is php based) I CAN make the bio page into a php page and flesh it out.. i have beenwaiting for him to create his own bio, but i just might do it myself. Make it llike a resume type of thing.

Thank you VERY VERY much for the feedback.. It really is MUCH MUCH appreciated.

No problem. I'll check back to see your progress! (:

Gretchen

lindsayanng · October 5, 2008

THANKS FOR POSTING THE SCREEN SHOT!! that really helps. I much not have deleted the entire code for the wedding section.. are they there on other sections like PETS and such??

I also HAVE to figure out why in the WORLD my shopping cart image does not show in IE.. were you using IE6, 7?

rvsupply · October 5, 2008

THANKS FOR POSTING THE SCREEN SHOT!! that really helps. I much not have deleted the entire code for the wedding section.. are they there on other sections like PETS and such??

I also HAVE to figure out why in the WORLD my shopping cart image does not show in IE.. were you using IE6, 7?

Both IE 6 & IE7 do not show the shoping cart image.

rvsupply · October 5, 2008

Both IE 6 & IE7 do not show the shoping cart image.

Here's your problem:

<td colspan=3 width="100%">  <a href="http://bscphoto.com/catalog/shopping_cart.php" <img border="0" src="images/boxes/table_shopping_cart.gif" alt="check out">

Firefox is smart. IE does not know where the hyperlink ends (or starts).

It should be:

<td colspan=3 width="100%">  <a href="http://bscphoto.com/catalog/shopping_cart.php"> <img border="0" src="images/boxes/table_shopping_cart.gif" alt="check out">

lindsayanng · October 6, 2008

actually, this is my code..i am using the tep_href_link style of img src...

 <td colspan=3 width="100%">  <a href="<?php echo tep_href_link(FILENAME_SHOPPING_CART); ?>" 
<img border="0" src="images/boxes/table_shopping_cart.gif" alt="check out">
</a></td>

I changed it to this:

 <td colspan=3 width="100%">  <a href="<?php echo tep_href_link(FILENAME_SHOPPING_CART); ?>" >
<img border="0" src="images/boxes/table_shopping_cart.gif" alt="check out">
</a></td>

If osmeone would check again, i would appreciate it.

playcraft · October 6, 2008

actually, this is my code..i am using the tep_href_link style of img src...

 <td colspan=3 width="100%">  <a href="<?php echo tep_href_link(FILENAME_SHOPPING_CART); ?>" 
<img border="0" src="images/boxes/table_shopping_cart.gif" alt="check out">
</a></td>

I changed it to this:

 <td colspan=3 width="100%">  <a href="<?php echo tep_href_link(FILENAME_SHOPPING_CART); ?>" >
<img border="0" src="images/boxes/table_shopping_cart.gif" alt="check out">
</a></td>

If osmeone would check again, i would appreciate it.

That did the trick. ;)

lindsayanng · October 6, 2008

AWESOME.. thanks for checking.. That was something that i THOUGHT was just my computer at work acting funny.. glad someone else pointed it out too!

playcraft · October 6, 2008

AWESOME.. thanks for checking.. That was something that i THOUGHT was just my computer at work acting funny.. glad someone else pointed it out too!

If you do a lot of web designing, it would be a great idea to have most browsers installed on your computer to test (ex. IE, FF, Safari, Opera, etc.).

FYI, most users are browsing with IE (I think its around 70%, FF is around 20% and growing).

lindsayanng · October 6, 2008

Yea, i actually JUST installed IE, but for some reason i was able to see the shopping cart image in IE on my mac.. Maybe it was because it was for mac.

webbydeb · October 6, 2008

Reported spam to moderator

Archived

ready for my first round of feedback on new photography site

Recommended Posts

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites

Share this post

Link to post

Share on other sites