Jump to content
Sign in to follow this  
DRIVE

[Contribution] POP-UPs MASSAGEed

Recommended Posts

Hi Stacey,

 

There is no manipulation of the images by this contrib. What you see is what you got basically. If your larger images are poor quality you need to replace them with new ones of better quality. Often times, there are "control" images which are much much larger (physically and in KB) and then persons export them to a smaller "larger" image and then the thumbnails. If you have "control" images then you just need to export them with different quality settings. Not sure what your file extensions are but if its a photography site, all images should be .jpg because gif's have a max of 256 colors. .jpg's can have millions of colors. Just my thoughts.

 

Thanks so much for your quick reply! I really appreciate it. The pop-ups look great (they are all jpgs, by the way) - and are high enough quality. Do you know how I would change the quality of just the thumbnails? Can I just go somewhere and manually replace them? (they're all different ratios, too, but there's not that many of them - relatively speaking.)

 

Thanks again for your help. You're a life-saver!

Edited by staceyd

Share this post


Link to post
Share on other sites
Thanks so much for your quick reply! I really appreciate it. The pop-ups look great (they are all jpgs, by the way) - and are high enough quality. Do you know how I would change the quality of just the thumbnails? Can I just go somewhere and manually replace them? (they're all different ratios, too, but there's not that many of them - relatively speaking.)

 

Thanks again for your help. You're a life-saver!

 

Hi Stacey,

 

A life-saver? hmmm... guess you are indebted to me forever then ;)

 

So which files are the ones that are low-quality. The thumbnails? You would just need to use Photoshop or Fireworks or any graphics editor. Open the files that have good quality, resize them, and save them. If you overwrite the existing thumbnails, you only have to upload the images. If you use new filenames for the thumbnails you have to go in and edit each product with the new thumbnail image.

Share this post


Link to post
Share on other sites
Thanks! I guess I'll do them by hand, then. That'll work. I appreciate your help. :)

 

Not a problem :)

 

There are also programs that you can take a directory of images, such as full size images and run a "batch" process that allows you to scale the images to parameters you set and to also create thumbnails at the same time such as with a -th added on to the image name. ACDSEE, Adobe... most of those allow you to do batch processes that you can tailor to suit your needs and do hundreds or even thousands of images quickly.

 

Good luck!!

Share this post


Link to post
Share on other sites

Hi,

 

I have just had this installed but two things on the pop up image on the right hand side there is a gap of about 1pixel between the image and the border (left hand side is fine). How would I fix this?

 

Number two is there a way I can make the text box have round corners?

 

Thanks Chris


Better to be looked over than overlooked!

Share this post


Link to post
Share on other sites

At the beginning, this contribution seemed to work fine and looked very pretty on my site, until I found I could look through the pop-up product image and see the text underneath! Is that "special effect" supposed to be normal? Am I doing something wrong? Thanks.

 

My website:

http://bugdorm.megaview.com.tw/product_inf...?products_id=46

Share this post


Link to post
Share on other sites

I am trying to add the script in PRODUCT_LISTING.PHP, I get an error "Parse error: syntax error, unexpected T_STRING or '<'" and it does not work.

I have added code in step 4 & 5 to INDEX.PHP and I nedd to add code in step 6 to my PRODUCT_LISTING.PHP.

 

my original code in PRODUCT_LISTING.PHP is here:

 

case 'PRODUCT_LIST_IMAGE':

$lc_align = 'center';

if (isset($HTTP_GET_VARS['manufacturers_id'])) {

$lc_text = '<a href="' . tep_href_link(FILENAME_PRODUCT_INFO, 'manufacturers_id=' . $HTTP_GET_VARS['manufacturers_id'] . '&products_id=' . $listing['products_id']) . '">' . tep_image(DIR_WS_IMAGES . $listing['products_image'], $listing['products_name'], SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT) . '</a>';

} else {

$lc_text = '<script language="javascript"><!--

document.write(\'<a href="java script:popupWindow(\\\'' . tep_href_link(FILENAME_POPUP_IMAGE, 'pID=' . $listing['products_id']) . '\\\')">' . tep_image(DIR_WS_IMAGES . $listing['products_image'], addslashes($listing['products_name']), SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT, 'hspace="5" vspace="5"') . '<br>' . tep_image(DIR_WS_IMAGES .

'image_enlarge.gif', 'Click to Enlarge') . '</a>\');

//--></script>

<noscript>

<a href="' . tep_href_link(DIR_WS_IMAGES . $listing['products_image']) . '" target="_blank">' . tep_image(DIR_WS_IMAGES . $listing['products_image'], $listing['products_name'], SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT, 'hspace="5" vspace="5"') . '<br>' . tep_image(DIR_WS_IMAGES .

'image_enlarge.gif', 'Click to Enlarge') . '</a>

</noscript>';

}

break;

-------------------------------------------------------------------------------------------

could someone help me to fit the script in step 6 in the above code?

 

Thank you very much in advance

Share this post


Link to post
Share on other sites

First of all sorry for my english.

 

Congratulation for this sweet contribution, working very well and is very easy to instal. Instalation instruction for this contribution is made profesional. But... is many other contribution where this contribution must be integrated (in my opinion). One of this contribution is "oscThumb" http://addons.oscommerce.com/info/5491 .

ex: watermark on the "oscThumb" don't work under "PopUps MASSAGEed V1_5". Please provide a solution for this problem.

 

Thanks in advance.

 

best regards

Share this post


Link to post
Share on other sites
I am trying to add the script in PRODUCT_LISTING.PHP, I get an error "Parse error: syntax error, unexpected T_STRING or '<'" and it does not work.

I have added code in step 4 & 5 to INDEX.PHP and I nedd to add code in step 6 to my PRODUCT_LISTING.PHP.

 

my original code in PRODUCT_LISTING.PHP is here:

 

case 'PRODUCT_LIST_IMAGE':

$lc_align = 'center';

if (isset($HTTP_GET_VARS['manufacturers_id'])) {

$lc_text = '<a href="' . tep_href_link(FILENAME_PRODUCT_INFO, 'manufacturers_id=' . $HTTP_GET_VARS['manufacturers_id'] . '&products_id=' . $listing['products_id']) . '">' . tep_image(DIR_WS_IMAGES . $listing['products_image'], $listing['products_name'], SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT) . '</a>';

} else {

$lc_text = '<<script language="javascript"><!--

document.write(\'<a href="java script:popupWindow(\\\'' . tep_href_link(FILENAME_POPUP_IMAGE, 'pID=' . $listing['products_id']) . '\\\')">' . tep_image(DIR_WS_IMAGES . $listing['products_image'], addslashes($listing['products_name']), SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT, 'hspace="5" vspace="5"') . '<br>' . tep_image(DIR_WS_IMAGES .

'image_enlarge.gif', 'Click to Enlarge') . '</a>\');

//--></script>

<noscript>

<a href="' . tep_href_link(DIR_WS_IMAGES . $listing['products_image']) . '" target="_blank">' . tep_image(DIR_WS_IMAGES . $listing['products_image'], $listing['products_name'], SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT, 'hspace="5" vspace="5"') . '<br>' . tep_image(DIR_WS_IMAGES .

'image_enlarge.gif', 'Click to Enlarge') . '</a>

</noscript>';

}

break;

-------------------------------------------------------------------------------------------

could someone help me to fit the script in step 6 in the above code?

 

Thank you very much in advance

 

I highlight in red what is wrong (is missing from your code), I hope ;)

 

best regards

Edited by navomodele

Share this post


Link to post
Share on other sites

A litle modification for version 1.5 ( in firefox, opera, remains [ x ] on the page).

 

in the enlarge.js original code is:

 

"

/* Layer for colored top bar - includes Product Name and a close [X} in right corner */

crossobj.innerHTML='<div id="dragbar"><table width="'+(imgwidth+2)+'" border="0" cellpadding="0" cellspacing="0"><tr><td width="97%" height="15" align="center" valign="middle"><span id="closetext" onClick="closepreview()" nowrap>'+altquery+'</span></td><td align="center"><span id="closetext" onClick="closepreview()"><font size="1" face="arial">[X]</span></td></tr></table></div><span class="imgborder"><img id="alt" src="'+which+'" onClick="closepreview();"></span>'

"

 

replace with :

 

"

/* Layer for colored top bar - includes Product Name and a close [X} in right corner */

crossobj.innerHTML='<div id="dragbar"><table width="'+(imgwidth+2)+'" border="0" cellpadding="0" cellspacing="0"><tr><td width="97%" height="15" align="center" valign="middle"><span id="closetext" onClick="closepreview()" nowrap>'+altquery+'</span></td><td align="center"><span class="closetext" onClick="closepreview()"><font size="1" face="arial">[X]</span></td></tr></table></div><span class="imgborder"><img id="alt" src="'+which+'" onClick="closepreview();"></span>'

"

best regards

Share this post


Link to post
Share on other sites

Hi, this Constribution is very great. But your version does not work with Mozilla Firefox! I have the original of www.dynamicdrive.com already worked a few times. Therefore, I know that the current version of dynamicdrive the following browsers: FF1+ IE5+ Opr7+. Can anyone here this Constribution on the current status as dynamicdrive? That would be very cool! Thank you. PS. I'am from Germany :-) My Englich is not so greatfull.....

Share this post


Link to post
Share on other sites

Hi there, totally new to this forum so I hope I havent't missed an answer to this question elsewhere (also very new to coding etc so to you all this is probably a numpty one :D ).

 

I've installed this contribution on my site and have recently simply centered the site using a div id "centerpage" in the header.php and related css. Having done so the pop up image appears as semi transparent and it can't be moved or closed down. If anyone can let me know how to correct this I'd appreciate your help.

Edited by Brooks552

Share this post


Link to post
Share on other sites

Hi all!

 

First of all, big thanks, he did a really great job with "POP-UPs MASSAGEed".

 

Question: How to disable background, when the Pop-up is ON!

 

any idea?

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
Sign in to follow this  

×