shashalien / upload-at-click

Automatically exported from code.google.com/p/upload-at-click
0 stars 0 forks source link

The newest version is broken under IE7 #7

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Using IE7, go to http://upload-at-click.narod.ru/demo.html
2. The Upload button doesn't respond to the click
3. There's an error dialog saying Line: 6 Char: 281 Error: Type mismatch

What version of the product are you using? On what operating system?
The latest version. This problem only occurs on Windows with Internet Explorer 7

Please provide any additional information below.
I traced the problem to the following line:
container.style.zIndex = params['zindex'];

The zIndex property doesn't accept the "auto" string, only numbers.
This doesn't work:
container.style.zIndex = "auto";
This does:
container.style.zIndex = 0;

This is very strange, most certainly a bug in IE7, since "z-index: auto" is 
standard in CSS. Possible way of fixing it:
try { container.style.zIndex = params['zindex']; } catch(err) { }

Unfortunately many people are still using IE7.

Original issue reported on code.google.com by tdem...@gmail.com on 16 Dec 2010 at 7:37

GoogleCodeExporter commented 8 years ago
Thank, tdemjen,

I am grateful you for the decision.
Realy, IE before 8 not support z-index: auto

New version be uploaded after testing.

Original comment by vital.fadeev on 17 Dec 2010 at 7:03

GoogleCodeExporter commented 8 years ago
New version uploaded.
New z-index autodetection, added 'target' parameter.
Tested with IE 6.

Please check with IE 7.
http://upload-at-click.narod.ru/demo.html

Original comment by vital.fadeev on 17 Dec 2010 at 10:08

GoogleCodeExporter commented 8 years ago
Yes it does. Thank you.

Original comment by tdem...@gmail.com on 17 Dec 2010 at 6:16

GoogleCodeExporter commented 8 years ago

Original comment by vital.fadeev on 17 Dec 2010 at 6:49