search

uujgii / web-to-print-scripts

Automatically exported from code.google.com/p/web-to-print-scripts
0 stars 0 forks source link

Slow IE error #35

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Some slow IE pops an error about a slow script.
Can be something to do with jQuery. 

See http://code.google.com/p/web-to-print-scripts/issues/detail?id=31#c18

Original issue reported on code.google.com by zetapri...@gmail.com on 16 Jun 2010 at 3:01

GoogleCodeExporter commented 9 years ago 
Cannot replicate

Original comment by trilobit.alex@gmail.com on 17 Jun 2010 at 9:49

GoogleCodeExporter commented 9 years ago 
Ok.. I tested this on 2 computers. Got the same result again so it can't just 
be me.

Instructions for replication:
Go to http://realestate.zetaprints.com in IE8 and log in as a printer
go to this template
http://realestate.zetaprints.com/?page=template;TemplateID=B3FA83FD-EEFE-4B1F-9F
FD-AF46DCE4E269

Get 5 images ready. 1 large image (above 2mb) and 4 smaller ones.
Change all image file tabs to "Upload" tab
Upload the large image in the first image field and then all the other images 
for the remaining image fields. In other words, you need to pile them up in the 
queue and leave the uploader to do its thing.
As soon as you've chosen the last image don't touch anything else. Just leave 
the uploader to work its way down the queue. IE may throw an err after the 
first image is uploaded or after the last one is, but it does show an err for 
me every time. Wait it out.

Original comment by agur...@gmail.com on 28 Jun 2010 at 1:56

GoogleCodeExporter commented 9 years ago 
We need to review the code. I'm sure there are a few jQuery selectors that 
shouldn't be there.

Original comment by zetapri...@gmail.com on 28 Jun 2010 at 9:23

GoogleCodeExporter commented 9 years ago

Original comment by zetapri...@gmail.com on 28 Jun 2010 at 9:23

GoogleCodeExporter commented 9 years ago 
Script has been optimized

Original comment by trilobit.alex@gmail.com on 30 Jun 2010 at 8:28

GoogleCodeExporter commented 9 years ago 
Links to the diff ?

Original comment by zetapri...@gmail.com on 30 Jun 2010 at 9:54

GoogleCodeExporter commented 9 years ago 
Script has been optimized, but works doesn't finished yet. Please, test in IE. 
Will the error message show?

Original comment by trilobit.alex@gmail.com on 30 Jun 2010 at 10:15

GoogleCodeExporter commented 9 years ago 
SVN release
http://code.google.com/p/web-to-print-scripts/source/detail?r=125

script works faster, now

Original comment by trilobit.alex@gmail.com on 30 Jun 2010 at 12:00

GoogleCodeExporter commented 9 years ago 
Err msg didn't show up. Tested it in IE on the same template:
http://realestate.zetaprints.com/?page=template;TemplateID=B3FA83FD-EEFE-4B1F-9F
FD-AF46DCE4E269

Piled up 5 and 6 images, combination of JPGs and PNGs, no script err came up.

Original comment by agur...@gmail.com on 30 Jun 2010 at 7:50

GoogleCodeExporter commented 9 years ago 
I want to see diffs, please. Release is not a diff.
Provide a link diffs for files involved in this fix.

Original comment by zetapri...@gmail.com on 30 Jun 2010 at 11:10

GoogleCodeExporter commented 9 years ago 
See comments here.

http://code.google.com/p/web-to-print-scripts/source/detail?r=125

Please, make your changes clear next time you report something TO TEST.

Original comment by zetapri...@gmail.com on 30 Jun 2010 at 11:43

GoogleCodeExporter commented 9 years ago

Original comment by zetapri...@gmail.com on 2 Jul 2010 at 11:05