As the Google Gears project has been as good as discontinued it would be better to go with an standards only (HTML5) approach in the main module.
It might, however, be worth looking into making the module a bit more modular to allow sub-modules to add support for third-party solutions until the web becomes a little more standards compliant.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | remove-google-gears-support-722320.patch | 5.35 KB | petsagouris |
| #3 | remove-google-gears-support-722320.patch | 10.64 KB | petsagouris |
| #2 | remove-google-gears-support-722320.patch | 10.64 KB | petsagouris |
| #1 | 722320.patch | 2.39 KB | drzraf |
Comments
Comment #1
drzraf commented+1 (obvious patch attached, untested)
Ideally, I think jQuery is the way to go on the subject of browser abstraction.
I know of three multiple files upload implementations:
https://github.com/blueimp/jQuery-File-Upload
http://code.google.com/p/jquery-html5-upload/
https://github.com/hermanbanken/html5-file-uploader
Let's choose the right one
I let you switch this issue to D7 if you want to but I mark "needs review" anyway
Comment #2
petsagouris commentedHere is a patch for this that removes the google gears support completely. (From README.txt, .module, and .js files).
Comment #3
petsagouris commentedSorry wrong patch attached in last reply. Here is the correct one.
Comment #4
drzraf commentedsorry to be so negative but from my short experience of Drupal, I'm quite sure your patch will be refused as it change indentation instead of Google Gears code only.
you may want to use git diff -w to skip whitespace related changes, but you'll still have to revert line-breaks ...
Comment #5
petsagouris commentedNo negativity received, thanks for your comment. I didn't know about the whitespace issue, here is another version of the same patch hopefully more complete.
Comment #6
decipheredCommitted to 7.x-1.x.
Cheers,
Deciphered.