amitmodak89 / magento-w2p

Automatically exported from code.google.com/p/magento-w2p
0 stars 0 forks source link

Highlighter image upload/selection in IE7 #469

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Image fields cannot be edited through the field highlighter in IE7. Replication 
instructions:

- Navigate to
http://ec2-174-129-95-130.compute-1.amazonaws.com/mageimage/index.php/magentot-0
1/cent21-square-01.html
in IE7. This products has zpadvancedtheme set.
- Click on any image field in the preview image (highlighter)
- Upload an image, select one from your library if there are images already in 
there... and Save
- Update Preview
- Your edits are not in the preview. The image you uploaded or selected through 
the highlighter is discarded from the gwenerated preview as if you selected 
"blank"

Using the image fields from the input form works. This only happens through the 
highlighter.

Original issue reported on code.google.com by agur...@gmail.com on 23 Feb 2011 at 11:18

GoogleCodeExporter commented 8 years ago

Original comment by agur...@gmail.com on 23 Feb 2011 at 11:18

GoogleCodeExporter commented 8 years ago
Just noticed that the settings are not actually saved after clicking the Save 
button. Reopen an edited image field and you'll see that the chosen image is 
not selected. 
The field is reset.

Original comment by agur...@gmail.com on 23 Feb 2011 at 1:25

GoogleCodeExporter commented 8 years ago
Why are we having these differences between normal and in-preview behavior?

Original comment by zetapri...@gmail.com on 24 Feb 2011 at 7:04

GoogleCodeExporter commented 8 years ago

Original comment by Anatoly....@gmail.com on 28 Feb 2011 at 2:24

GoogleCodeExporter commented 8 years ago
Please, re-test it with 1.8.4.0alpha29 release. I can not reproduce the issue.

Original comment by Anatoly....@gmail.com on 28 Feb 2011 at 2:55

GoogleCodeExporter commented 8 years ago
Tested on
web_to_print_store_incl_theme   1.8.4.0alpha30 (alpha) 

http://ec2-174-129-95-130.compute-1.amazonaws.com/mageimage/index.php/magentot-0
1/1pg-template.html

Bug remains. Note that this is in IE7 only, if you're testing in IE8, it won't 
happen.

Original comment by agur...@gmail.com on 3 Mar 2011 at 9:22

GoogleCodeExporter commented 8 years ago
I'm using IE8 in IE7 mode. I don't have IE7.

Original comment by Anatoly....@gmail.com on 3 Mar 2011 at 11:10

GoogleCodeExporter commented 8 years ago
Petar, could you please test it in IE8 with IE7 mode?

Original comment by Anatoly....@gmail.com on 9 Mar 2011 at 8:24

GoogleCodeExporter commented 8 years ago
I tried and it worked fine in IE8 and comparability mode. Something is wrong 
with developer tools and couldn't switch to IE7

Original comment by jamb...@gmail.com on 9 Mar 2011 at 9:56

GoogleCodeExporter commented 8 years ago
Atanas, Can you reliably replicate the issue in IE7?

Original comment by zetapri...@gmail.com on 9 Mar 2011 at 9:44

GoogleCodeExporter commented 8 years ago

Original comment by zetapri...@gmail.com on 15 Mar 2011 at 9:20

GoogleCodeExporter commented 8 years ago
Install XP with IE7 for this and future testing.

Original comment by zetapri...@gmail.com on 17 Mar 2011 at 9:52

GoogleCodeExporter commented 8 years ago
Confirmed on
http://ec2-174-129-95-130.compute-1.amazonaws.com/mageimage/index.php/magentot-0
1/1pg-template.html

web_to_print_store_incl_theme   1.8.4.0alpha30 (alpha) 

Original comment by agur...@gmail.com on 17 Mar 2011 at 10:40

GoogleCodeExporter commented 8 years ago
Diff: 
http://code.google.com/p/magento-w2p/source/diff?spec=svn1448&r=1448&format=side
&path=/trunk/skin/frontend/default/zptheme/js/zp-in-preview-edit.js

Original comment by Anatoly....@gmail.com on 18 Mar 2011 at 11:07

GoogleCodeExporter commented 8 years ago
Please, test it with 1.8.4.0alpha32 release.

Original comment by Anatoly....@gmail.com on 22 Mar 2011 at 6:06

GoogleCodeExporter commented 8 years ago
Tested on
http://ec2-174-129-95-130.compute-1.amazonaws.com/mageimage/index.php/magentot-0
1/1pg-template.html

web_to_print_store_incl_theme   1.8.4.0alpha34 (alpha)

I'm able to upload an image in IE7, select one of the images in my library, my 
selection is remembered and included in the generated preview.

Original comment by agur...@gmail.com on 23 Mar 2011 at 11:17

GoogleCodeExporter commented 8 years ago

Original comment by zetapri...@gmail.com on 27 Mar 2011 at 7:43