ubaidrahman / gwtupload

Automatically exported from code.google.com/p/gwtupload
Other
0 stars 0 forks source link

File input type does not react to click with FF 22 #180

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?

I did not reproduce the issue using the demos. This issue is probably related 
to my layout or the other scripts i am using.

Under Firefox 21 and Chrome 27, everything is fine. With the upgrade of FF 22, 
both under Linux and Windows, the file input does not react to click. 
More precisely it does react only if of type Browser file input.

I found a fix in my case by using latest git version and commenting line 304 in 
gwtupload/client/DecoratedFileUpload.java.

//      DOM.setStyleAttribute(input.getElement(), "marginLeft", "-1500px");

Why are these margin / top / left values needed ?
I found nothing in Firefox bug tracker that may explain this change in 
behaviour.

Original issue reported on code.google.com by cyril.ha...@gmail.com on 10 Jul 2013 at 10:03

GoogleCodeExporter commented 8 years ago
Could you use the last snapshot?
It uses a modern approach for FF instead of the hack we used to have for old 
versions of FF and opera.

Notice that last snapshot could break some things, let me know if it works for 
you

Original comment by manuel.carrasco.m on 16 Jul 2013 at 10:31

GoogleCodeExporter commented 8 years ago
I switched to your fix in my project. Things seems ok. This version is slowly 
tested is will probably move to production in the middle of August 

Original comment by cyril.ha...@gmail.com on 18 Jul 2013 at 4:28

GoogleCodeExporter commented 8 years ago
Thanks for testing, I have caught a couple of minor issues and api breaks, I 
hope to release it in a couple of weeks

Original comment by manuel.carrasco.m on 19 Jul 2013 at 6:41

GoogleCodeExporter commented 8 years ago

Original comment by manuel.carrasco.m on 30 Oct 2013 at 3:52

GoogleCodeExporter commented 8 years ago

Original comment by manuel.carrasco.m on 31 Dec 2013 at 3:16