search

uujgii / web-to-print-scripts

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

Image library double selection in IE7 #55

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
There is a problem with async image uploading on IE7.

When the file finishes uploading, it gets selected by default but "Blank" 
option for the same image field also remains selected (double_selection.png).

Try it here:
http://realestate.zetaprints.com/?page=template;TemplateID=EA336018-75F3-471B-A5
FB-888029E57B01

Original issue reported on code.google.com by agur...@gmail.com on 22 Nov 2010 at 9:08

Attachments:

GoogleCodeExporter commented 9 years ago 
I noticed that the script that generates the blank radio sets its state to 
cheked and later doesn't alter it. Probably this is what is causing the 
problem. 
Is there way to alter actual JavaScript that is on the site, or should I build 
my own page?

Original comment by jamb...@gmail.com on 22 Nov 2010 at 4:27

GoogleCodeExporter commented 9 years ago 
Made some changes, try it on that link again.

Original comment by jamb...@gmail.com on 23 Nov 2010 at 9:33

GoogleCodeExporter commented 9 years ago 
Still get the same thing. Both the uploaded image and "Blank" field are 
selected.

Original comment by agur...@gmail.com on 23 Nov 2010 at 11:51

GoogleCodeExporter commented 9 years ago 
Can you try now? I have made more changes and this is what I get:

Original comment by jamb...@gmail.com on 25 Nov 2010 at 11:19

Attachments:

GoogleCodeExporter commented 9 years ago 
I can't seem to be able to change status here so consider this to be status : 
To test

Original comment by jamb...@gmail.com on 25 Nov 2010 at 11:20

GoogleCodeExporter commented 9 years ago 
Hold on the test, it still has other problems

Original comment by jamb...@gmail.com on 25 Nov 2010 at 11:21

GoogleCodeExporter commented 9 years ago 
ok, let me know when ready

Original comment by agur...@gmail.com on 25 Nov 2010 at 11:38

GoogleCodeExporter commented 9 years ago 
Try it now.

Original comment by jamb...@gmail.com on 25 Nov 2010 at 12:19

GoogleCodeExporter commented 9 years ago 
Yep, looks good to me.

Tested in all major browsers using single and multi-image templates.

Original comment by agur...@gmail.com on 25 Nov 2010 at 1:19

GoogleCodeExporter commented 9 years ago 
Updated file
http://code.google.com/p/web-to-print-scripts/source/browse/trunk/js-system/file
-upload/fileUpload.js?spec=svn152&r=152

Original comment by jamb...@gmail.com on 25 Nov 2010 at 6:14

GoogleCodeExporter commented 9 years ago

Original comment by zetapri...@gmail.com on 25 Nov 2010 at 6:22

GoogleCodeExporter commented 9 years ago

Original comment by zetapri...@gmail.com on 26 Nov 2010 at 12:31

GoogleCodeExporter commented 9 years ago 
The diff to the previous release doesn't make sense cost the base version for 
your changes was in a branch.

Can you commit the base you started with and then the final release again?

Original comment by zetapri...@gmail.com on 26 Nov 2010 at 3:31

GoogleCodeExporter commented 9 years ago 
Version in trunk and version in branch differ in one line only. 
I have taken the file I have worked with from live site.
I will commit it in trunk, then overwrite my version.

Original comment by jamb...@gmail.com on 26 Nov 2010 at 10:34

GoogleCodeExporter commented 9 years ago 
This is fixed. Test it.
Diff:
http://code.google.com/p/web-to-print-scripts/source/diff?spec=svn155&r=155&form
at=side&path=/trunk/js-system/file-upload/fileUpload.js

Original comment by jamb...@gmail.com on 2 Dec 2010 at 7:37

GoogleCodeExporter commented 9 years ago

Original comment by agur...@gmail.com on 3 Dec 2010 at 9:22