From drupal-devel@drupal.org Tue Aug  9 01:39:08 2005
From: Steven <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Tue, 09 Aug 2005 01:39:09 +0000
Message-ID: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============8652196137043266260=="

--===============8652196137043266260==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      4.6.0
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   Steven
 Status:       patch (code needs review)
 Attachment:   http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress




Steven


--===============8652196137043266260==--


From drupal-devel@drupal.org Tue Aug  9 01:39:45 2005
From: Steven <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Tue, 09 Aug 2005 01:39:45 +0000
Message-ID: <type=project&nid=28483&cid=38532&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============7881998504559155710=="

--===============7881998504559155710==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      4.6.0
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   Steven
 Status:       patch (code needs review)
 Attachment:   http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




Steven



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress





--===============7881998504559155710==--


From drupal-devel@drupal.org Tue Aug  9 15:38:26 2005
From: m3avrck <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Tue, 09 Aug 2005 15:38:26 +0000
Message-ID: <type=project&nid=28483&cid=38594&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============0919898710557231652=="

--===============0919898710557231652==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      4.6.0
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   m3avrck
 Status:       patch (code needs review)

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.




m3avrck



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.





--===============0919898710557231652==--


From kristjan.jansen@gmail.com Tue Aug  9 17:18:04 2005
From: Kristjan Jansen <kristjan.jansen@gmail.com>
To: development@drupal.org
Subject: Re: [drupal-devel] [feature] JS-based file upload
Date: Tue, 09 Aug 2005 17:18:05 +0000
Message-ID: <37a603d0050809101873d91ca4@mail.gmail.com>
In-Reply-To: <type=project&nid=28483&cid=38594&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============6746772673148590033=="

--===============6746772673148590033==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Great addition. The only suggestion - perhaps some css class or "new"
label next to a freshly added file. When adding another file to a
already long list, the latest addition might  get unnotified.
A Drupal Fade Technique? ;) http://www.axentric.com/posts/default/7

> http://www.acko.net/yay-progress

hm, quite an interesting UI concept to mix classic "progressing" bar
and "endless loop" bar. I'd still keep those separate. Also, the
progress % is hard to read on animated background

--===============6746772673148590033==--


From steven@acko.net Tue Aug  9 17:48:46 2005
From: Steven Wittens <steven@acko.net>
To: development@drupal.org
Subject: Re: [drupal-devel] [feature] JS-based file upload
Date: Tue, 09 Aug 2005 17:48:46 +0000
Message-ID: <42F8EC34.9020401@acko.net>
In-Reply-To: <37a603d0050809101873d91ca4@mail.gmail.com>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============5018686999723174380=="

--===============5018686999723174380==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit


>Great addition. The only suggestion - perhaps some css class or "new"
>label next to a freshly added file. When adding another file to a
>already long list, the latest addition might  get unnotified.
>A Drupal Fade Technique? ;) http://www.axentric.com/posts/default/7
>  
>
Yeah I wasn't sure if we wanted something like that here: the new file 
entry shows up where the file form and progressbar was. The visual link 
is quite clear. Implementing a generic Yellow-Fade-Technique in Drupal 
is also tricky because you need to deal with the page cache.

>hm, quite an interesting UI concept to mix classic "progressing" bar
>and "endless loop" bar. I'd still keep those separate. Also, the
>progress % is hard to read on animated background
>  
>
The percentage should be above the bar, not in it... you're probably 
seeing some clearing issue with my theme. Because it will be monitoring 
the progress through Ajax, it will not update very smoothly, so a 
constant animation is not a bad idea.

Thanks for the review,
Steven Wittens

--===============5018686999723174380==--


From drupal-devel@drupal.org Wed Aug 10 21:44:21 2005
From: drumm <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Wed, 10 Aug 2005 21:44:21 +0000
Message-ID: <type=project&nid=28483&cid=38855&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============7823865521612416599=="

--===============7823865521612416599==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      4.6.0
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   drumm
 Status:       patch (code needs review)

+1


Works as expected and it degrades well (at first I didn't do a hard
refresh and my browser wasn't seeing the new js or css).




drumm



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




------------------------------------------------------------------------

Tue, 09 Aug 2005 15:38:19 +0000 : m3avrck

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.





--===============7823865521612416599==--


From drupal-devel@drupal.org Wed Aug 17 00:41:18 2005
From: Steven <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Wed, 17 Aug 2005 00:41:18 +0000
Message-ID: <type=project&nid=28483&cid=39536&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============7377640418774728094=="

--===============7377640418774728094==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      cvs
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   Steven
 Status:       patch (code needs review)
 Attachment:   http://drupal.org/files/issues/jsupload_0.patch (14.6 KB)

Keeping up to date. Still needs confirmed testing on KHTML browsers like
Konqueror and Safari, please.




Steven



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




------------------------------------------------------------------------

Tue, 09 Aug 2005 15:38:19 +0000 : m3avrck

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.




------------------------------------------------------------------------

Wed, 10 Aug 2005 21:44:18 +0000 : drumm

+1


Works as expected and it degrades well (at first I didn't do a hard
refresh and my browser wasn't seeing the new js or css).





--===============7377640418774728094==--


From drupal-devel@drupal.org Wed Aug 17 15:58:16 2005
From: Gabriel Radic <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Wed, 17 Aug 2005 15:58:20 +0000
Message-ID: <type=project&nid=28483&cid=39596&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============5325013334446850376=="

--===============5325013334446850376==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      cvs
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   Gabriel Radic
 Status:       patch (code needs review)

I'd be happy to test on Safari (1.2 and 2/1.3) and other WebKit browsers
and report the results here, but I can't apply the patch on any of my
installs (too risky) and don't have the time to set up a new Drupal
installation.


Steven, any chance for you to share a web page for me to test?


Thanks.




Gabriel Radic



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




------------------------------------------------------------------------

Tue, 09 Aug 2005 15:38:19 +0000 : m3avrck

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.




------------------------------------------------------------------------

Wed, 10 Aug 2005 21:44:18 +0000 : drumm

+1


Works as expected and it degrades well (at first I didn't do a hard
refresh and my browser wasn't seeing the new js or css).




------------------------------------------------------------------------

Wed, 17 Aug 2005 00:41:14 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_0.patch (14.6 KB)

Keeping up to date. Still needs confirmed testing on KHTML browsers like
Konqueror and Safari, please.





--===============5325013334446850376==--


From drupal-devel@drupal.org Wed Aug 17 16:09:47 2005
From: stefan nagtegaal <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Wed, 17 Aug 2005 16:09:47 +0000
Message-ID: <type=project&nid=28483&cid=39607&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============4493985475249401563=="

--===============4493985475249401563==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      cvs
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   stefan nagtegaal
 Status:       patch (code needs review)

Maybe it is me, but Steven why do you bother about the fact that this
will work on Safari? All other JS/AJAX we currently have in core
doesn't work with Safari...


Is it something special, or is t just curiocity?


Stefan




stefan nagtegaal



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




------------------------------------------------------------------------

Tue, 09 Aug 2005 15:38:19 +0000 : m3avrck

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.




------------------------------------------------------------------------

Wed, 10 Aug 2005 21:44:18 +0000 : drumm

+1


Works as expected and it degrades well (at first I didn't do a hard
refresh and my browser wasn't seeing the new js or css).




------------------------------------------------------------------------

Wed, 17 Aug 2005 00:41:14 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_0.patch (14.6 KB)

Keeping up to date. Still needs confirmed testing on KHTML browsers like
Konqueror and Safari, please.




------------------------------------------------------------------------

Wed, 17 Aug 2005 15:58:07 +0000 : Gabriel Radic

I'd be happy to test on Safari (1.2 and 2/1.3) and other WebKit browsers
and report the results here, but I can't apply the patch on any of my
installs (too risky) and don't have the time to set up a new Drupal
installation.


Steven, any chance for you to share a web page for me to test?


Thanks.





--===============4493985475249401563==--


From drupal-devel@drupal.org Wed Aug 17 16:35:50 2005
From: Steven <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Wed, 17 Aug 2005 16:35:51 +0000
Message-ID: <type=project&nid=28483&cid=39610&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============8933496869573636252=="

--===============8933496869573636252==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      cvs
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   Steven
 Status:       patch (code needs review)

Mostly because I can't test it, and Javascript is a fidgety business.
But I have to admit I was surprised when I found out (recently!) that
Drupal JS doesn't work in Safari. The features we check for are all
basic W3C DOM stuff, I assumed Safari would support them.


Do we know why Safari doesn't work?




Steven



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




------------------------------------------------------------------------

Tue, 09 Aug 2005 15:38:19 +0000 : m3avrck

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.




------------------------------------------------------------------------

Wed, 10 Aug 2005 21:44:18 +0000 : drumm

+1


Works as expected and it degrades well (at first I didn't do a hard
refresh and my browser wasn't seeing the new js or css).




------------------------------------------------------------------------

Wed, 17 Aug 2005 00:41:14 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_0.patch (14.6 KB)

Keeping up to date. Still needs confirmed testing on KHTML browsers like
Konqueror and Safari, please.




------------------------------------------------------------------------

Wed, 17 Aug 2005 15:58:07 +0000 : Gabriel Radic

I'd be happy to test on Safari (1.2 and 2/1.3) and other WebKit browsers
and report the results here, but I can't apply the patch on any of my
installs (too risky) and don't have the time to set up a new Drupal
installation.


Steven, any chance for you to share a web page for me to test?


Thanks.




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:09:43 +0000 : stefan nagtegaal

Maybe it is me, but Steven why do you bother about the fact that this
will work on Safari? All other JS/AJAX we currently have in core
doesn't work with Safari...


Is it something special, or is t just curiocity?


Stefan





--===============8933496869573636252==--


From drupal-devel@drupal.org Wed Aug 17 19:44:31 2005
From: Crell <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Wed, 17 Aug 2005 19:44:31 +0000
Message-ID: <type=project&nid=28483&cid=39635&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============1568088059356025725=="

--===============1568088059356025725==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      cvs
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   Crell
 Status:       patch (code needs review)

I'm not sure about Drupal in particular, but Safari is missing a LOT Of
W3C DOM "stuff".  Many functions are there, but stubbed out and do
nothing.  Others are badly implemented.  It's closer to the standard
than IE is, but with the missing functionality I'm not sure which is
really easier to code for.  I don't know if Konqueror is crippled in
the same way, but I would suspect it is.  (I've not gotten any decent
Ajax to work in Konqueror myself, although I freely admit to not being
an expert on the subject.)




Crell



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




------------------------------------------------------------------------

Tue, 09 Aug 2005 15:38:19 +0000 : m3avrck

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.




------------------------------------------------------------------------

Wed, 10 Aug 2005 21:44:18 +0000 : drumm

+1


Works as expected and it degrades well (at first I didn't do a hard
refresh and my browser wasn't seeing the new js or css).




------------------------------------------------------------------------

Wed, 17 Aug 2005 00:41:14 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_0.patch (14.6 KB)

Keeping up to date. Still needs confirmed testing on KHTML browsers like
Konqueror and Safari, please.




------------------------------------------------------------------------

Wed, 17 Aug 2005 15:58:07 +0000 : Gabriel Radic

I'd be happy to test on Safari (1.2 and 2/1.3) and other WebKit browsers
and report the results here, but I can't apply the patch on any of my
installs (too risky) and don't have the time to set up a new Drupal
installation.


Steven, any chance for you to share a web page for me to test?


Thanks.




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:09:43 +0000 : stefan nagtegaal

Maybe it is me, but Steven why do you bother about the fact that this
will work on Safari? All other JS/AJAX we currently have in core
doesn't work with Safari...


Is it something special, or is t just curiocity?


Stefan




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:35:43 +0000 : Steven

Mostly because I can't test it, and Javascript is a fidgety business.
But I have to admit I was surprised when I found out (recently!) that
Drupal JS doesn't work in Safari. The features we check for are all
basic W3C DOM stuff, I assumed Safari would support them.


Do we know why Safari doesn't work?





--===============1568088059356025725==--


From drupal-devel@drupal.org Wed Aug 17 21:31:56 2005
From: stefan nagtegaal <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Wed, 17 Aug 2005 21:31:58 +0000
Message-ID: <type=project&nid=28483&cid=39659&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============2929614149535179747=="

--===============2929614149535179747==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      cvs
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   stefan nagtegaal
 Status:       patch (code needs review)

Okay, I found out how to debug with Safari..


When I goto: '?q=node/add/$type', I get these errors:
- SyntaxError - Parse error (line 37)
  drupal.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 3)
  collapse.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 4)
  autocomplete.js


Maybe you know what's wrong or what todo about it?




stefan nagtegaal



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




------------------------------------------------------------------------

Tue, 09 Aug 2005 15:38:19 +0000 : m3avrck

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.




------------------------------------------------------------------------

Wed, 10 Aug 2005 21:44:18 +0000 : drumm

+1


Works as expected and it degrades well (at first I didn't do a hard
refresh and my browser wasn't seeing the new js or css).




------------------------------------------------------------------------

Wed, 17 Aug 2005 00:41:14 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_0.patch (14.6 KB)

Keeping up to date. Still needs confirmed testing on KHTML browsers like
Konqueror and Safari, please.




------------------------------------------------------------------------

Wed, 17 Aug 2005 15:58:07 +0000 : Gabriel Radic

I'd be happy to test on Safari (1.2 and 2/1.3) and other WebKit browsers
and report the results here, but I can't apply the patch on any of my
installs (too risky) and don't have the time to set up a new Drupal
installation.


Steven, any chance for you to share a web page for me to test?


Thanks.




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:09:43 +0000 : stefan nagtegaal

Maybe it is me, but Steven why do you bother about the fact that this
will work on Safari? All other JS/AJAX we currently have in core
doesn't work with Safari...


Is it something special, or is t just curiocity?


Stefan




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:35:43 +0000 : Steven

Mostly because I can't test it, and Javascript is a fidgety business.
But I have to admit I was surprised when I found out (recently!) that
Drupal JS doesn't work in Safari. The features we check for are all
basic W3C DOM stuff, I assumed Safari would support them.


Do we know why Safari doesn't work?




------------------------------------------------------------------------

Wed, 17 Aug 2005 19:44:23 +0000 : Crell

I'm not sure about Drupal in particular, but Safari is missing a LOT Of
W3C DOM "stuff".  Many functions are there, but stubbed out and do
nothing.  Others are badly implemented.  It's closer to the standard
than IE is, but with the missing functionality I'm not sure which is
really easier to code for.  I don't know if Konqueror is crippled in
the same way, but I would suspect it is.  (I've not gotten any decent
Ajax to work in Konqueror myself, although I freely admit to not being
an expert on the subject.)





--===============2929614149535179747==--


From drupal-devel@drupal.org Thu Aug 18 08:49:03 2005
From: Thox <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Thu, 18 Aug 2005 08:49:03 +0000
Message-ID: <type=project&nid=28483&cid=39706&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============5122860418994619731=="

--===============5122860418994619731==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      cvs
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   Thox
-Status:       patch (code needs review)
+Status:       patch (code needs work)

After successfully attaching a file, the user can navigate away from the
page without pressing submit. The attachment shows up in the table, but
in this case the attachment isn't linked to the node.




Thox



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




------------------------------------------------------------------------

Tue, 09 Aug 2005 15:38:19 +0000 : m3avrck

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.




------------------------------------------------------------------------

Wed, 10 Aug 2005 21:44:18 +0000 : drumm

+1


Works as expected and it degrades well (at first I didn't do a hard
refresh and my browser wasn't seeing the new js or css).




------------------------------------------------------------------------

Wed, 17 Aug 2005 00:41:14 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_0.patch (14.6 KB)

Keeping up to date. Still needs confirmed testing on KHTML browsers like
Konqueror and Safari, please.




------------------------------------------------------------------------

Wed, 17 Aug 2005 15:58:07 +0000 : Gabriel Radic

I'd be happy to test on Safari (1.2 and 2/1.3) and other WebKit browsers
and report the results here, but I can't apply the patch on any of my
installs (too risky) and don't have the time to set up a new Drupal
installation.


Steven, any chance for you to share a web page for me to test?


Thanks.




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:09:43 +0000 : stefan nagtegaal

Maybe it is me, but Steven why do you bother about the fact that this
will work on Safari? All other JS/AJAX we currently have in core
doesn't work with Safari...


Is it something special, or is t just curiocity?


Stefan




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:35:43 +0000 : Steven

Mostly because I can't test it, and Javascript is a fidgety business.
But I have to admit I was surprised when I found out (recently!) that
Drupal JS doesn't work in Safari. The features we check for are all
basic W3C DOM stuff, I assumed Safari would support them.


Do we know why Safari doesn't work?




------------------------------------------------------------------------

Wed, 17 Aug 2005 19:44:23 +0000 : Crell

I'm not sure about Drupal in particular, but Safari is missing a LOT Of
W3C DOM "stuff".  Many functions are there, but stubbed out and do
nothing.  Others are badly implemented.  It's closer to the standard
than IE is, but with the missing functionality I'm not sure which is
really easier to code for.  I don't know if Konqueror is crippled in
the same way, but I would suspect it is.  (I've not gotten any decent
Ajax to work in Konqueror myself, although I freely admit to not being
an expert on the subject.)




------------------------------------------------------------------------

Wed, 17 Aug 2005 21:31:50 +0000 : stefan nagtegaal

Okay, I found out how to debug with Safari..


When I goto: '?q=node/add/$type', I get these errors:
- SyntaxError - Parse error (line 37)
  drupal.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 3)
  collapse.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 4)
  autocomplete.js


Maybe you know what's wrong or what todo about it?





--===============5122860418994619731==--


From drupal-devel@drupal.org Thu Aug 18 14:17:07 2005
From: m3avrck <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Thu, 18 Aug 2005 14:17:08 +0000
Message-ID: <type=project&nid=28483&cid=39732&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============6919197087864705315=="

--===============6919197087864705315==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      cvs
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   m3avrck
 Status:       patch (code needs work)

Now I haven't had time to play with this patch yet, but that last
comment gave me an idea.


Is there a JS check to see if the user is trying to navigate away from
the page? For example, they are uploading a file and click a link in
another app, all of a sudden that upload page they are on is replaced
by some other page. A simple JS check for this saying "Do you want to
leave this page and cancel the upload?" would certainly be a great
usability boost and prevent this from happening.




m3avrck



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




------------------------------------------------------------------------

Tue, 09 Aug 2005 15:38:19 +0000 : m3avrck

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.




------------------------------------------------------------------------

Wed, 10 Aug 2005 21:44:18 +0000 : drumm

+1


Works as expected and it degrades well (at first I didn't do a hard
refresh and my browser wasn't seeing the new js or css).




------------------------------------------------------------------------

Wed, 17 Aug 2005 00:41:14 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_0.patch (14.6 KB)

Keeping up to date. Still needs confirmed testing on KHTML browsers like
Konqueror and Safari, please.




------------------------------------------------------------------------

Wed, 17 Aug 2005 15:58:07 +0000 : Gabriel Radic

I'd be happy to test on Safari (1.2 and 2/1.3) and other WebKit browsers
and report the results here, but I can't apply the patch on any of my
installs (too risky) and don't have the time to set up a new Drupal
installation.


Steven, any chance for you to share a web page for me to test?


Thanks.




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:09:43 +0000 : stefan nagtegaal

Maybe it is me, but Steven why do you bother about the fact that this
will work on Safari? All other JS/AJAX we currently have in core
doesn't work with Safari...


Is it something special, or is t just curiocity?


Stefan




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:35:43 +0000 : Steven

Mostly because I can't test it, and Javascript is a fidgety business.
But I have to admit I was surprised when I found out (recently!) that
Drupal JS doesn't work in Safari. The features we check for are all
basic W3C DOM stuff, I assumed Safari would support them.


Do we know why Safari doesn't work?




------------------------------------------------------------------------

Wed, 17 Aug 2005 19:44:23 +0000 : Crell

I'm not sure about Drupal in particular, but Safari is missing a LOT Of
W3C DOM "stuff".  Many functions are there, but stubbed out and do
nothing.  Others are badly implemented.  It's closer to the standard
than IE is, but with the missing functionality I'm not sure which is
really easier to code for.  I don't know if Konqueror is crippled in
the same way, but I would suspect it is.  (I've not gotten any decent
Ajax to work in Konqueror myself, although I freely admit to not being
an expert on the subject.)




------------------------------------------------------------------------

Wed, 17 Aug 2005 21:31:50 +0000 : stefan nagtegaal

Okay, I found out how to debug with Safari..


When I goto: '?q=node/add/$type', I get these errors:
- SyntaxError - Parse error (line 37)
  drupal.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 3)
  collapse.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 4)
  autocomplete.js


Maybe you know what's wrong or what todo about it?




------------------------------------------------------------------------

Thu, 18 Aug 2005 08:49:00 +0000 : Thox

After successfully attaching a file, the user can navigate away from the
page without pressing submit. The attachment shows up in the table, but
in this case the attachment isn't linked to the node.





--===============6919197087864705315==--


From drupal-devel@drupal.org Thu Aug 18 14:54:04 2005
From: tostinni <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Thu, 18 Aug 2005 14:54:04 +0000
Message-ID: <type=project&nid=28483&cid=39740&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============7111553919928585738=="

--===============7111553919928585738==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      cvs
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   tostinni
 Status:       patch (code needs work)

Retaking this idea of controlling if user is living a page or not, I
recently saw the new IPB permission panel demo, it makes an intense use
of AJAX and provide a way to keep track if the user is living the page.
See the demo here [1].
[1] http://blog.mattmecham.com/archives/2005/07/ipb_21_acp_perm.html




tostinni



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [2]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [3] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[2] http://www.acko.net/dumpx/jsupload.png
[3] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




------------------------------------------------------------------------

Tue, 09 Aug 2005 15:38:19 +0000 : m3avrck

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.




------------------------------------------------------------------------

Wed, 10 Aug 2005 21:44:18 +0000 : drumm

+1


Works as expected and it degrades well (at first I didn't do a hard
refresh and my browser wasn't seeing the new js or css).




------------------------------------------------------------------------

Wed, 17 Aug 2005 00:41:14 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_0.patch (14.6 KB)

Keeping up to date. Still needs confirmed testing on KHTML browsers like
Konqueror and Safari, please.




------------------------------------------------------------------------

Wed, 17 Aug 2005 15:58:07 +0000 : Gabriel Radic

I'd be happy to test on Safari (1.2 and 2/1.3) and other WebKit browsers
and report the results here, but I can't apply the patch on any of my
installs (too risky) and don't have the time to set up a new Drupal
installation.


Steven, any chance for you to share a web page for me to test?


Thanks.




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:09:43 +0000 : stefan nagtegaal

Maybe it is me, but Steven why do you bother about the fact that this
will work on Safari? All other JS/AJAX we currently have in core
doesn't work with Safari...


Is it something special, or is t just curiocity?


Stefan




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:35:43 +0000 : Steven

Mostly because I can't test it, and Javascript is a fidgety business.
But I have to admit I was surprised when I found out (recently!) that
Drupal JS doesn't work in Safari. The features we check for are all
basic W3C DOM stuff, I assumed Safari would support them.


Do we know why Safari doesn't work?




------------------------------------------------------------------------

Wed, 17 Aug 2005 19:44:23 +0000 : Crell

I'm not sure about Drupal in particular, but Safari is missing a LOT Of
W3C DOM "stuff".  Many functions are there, but stubbed out and do
nothing.  Others are badly implemented.  It's closer to the standard
than IE is, but with the missing functionality I'm not sure which is
really easier to code for.  I don't know if Konqueror is crippled in
the same way, but I would suspect it is.  (I've not gotten any decent
Ajax to work in Konqueror myself, although I freely admit to not being
an expert on the subject.)




------------------------------------------------------------------------

Wed, 17 Aug 2005 21:31:50 +0000 : stefan nagtegaal

Okay, I found out how to debug with Safari..


When I goto: '?q=node/add/$type', I get these errors:
- SyntaxError - Parse error (line 37)
  drupal.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 3)
  collapse.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 4)
  autocomplete.js


Maybe you know what's wrong or what todo about it?




------------------------------------------------------------------------

Thu, 18 Aug 2005 08:49:00 +0000 : Thox

After successfully attaching a file, the user can navigate away from the
page without pressing submit. The attachment shows up in the table, but
in this case the attachment isn't linked to the node.




------------------------------------------------------------------------

Thu, 18 Aug 2005 14:17:01 +0000 : m3avrck

Now I haven't had time to play with this patch yet, but that last
comment gave me an idea.


Is there a JS check to see if the user is trying to navigate away from
the page? For example, they are uploading a file and click a link in
another app, all of a sudden that upload page they are on is replaced
by some other page. A simple JS check for this saying "Do you want to
leave this page and cancel the upload?" would certainly be a great
usability boost and prevent this from happening.





--===============7111553919928585738==--


From drupal-devel@drupal.org Thu Aug 18 15:29:33 2005
From: Steven <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Thu, 18 Aug 2005 15:29:34 +0000
Message-ID: <type=project&nid=28483&cid=39742&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============5854314444177710267=="

--===============5854314444177710267==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      cvs
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   Steven
-Status:       patch (code needs work)
+Status:       patch (code needs review)

Thox: this is by design, and no different from submitting a file by
doing a preview, and then navigating away from the edit page. "Changes
made to the attachments list will not be permanent until you save this
post."


The patch simply streamlines the workflow.




Steven



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




------------------------------------------------------------------------

Tue, 09 Aug 2005 15:38:19 +0000 : m3avrck

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.




------------------------------------------------------------------------

Wed, 10 Aug 2005 21:44:18 +0000 : drumm

+1


Works as expected and it degrades well (at first I didn't do a hard
refresh and my browser wasn't seeing the new js or css).




------------------------------------------------------------------------

Wed, 17 Aug 2005 00:41:14 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_0.patch (14.6 KB)

Keeping up to date. Still needs confirmed testing on KHTML browsers like
Konqueror and Safari, please.




------------------------------------------------------------------------

Wed, 17 Aug 2005 15:58:07 +0000 : Gabriel Radic

I'd be happy to test on Safari (1.2 and 2/1.3) and other WebKit browsers
and report the results here, but I can't apply the patch on any of my
installs (too risky) and don't have the time to set up a new Drupal
installation.


Steven, any chance for you to share a web page for me to test?


Thanks.




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:09:43 +0000 : stefan nagtegaal

Maybe it is me, but Steven why do you bother about the fact that this
will work on Safari? All other JS/AJAX we currently have in core
doesn't work with Safari...


Is it something special, or is t just curiocity?


Stefan




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:35:43 +0000 : Steven

Mostly because I can't test it, and Javascript is a fidgety business.
But I have to admit I was surprised when I found out (recently!) that
Drupal JS doesn't work in Safari. The features we check for are all
basic W3C DOM stuff, I assumed Safari would support them.


Do we know why Safari doesn't work?




------------------------------------------------------------------------

Wed, 17 Aug 2005 19:44:23 +0000 : Crell

I'm not sure about Drupal in particular, but Safari is missing a LOT Of
W3C DOM "stuff".  Many functions are there, but stubbed out and do
nothing.  Others are badly implemented.  It's closer to the standard
than IE is, but with the missing functionality I'm not sure which is
really easier to code for.  I don't know if Konqueror is crippled in
the same way, but I would suspect it is.  (I've not gotten any decent
Ajax to work in Konqueror myself, although I freely admit to not being
an expert on the subject.)




------------------------------------------------------------------------

Wed, 17 Aug 2005 21:31:50 +0000 : stefan nagtegaal

Okay, I found out how to debug with Safari..


When I goto: '?q=node/add/$type', I get these errors:
- SyntaxError - Parse error (line 37)
  drupal.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 3)
  collapse.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 4)
  autocomplete.js


Maybe you know what's wrong or what todo about it?




------------------------------------------------------------------------

Thu, 18 Aug 2005 08:49:00 +0000 : Thox

After successfully attaching a file, the user can navigate away from the
page without pressing submit. The attachment shows up in the table, but
in this case the attachment isn't linked to the node.




------------------------------------------------------------------------

Thu, 18 Aug 2005 14:17:01 +0000 : m3avrck

Now I haven't had time to play with this patch yet, but that last
comment gave me an idea.


Is there a JS check to see if the user is trying to navigate away from
the page? For example, they are uploading a file and click a link in
another app, all of a sudden that upload page they are on is replaced
by some other page. A simple JS check for this saying "Do you want to
leave this page and cancel the upload?" would certainly be a great
usability boost and prevent this from happening.




------------------------------------------------------------------------

Thu, 18 Aug 2005 14:53:58 +0000 : tostinni

Retaking this idea of controlling if user is living a page or not, I
recently saw the new IPB permission panel demo, it makes an intense use
of AJAX and provide a way to keep track if the user is living the page.
See the demo here [3].
[3] http://blog.mattmecham.com/archives/2005/07/ipb_21_acp_perm.html





--===============5854314444177710267==--


From drupal-devel@drupal.org Thu Aug 18 22:03:13 2005
From: Dries <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Thu, 18 Aug 2005 22:03:28 +0000
Message-ID: <type=project&nid=28483&cid=39804&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============0394958314802031784=="

--===============0394958314802031784==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      cvs
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   Dries
-Status:       patch (code needs review)
+Status:       patch (code needs work)

Patch doesn't apply.


(BTW, is it me or is the progress bar somewhat toy-ish and somewhat
flashy?)




Dries



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




------------------------------------------------------------------------

Tue, 09 Aug 2005 15:38:19 +0000 : m3avrck

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.




------------------------------------------------------------------------

Wed, 10 Aug 2005 21:44:18 +0000 : drumm

+1


Works as expected and it degrades well (at first I didn't do a hard
refresh and my browser wasn't seeing the new js or css).




------------------------------------------------------------------------

Wed, 17 Aug 2005 00:41:14 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_0.patch (14.6 KB)

Keeping up to date. Still needs confirmed testing on KHTML browsers like
Konqueror and Safari, please.




------------------------------------------------------------------------

Wed, 17 Aug 2005 15:58:07 +0000 : Gabriel Radic

I'd be happy to test on Safari (1.2 and 2/1.3) and other WebKit browsers
and report the results here, but I can't apply the patch on any of my
installs (too risky) and don't have the time to set up a new Drupal
installation.


Steven, any chance for you to share a web page for me to test?


Thanks.




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:09:43 +0000 : stefan nagtegaal

Maybe it is me, but Steven why do you bother about the fact that this
will work on Safari? All other JS/AJAX we currently have in core
doesn't work with Safari...


Is it something special, or is t just curiocity?


Stefan




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:35:43 +0000 : Steven

Mostly because I can't test it, and Javascript is a fidgety business.
But I have to admit I was surprised when I found out (recently!) that
Drupal JS doesn't work in Safari. The features we check for are all
basic W3C DOM stuff, I assumed Safari would support them.


Do we know why Safari doesn't work?




------------------------------------------------------------------------

Wed, 17 Aug 2005 19:44:23 +0000 : Crell

I'm not sure about Drupal in particular, but Safari is missing a LOT Of
W3C DOM "stuff".  Many functions are there, but stubbed out and do
nothing.  Others are badly implemented.  It's closer to the standard
than IE is, but with the missing functionality I'm not sure which is
really easier to code for.  I don't know if Konqueror is crippled in
the same way, but I would suspect it is.  (I've not gotten any decent
Ajax to work in Konqueror myself, although I freely admit to not being
an expert on the subject.)




------------------------------------------------------------------------

Wed, 17 Aug 2005 21:31:50 +0000 : stefan nagtegaal

Okay, I found out how to debug with Safari..


When I goto: '?q=node/add/$type', I get these errors:
- SyntaxError - Parse error (line 37)
  drupal.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 3)
  collapse.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 4)
  autocomplete.js


Maybe you know what's wrong or what todo about it?




------------------------------------------------------------------------

Thu, 18 Aug 2005 08:49:00 +0000 : Thox

After successfully attaching a file, the user can navigate away from the
page without pressing submit. The attachment shows up in the table, but
in this case the attachment isn't linked to the node.




------------------------------------------------------------------------

Thu, 18 Aug 2005 14:17:01 +0000 : m3avrck

Now I haven't had time to play with this patch yet, but that last
comment gave me an idea.


Is there a JS check to see if the user is trying to navigate away from
the page? For example, they are uploading a file and click a link in
another app, all of a sudden that upload page they are on is replaced
by some other page. A simple JS check for this saying "Do you want to
leave this page and cancel the upload?" would certainly be a great
usability boost and prevent this from happening.




------------------------------------------------------------------------

Thu, 18 Aug 2005 14:53:58 +0000 : tostinni

Retaking this idea of controlling if user is living a page or not, I
recently saw the new IPB permission panel demo, it makes an intense use
of AJAX and provide a way to keep track if the user is living the page.
See the demo here [3].
[3] http://blog.mattmecham.com/archives/2005/07/ipb_21_acp_perm.html




------------------------------------------------------------------------

Thu, 18 Aug 2005 15:29:27 +0000 : Steven

Thox: this is by design, and no different from submitting a file by
doing a preview, and then navigating away from the edit page. "Changes
made to the attachments list will not be permanent until you save this
post."


The patch simply streamlines the workflow.





--===============0394958314802031784==--


From drupal-devel@drupal.org Fri Aug 19 08:40:18 2005
From: Thox <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Fri, 19 Aug 2005 08:40:21 +0000
Message-ID: <type=project&nid=28483&cid=39852&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============8311458313862908634=="

--===============8311458313862908634==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      cvs
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   Thox
 Status:       patch (code needs work)

When attaching two or more files with the same without pressing submit,
the original file is overwritten and the table shows two files with the
same name. After pressing submit, the table shows one file.




Thox



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




------------------------------------------------------------------------

Tue, 09 Aug 2005 15:38:19 +0000 : m3avrck

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.




------------------------------------------------------------------------

Wed, 10 Aug 2005 21:44:18 +0000 : drumm

+1


Works as expected and it degrades well (at first I didn't do a hard
refresh and my browser wasn't seeing the new js or css).




------------------------------------------------------------------------

Wed, 17 Aug 2005 00:41:14 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_0.patch (14.6 KB)

Keeping up to date. Still needs confirmed testing on KHTML browsers like
Konqueror and Safari, please.




------------------------------------------------------------------------

Wed, 17 Aug 2005 15:58:07 +0000 : Gabriel Radic

I'd be happy to test on Safari (1.2 and 2/1.3) and other WebKit browsers
and report the results here, but I can't apply the patch on any of my
installs (too risky) and don't have the time to set up a new Drupal
installation.


Steven, any chance for you to share a web page for me to test?


Thanks.




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:09:43 +0000 : stefan nagtegaal

Maybe it is me, but Steven why do you bother about the fact that this
will work on Safari? All other JS/AJAX we currently have in core
doesn't work with Safari...


Is it something special, or is t just curiocity?


Stefan




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:35:43 +0000 : Steven

Mostly because I can't test it, and Javascript is a fidgety business.
But I have to admit I was surprised when I found out (recently!) that
Drupal JS doesn't work in Safari. The features we check for are all
basic W3C DOM stuff, I assumed Safari would support them.


Do we know why Safari doesn't work?




------------------------------------------------------------------------

Wed, 17 Aug 2005 19:44:23 +0000 : Crell

I'm not sure about Drupal in particular, but Safari is missing a LOT Of
W3C DOM "stuff".  Many functions are there, but stubbed out and do
nothing.  Others are badly implemented.  It's closer to the standard
than IE is, but with the missing functionality I'm not sure which is
really easier to code for.  I don't know if Konqueror is crippled in
the same way, but I would suspect it is.  (I've not gotten any decent
Ajax to work in Konqueror myself, although I freely admit to not being
an expert on the subject.)




------------------------------------------------------------------------

Wed, 17 Aug 2005 21:31:50 +0000 : stefan nagtegaal

Okay, I found out how to debug with Safari..


When I goto: '?q=node/add/$type', I get these errors:
- SyntaxError - Parse error (line 37)
  drupal.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 3)
  collapse.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 4)
  autocomplete.js


Maybe you know what's wrong or what todo about it?




------------------------------------------------------------------------

Thu, 18 Aug 2005 08:49:00 +0000 : Thox

After successfully attaching a file, the user can navigate away from the
page without pressing submit. The attachment shows up in the table, but
in this case the attachment isn't linked to the node.




------------------------------------------------------------------------

Thu, 18 Aug 2005 14:17:01 +0000 : m3avrck

Now I haven't had time to play with this patch yet, but that last
comment gave me an idea.


Is there a JS check to see if the user is trying to navigate away from
the page? For example, they are uploading a file and click a link in
another app, all of a sudden that upload page they are on is replaced
by some other page. A simple JS check for this saying "Do you want to
leave this page and cancel the upload?" would certainly be a great
usability boost and prevent this from happening.




------------------------------------------------------------------------

Thu, 18 Aug 2005 14:53:58 +0000 : tostinni

Retaking this idea of controlling if user is living a page or not, I
recently saw the new IPB permission panel demo, it makes an intense use
of AJAX and provide a way to keep track if the user is living the page.
See the demo here [3].
[3] http://blog.mattmecham.com/archives/2005/07/ipb_21_acp_perm.html




------------------------------------------------------------------------

Thu, 18 Aug 2005 15:29:27 +0000 : Steven

Thox: this is by design, and no different from submitting a file by
doing a preview, and then navigating away from the edit page. "Changes
made to the attachments list will not be permanent until you save this
post."


The patch simply streamlines the workflow.




------------------------------------------------------------------------

Thu, 18 Aug 2005 22:02:17 +0000 : Dries

Patch doesn't apply.


(BTW, is it me or is the progress bar somewhat toy-ish and somewhat
flashy?)





--===============8311458313862908634==--


From drupal-devel@drupal.org Fri Aug 19 13:08:08 2005
From: Steven <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Fri, 19 Aug 2005 13:08:08 +0000
Message-ID: <type=project&nid=28483&cid=39881&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============1930133313516331255=="

--===============1930133313516331255==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      cvs
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   Steven
-Status:       patch (code needs work)
+Status:       patch (code needs review)
 Attachment:   http://drupal.org/files/issues/jsupload_1.patch (14.63 KB)

Thox: this is also an existing behaviour/bug in upload.module. This
patch does not affect any of the underlying file saving or storage
mechanisms.


It simply introduces an asynchronous form submission and result
display, but otherwise uses exactly the same code as the non-JS
version.


Attached is an up-to-date, but otherwise unchanged version.




Steven



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




------------------------------------------------------------------------

Tue, 09 Aug 2005 15:38:19 +0000 : m3avrck

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.




------------------------------------------------------------------------

Wed, 10 Aug 2005 21:44:18 +0000 : drumm

+1


Works as expected and it degrades well (at first I didn't do a hard
refresh and my browser wasn't seeing the new js or css).




------------------------------------------------------------------------

Wed, 17 Aug 2005 00:41:14 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_0.patch (14.6 KB)

Keeping up to date. Still needs confirmed testing on KHTML browsers like
Konqueror and Safari, please.




------------------------------------------------------------------------

Wed, 17 Aug 2005 15:58:07 +0000 : Gabriel Radic

I'd be happy to test on Safari (1.2 and 2/1.3) and other WebKit browsers
and report the results here, but I can't apply the patch on any of my
installs (too risky) and don't have the time to set up a new Drupal
installation.


Steven, any chance for you to share a web page for me to test?


Thanks.




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:09:43 +0000 : stefan nagtegaal

Maybe it is me, but Steven why do you bother about the fact that this
will work on Safari? All other JS/AJAX we currently have in core
doesn't work with Safari...


Is it something special, or is t just curiocity?


Stefan




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:35:43 +0000 : Steven

Mostly because I can't test it, and Javascript is a fidgety business.
But I have to admit I was surprised when I found out (recently!) that
Drupal JS doesn't work in Safari. The features we check for are all
basic W3C DOM stuff, I assumed Safari would support them.


Do we know why Safari doesn't work?




------------------------------------------------------------------------

Wed, 17 Aug 2005 19:44:23 +0000 : Crell

I'm not sure about Drupal in particular, but Safari is missing a LOT Of
W3C DOM "stuff".  Many functions are there, but stubbed out and do
nothing.  Others are badly implemented.  It's closer to the standard
than IE is, but with the missing functionality I'm not sure which is
really easier to code for.  I don't know if Konqueror is crippled in
the same way, but I would suspect it is.  (I've not gotten any decent
Ajax to work in Konqueror myself, although I freely admit to not being
an expert on the subject.)




------------------------------------------------------------------------

Wed, 17 Aug 2005 21:31:50 +0000 : stefan nagtegaal

Okay, I found out how to debug with Safari..


When I goto: '?q=node/add/$type', I get these errors:
- SyntaxError - Parse error (line 37)
  drupal.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 3)
  collapse.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 4)
  autocomplete.js


Maybe you know what's wrong or what todo about it?




------------------------------------------------------------------------

Thu, 18 Aug 2005 08:49:00 +0000 : Thox

After successfully attaching a file, the user can navigate away from the
page without pressing submit. The attachment shows up in the table, but
in this case the attachment isn't linked to the node.




------------------------------------------------------------------------

Thu, 18 Aug 2005 14:17:01 +0000 : m3avrck

Now I haven't had time to play with this patch yet, but that last
comment gave me an idea.


Is there a JS check to see if the user is trying to navigate away from
the page? For example, they are uploading a file and click a link in
another app, all of a sudden that upload page they are on is replaced
by some other page. A simple JS check for this saying "Do you want to
leave this page and cancel the upload?" would certainly be a great
usability boost and prevent this from happening.




------------------------------------------------------------------------

Thu, 18 Aug 2005 14:53:58 +0000 : tostinni

Retaking this idea of controlling if user is living a page or not, I
recently saw the new IPB permission panel demo, it makes an intense use
of AJAX and provide a way to keep track if the user is living the page.
See the demo here [3].
[3] http://blog.mattmecham.com/archives/2005/07/ipb_21_acp_perm.html




------------------------------------------------------------------------

Thu, 18 Aug 2005 15:29:27 +0000 : Steven

Thox: this is by design, and no different from submitting a file by
doing a preview, and then navigating away from the edit page. "Changes
made to the attachments list will not be permanent until you save this
post."


The patch simply streamlines the workflow.




------------------------------------------------------------------------

Thu, 18 Aug 2005 22:02:17 +0000 : Dries

Patch doesn't apply.


(BTW, is it me or is the progress bar somewhat toy-ish and somewhat
flashy?)




------------------------------------------------------------------------

Fri, 19 Aug 2005 08:39:58 +0000 : Thox

When attaching two or more files with the same without pressing submit,
the original file is overwritten and the table shows two files with the
same name. After pressing submit, the table shows one file.





--===============1930133313516331255==--


From drupal-devel@drupal.org Fri Aug 19 21:12:16 2005
From: =?utf-8?q?B=C3=A8r?= Kessels <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Fri, 19 Aug 2005 21:12:17 +0000
Message-ID: <type=project&nid=28483&cid=39933&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============6134861187325390113=="

--===============6134861187325390113==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      cvs
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   Bèr Kessels
 Status:       patch (code needs review)

Steven: I will test this on KHTML, but might need some help, for I am
very limited in time (read: reallyy have no time for testing). HOwever:
in contrary to what Stefan says, the current ajax/js works well on konq.
AND on safari (and beleive me I have a hell of a testing environment :)
[1])
[1] http://bler.webschuur.com/big_bash




Bèr Kessels



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [2]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [3] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[2] http://www.acko.net/dumpx/jsupload.png
[3] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




------------------------------------------------------------------------

Tue, 09 Aug 2005 15:38:19 +0000 : m3avrck

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.




------------------------------------------------------------------------

Wed, 10 Aug 2005 21:44:18 +0000 : drumm

+1


Works as expected and it degrades well (at first I didn't do a hard
refresh and my browser wasn't seeing the new js or css).




------------------------------------------------------------------------

Wed, 17 Aug 2005 00:41:14 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_0.patch (14.6 KB)

Keeping up to date. Still needs confirmed testing on KHTML browsers like
Konqueror and Safari, please.




------------------------------------------------------------------------

Wed, 17 Aug 2005 15:58:07 +0000 : Gabriel Radic

I'd be happy to test on Safari (1.2 and 2/1.3) and other WebKit browsers
and report the results here, but I can't apply the patch on any of my
installs (too risky) and don't have the time to set up a new Drupal
installation.


Steven, any chance for you to share a web page for me to test?


Thanks.




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:09:43 +0000 : stefan nagtegaal

Maybe it is me, but Steven why do you bother about the fact that this
will work on Safari? All other JS/AJAX we currently have in core
doesn't work with Safari...


Is it something special, or is t just curiocity?


Stefan




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:35:43 +0000 : Steven

Mostly because I can't test it, and Javascript is a fidgety business.
But I have to admit I was surprised when I found out (recently!) that
Drupal JS doesn't work in Safari. The features we check for are all
basic W3C DOM stuff, I assumed Safari would support them.


Do we know why Safari doesn't work?




------------------------------------------------------------------------

Wed, 17 Aug 2005 19:44:23 +0000 : Crell

I'm not sure about Drupal in particular, but Safari is missing a LOT Of
W3C DOM "stuff".  Many functions are there, but stubbed out and do
nothing.  Others are badly implemented.  It's closer to the standard
than IE is, but with the missing functionality I'm not sure which is
really easier to code for.  I don't know if Konqueror is crippled in
the same way, but I would suspect it is.  (I've not gotten any decent
Ajax to work in Konqueror myself, although I freely admit to not being
an expert on the subject.)




------------------------------------------------------------------------

Wed, 17 Aug 2005 21:31:50 +0000 : stefan nagtegaal

Okay, I found out how to debug with Safari..


When I goto: '?q=node/add/$type', I get these errors:
- SyntaxError - Parse error (line 37)
  drupal.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 3)
  collapse.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 4)
  autocomplete.js


Maybe you know what's wrong or what todo about it?




------------------------------------------------------------------------

Thu, 18 Aug 2005 08:49:00 +0000 : Thox

After successfully attaching a file, the user can navigate away from the
page without pressing submit. The attachment shows up in the table, but
in this case the attachment isn't linked to the node.




------------------------------------------------------------------------

Thu, 18 Aug 2005 14:17:01 +0000 : m3avrck

Now I haven't had time to play with this patch yet, but that last
comment gave me an idea.


Is there a JS check to see if the user is trying to navigate away from
the page? For example, they are uploading a file and click a link in
another app, all of a sudden that upload page they are on is replaced
by some other page. A simple JS check for this saying "Do you want to
leave this page and cancel the upload?" would certainly be a great
usability boost and prevent this from happening.




------------------------------------------------------------------------

Thu, 18 Aug 2005 14:53:58 +0000 : tostinni

Retaking this idea of controlling if user is living a page or not, I
recently saw the new IPB permission panel demo, it makes an intense use
of AJAX and provide a way to keep track if the user is living the page.
See the demo here [4].
[4] http://blog.mattmecham.com/archives/2005/07/ipb_21_acp_perm.html




------------------------------------------------------------------------

Thu, 18 Aug 2005 15:29:27 +0000 : Steven

Thox: this is by design, and no different from submitting a file by
doing a preview, and then navigating away from the edit page. "Changes
made to the attachments list will not be permanent until you save this
post."


The patch simply streamlines the workflow.




------------------------------------------------------------------------

Thu, 18 Aug 2005 22:02:17 +0000 : Dries

Patch doesn't apply.


(BTW, is it me or is the progress bar somewhat toy-ish and somewhat
flashy?)




------------------------------------------------------------------------

Fri, 19 Aug 2005 08:39:58 +0000 : Thox

When attaching two or more files with the same without pressing submit,
the original file is overwritten and the table shows two files with the
same name. After pressing submit, the table shows one file.




------------------------------------------------------------------------

Fri, 19 Aug 2005 13:07:56 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_1.patch (14.63 KB)

Thox: this is also an existing behaviour/bug in upload.module. This
patch does not affect any of the underlying file saving or storage
mechanisms.


It simply introduces an asynchronous form submission and result
display, but otherwise uses exactly the same code as the non-JS
version.


Attached is an up-to-date, but otherwise unchanged version.





--===============6134861187325390113==--


From drupal-devel@drupal.org Mon Aug 29 19:17:28 2005
From: m3avrck <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Mon, 29 Aug 2005 19:17:28 +0000
Message-ID: <type=project&nid=28483&cid=41570&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============3412974158425150376=="

--===============3412974158425150376==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      cvs
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   m3avrck
 Status:       patch (code needs review)

What's the status of this? Can we get this into core before the freeze,
seems to be working great for me!




m3avrck



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




------------------------------------------------------------------------

Tue, 09 Aug 2005 15:38:19 +0000 : m3avrck

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.




------------------------------------------------------------------------

Wed, 10 Aug 2005 21:44:18 +0000 : drumm

+1


Works as expected and it degrades well (at first I didn't do a hard
refresh and my browser wasn't seeing the new js or css).




------------------------------------------------------------------------

Wed, 17 Aug 2005 00:41:14 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_0.patch (14.6 KB)

Keeping up to date. Still needs confirmed testing on KHTML browsers like
Konqueror and Safari, please.




------------------------------------------------------------------------

Wed, 17 Aug 2005 15:58:07 +0000 : Gabriel Radic

I'd be happy to test on Safari (1.2 and 2/1.3) and other WebKit browsers
and report the results here, but I can't apply the patch on any of my
installs (too risky) and don't have the time to set up a new Drupal
installation.


Steven, any chance for you to share a web page for me to test?


Thanks.




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:09:43 +0000 : stefan nagtegaal

Maybe it is me, but Steven why do you bother about the fact that this
will work on Safari? All other JS/AJAX we currently have in core
doesn't work with Safari...


Is it something special, or is t just curiocity?


Stefan




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:35:43 +0000 : Steven

Mostly because I can't test it, and Javascript is a fidgety business.
But I have to admit I was surprised when I found out (recently!) that
Drupal JS doesn't work in Safari. The features we check for are all
basic W3C DOM stuff, I assumed Safari would support them.


Do we know why Safari doesn't work?




------------------------------------------------------------------------

Wed, 17 Aug 2005 19:44:23 +0000 : Crell

I'm not sure about Drupal in particular, but Safari is missing a LOT Of
W3C DOM "stuff".  Many functions are there, but stubbed out and do
nothing.  Others are badly implemented.  It's closer to the standard
than IE is, but with the missing functionality I'm not sure which is
really easier to code for.  I don't know if Konqueror is crippled in
the same way, but I would suspect it is.  (I've not gotten any decent
Ajax to work in Konqueror myself, although I freely admit to not being
an expert on the subject.)




------------------------------------------------------------------------

Wed, 17 Aug 2005 21:31:50 +0000 : stefan nagtegaal

Okay, I found out how to debug with Safari..


When I goto: '?q=node/add/$type', I get these errors:
- SyntaxError - Parse error (line 37)
  drupal.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 3)
  collapse.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 4)
  autocomplete.js


Maybe you know what's wrong or what todo about it?




------------------------------------------------------------------------

Thu, 18 Aug 2005 08:49:00 +0000 : Thox

After successfully attaching a file, the user can navigate away from the
page without pressing submit. The attachment shows up in the table, but
in this case the attachment isn't linked to the node.




------------------------------------------------------------------------

Thu, 18 Aug 2005 14:17:01 +0000 : m3avrck

Now I haven't had time to play with this patch yet, but that last
comment gave me an idea.


Is there a JS check to see if the user is trying to navigate away from
the page? For example, they are uploading a file and click a link in
another app, all of a sudden that upload page they are on is replaced
by some other page. A simple JS check for this saying "Do you want to
leave this page and cancel the upload?" would certainly be a great
usability boost and prevent this from happening.




------------------------------------------------------------------------

Thu, 18 Aug 2005 14:53:58 +0000 : tostinni

Retaking this idea of controlling if user is living a page or not, I
recently saw the new IPB permission panel demo, it makes an intense use
of AJAX and provide a way to keep track if the user is living the page.
See the demo here [3].
[3] http://blog.mattmecham.com/archives/2005/07/ipb_21_acp_perm.html




------------------------------------------------------------------------

Thu, 18 Aug 2005 15:29:27 +0000 : Steven

Thox: this is by design, and no different from submitting a file by
doing a preview, and then navigating away from the edit page. "Changes
made to the attachments list will not be permanent until you save this
post."


The patch simply streamlines the workflow.




------------------------------------------------------------------------

Thu, 18 Aug 2005 22:02:17 +0000 : Dries

Patch doesn't apply.


(BTW, is it me or is the progress bar somewhat toy-ish and somewhat
flashy?)




------------------------------------------------------------------------

Fri, 19 Aug 2005 08:39:58 +0000 : Thox

When attaching two or more files with the same without pressing submit,
the original file is overwritten and the table shows two files with the
same name. After pressing submit, the table shows one file.




------------------------------------------------------------------------

Fri, 19 Aug 2005 13:07:56 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_1.patch (14.63 KB)

Thox: this is also an existing behaviour/bug in upload.module. This
patch does not affect any of the underlying file saving or storage
mechanisms.


It simply introduces an asynchronous form submission and result
display, but otherwise uses exactly the same code as the non-JS
version.


Attached is an up-to-date, but otherwise unchanged version.




------------------------------------------------------------------------

Fri, 19 Aug 2005 21:12:13 +0000 : Bèr Kessels

Steven: I will test this on KHTML, but might need some help, for I am
very limited in time (read: reallyy have no time for testing). HOwever:
in contrary to what Stefan says, the current ajax/js works well on konq.
AND on safari (and beleive me I have a hell of a testing environment :)
[4])
[4] http://bler.webschuur.com/big_bash





--===============3412974158425150376==--


From drupal-devel@drupal.org Wed Aug 31 08:18:10 2005
From: stefan nagtegaal <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Wed, 31 Aug 2005 08:18:10 +0000
Message-ID: <type=project&nid=28483&cid=41808&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============7618244483626616338=="

--===============7618244483626616338==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      cvs
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   stefan nagtegaal
 Status:       patch (code needs review)

I retested this patch and I can verify that it works the way it would
work on Safari..
Please apply..


BTW Ber: Macs just works! Only microsoft screws it!




stefan nagtegaal



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




------------------------------------------------------------------------

Tue, 09 Aug 2005 15:38:19 +0000 : m3avrck

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.




------------------------------------------------------------------------

Wed, 10 Aug 2005 21:44:18 +0000 : drumm

+1


Works as expected and it degrades well (at first I didn't do a hard
refresh and my browser wasn't seeing the new js or css).




------------------------------------------------------------------------

Wed, 17 Aug 2005 00:41:14 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_0.patch (14.6 KB)

Keeping up to date. Still needs confirmed testing on KHTML browsers like
Konqueror and Safari, please.




------------------------------------------------------------------------

Wed, 17 Aug 2005 15:58:07 +0000 : Gabriel Radic

I'd be happy to test on Safari (1.2 and 2/1.3) and other WebKit browsers
and report the results here, but I can't apply the patch on any of my
installs (too risky) and don't have the time to set up a new Drupal
installation.


Steven, any chance for you to share a web page for me to test?


Thanks.




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:09:43 +0000 : stefan nagtegaal

Maybe it is me, but Steven why do you bother about the fact that this
will work on Safari? All other JS/AJAX we currently have in core
doesn't work with Safari...


Is it something special, or is t just curiocity?


Stefan




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:35:43 +0000 : Steven

Mostly because I can't test it, and Javascript is a fidgety business.
But I have to admit I was surprised when I found out (recently!) that
Drupal JS doesn't work in Safari. The features we check for are all
basic W3C DOM stuff, I assumed Safari would support them.


Do we know why Safari doesn't work?




------------------------------------------------------------------------

Wed, 17 Aug 2005 19:44:23 +0000 : Crell

I'm not sure about Drupal in particular, but Safari is missing a LOT Of
W3C DOM "stuff".  Many functions are there, but stubbed out and do
nothing.  Others are badly implemented.  It's closer to the standard
than IE is, but with the missing functionality I'm not sure which is
really easier to code for.  I don't know if Konqueror is crippled in
the same way, but I would suspect it is.  (I've not gotten any decent
Ajax to work in Konqueror myself, although I freely admit to not being
an expert on the subject.)




------------------------------------------------------------------------

Wed, 17 Aug 2005 21:31:50 +0000 : stefan nagtegaal

Okay, I found out how to debug with Safari..


When I goto: '?q=node/add/$type', I get these errors:
- SyntaxError - Parse error (line 37)
  drupal.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 3)
  collapse.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 4)
  autocomplete.js


Maybe you know what's wrong or what todo about it?




------------------------------------------------------------------------

Thu, 18 Aug 2005 08:49:00 +0000 : Thox

After successfully attaching a file, the user can navigate away from the
page without pressing submit. The attachment shows up in the table, but
in this case the attachment isn't linked to the node.




------------------------------------------------------------------------

Thu, 18 Aug 2005 14:17:01 +0000 : m3avrck

Now I haven't had time to play with this patch yet, but that last
comment gave me an idea.


Is there a JS check to see if the user is trying to navigate away from
the page? For example, they are uploading a file and click a link in
another app, all of a sudden that upload page they are on is replaced
by some other page. A simple JS check for this saying "Do you want to
leave this page and cancel the upload?" would certainly be a great
usability boost and prevent this from happening.




------------------------------------------------------------------------

Thu, 18 Aug 2005 14:53:58 +0000 : tostinni

Retaking this idea of controlling if user is living a page or not, I
recently saw the new IPB permission panel demo, it makes an intense use
of AJAX and provide a way to keep track if the user is living the page.
See the demo here [3].
[3] http://blog.mattmecham.com/archives/2005/07/ipb_21_acp_perm.html




------------------------------------------------------------------------

Thu, 18 Aug 2005 15:29:27 +0000 : Steven

Thox: this is by design, and no different from submitting a file by
doing a preview, and then navigating away from the edit page. "Changes
made to the attachments list will not be permanent until you save this
post."


The patch simply streamlines the workflow.




------------------------------------------------------------------------

Thu, 18 Aug 2005 22:02:17 +0000 : Dries

Patch doesn't apply.


(BTW, is it me or is the progress bar somewhat toy-ish and somewhat
flashy?)




------------------------------------------------------------------------

Fri, 19 Aug 2005 08:39:58 +0000 : Thox

When attaching two or more files with the same without pressing submit,
the original file is overwritten and the table shows two files with the
same name. After pressing submit, the table shows one file.




------------------------------------------------------------------------

Fri, 19 Aug 2005 13:07:56 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_1.patch (14.63 KB)

Thox: this is also an existing behaviour/bug in upload.module. This
patch does not affect any of the underlying file saving or storage
mechanisms.


It simply introduces an asynchronous form submission and result
display, but otherwise uses exactly the same code as the non-JS
version.


Attached is an up-to-date, but otherwise unchanged version.




------------------------------------------------------------------------

Fri, 19 Aug 2005 21:12:13 +0000 : Bèr Kessels

Steven: I will test this on KHTML, but might need some help, for I am
very limited in time (read: reallyy have no time for testing). HOwever:
in contrary to what Stefan says, the current ajax/js works well on konq.
AND on safari (and beleive me I have a hell of a testing environment :)
[4])
[4] http://bler.webschuur.com/big_bash




------------------------------------------------------------------------

Mon, 29 Aug 2005 19:17:20 +0000 : m3avrck

What's the status of this? Can we get this into core before the freeze,
seems to be working great for me!





--===============7618244483626616338==--


From drupal-devel@drupal.org Wed Aug 31 15:47:36 2005
From: Steven <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Wed, 31 Aug 2005 15:47:36 +0000
Message-ID: <type=project&nid=28483&cid=41875&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============4907575929787988964=="

--===============4907575929787988964==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      cvs
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   Steven
-Status:       patch (code needs review)
+Status:       patch (ready to be committed)
 Attachment:   http://drupal.org/files/issues/jsupload_2.patch (14.63 KB)

Keeping up to date, setting to commit-ready given that there were no
bugs reported that are caused by this patch.




Steven



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




------------------------------------------------------------------------

Tue, 09 Aug 2005 15:38:19 +0000 : m3avrck

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.




------------------------------------------------------------------------

Wed, 10 Aug 2005 21:44:18 +0000 : drumm

+1


Works as expected and it degrades well (at first I didn't do a hard
refresh and my browser wasn't seeing the new js or css).




------------------------------------------------------------------------

Wed, 17 Aug 2005 00:41:14 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_0.patch (14.6 KB)

Keeping up to date. Still needs confirmed testing on KHTML browsers like
Konqueror and Safari, please.




------------------------------------------------------------------------

Wed, 17 Aug 2005 15:58:07 +0000 : Gabriel Radic

I'd be happy to test on Safari (1.2 and 2/1.3) and other WebKit browsers
and report the results here, but I can't apply the patch on any of my
installs (too risky) and don't have the time to set up a new Drupal
installation.


Steven, any chance for you to share a web page for me to test?


Thanks.




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:09:43 +0000 : stefan nagtegaal

Maybe it is me, but Steven why do you bother about the fact that this
will work on Safari? All other JS/AJAX we currently have in core
doesn't work with Safari...


Is it something special, or is t just curiocity?


Stefan




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:35:43 +0000 : Steven

Mostly because I can't test it, and Javascript is a fidgety business.
But I have to admit I was surprised when I found out (recently!) that
Drupal JS doesn't work in Safari. The features we check for are all
basic W3C DOM stuff, I assumed Safari would support them.


Do we know why Safari doesn't work?




------------------------------------------------------------------------

Wed, 17 Aug 2005 19:44:23 +0000 : Crell

I'm not sure about Drupal in particular, but Safari is missing a LOT Of
W3C DOM "stuff".  Many functions are there, but stubbed out and do
nothing.  Others are badly implemented.  It's closer to the standard
than IE is, but with the missing functionality I'm not sure which is
really easier to code for.  I don't know if Konqueror is crippled in
the same way, but I would suspect it is.  (I've not gotten any decent
Ajax to work in Konqueror myself, although I freely admit to not being
an expert on the subject.)




------------------------------------------------------------------------

Wed, 17 Aug 2005 21:31:50 +0000 : stefan nagtegaal

Okay, I found out how to debug with Safari..


When I goto: '?q=node/add/$type', I get these errors:
- SyntaxError - Parse error (line 37)
  drupal.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 3)
  collapse.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 4)
  autocomplete.js


Maybe you know what's wrong or what todo about it?




------------------------------------------------------------------------

Thu, 18 Aug 2005 08:49:00 +0000 : Thox

After successfully attaching a file, the user can navigate away from the
page without pressing submit. The attachment shows up in the table, but
in this case the attachment isn't linked to the node.




------------------------------------------------------------------------

Thu, 18 Aug 2005 14:17:01 +0000 : m3avrck

Now I haven't had time to play with this patch yet, but that last
comment gave me an idea.


Is there a JS check to see if the user is trying to navigate away from
the page? For example, they are uploading a file and click a link in
another app, all of a sudden that upload page they are on is replaced
by some other page. A simple JS check for this saying "Do you want to
leave this page and cancel the upload?" would certainly be a great
usability boost and prevent this from happening.




------------------------------------------------------------------------

Thu, 18 Aug 2005 14:53:58 +0000 : tostinni

Retaking this idea of controlling if user is living a page or not, I
recently saw the new IPB permission panel demo, it makes an intense use
of AJAX and provide a way to keep track if the user is living the page.
See the demo here [3].
[3] http://blog.mattmecham.com/archives/2005/07/ipb_21_acp_perm.html




------------------------------------------------------------------------

Thu, 18 Aug 2005 15:29:27 +0000 : Steven

Thox: this is by design, and no different from submitting a file by
doing a preview, and then navigating away from the edit page. "Changes
made to the attachments list will not be permanent until you save this
post."


The patch simply streamlines the workflow.




------------------------------------------------------------------------

Thu, 18 Aug 2005 22:02:17 +0000 : Dries

Patch doesn't apply.


(BTW, is it me or is the progress bar somewhat toy-ish and somewhat
flashy?)




------------------------------------------------------------------------

Fri, 19 Aug 2005 08:39:58 +0000 : Thox

When attaching two or more files with the same without pressing submit,
the original file is overwritten and the table shows two files with the
same name. After pressing submit, the table shows one file.




------------------------------------------------------------------------

Fri, 19 Aug 2005 13:07:56 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_1.patch (14.63 KB)

Thox: this is also an existing behaviour/bug in upload.module. This
patch does not affect any of the underlying file saving or storage
mechanisms.


It simply introduces an asynchronous form submission and result
display, but otherwise uses exactly the same code as the non-JS
version.


Attached is an up-to-date, but otherwise unchanged version.




------------------------------------------------------------------------

Fri, 19 Aug 2005 21:12:13 +0000 : Bèr Kessels

Steven: I will test this on KHTML, but might need some help, for I am
very limited in time (read: reallyy have no time for testing). HOwever:
in contrary to what Stefan says, the current ajax/js works well on konq.
AND on safari (and beleive me I have a hell of a testing environment :)
[4])
[4] http://bler.webschuur.com/big_bash




------------------------------------------------------------------------

Mon, 29 Aug 2005 19:17:20 +0000 : m3avrck

What's the status of this? Can we get this into core before the freeze,
seems to be working great for me!




------------------------------------------------------------------------

Wed, 31 Aug 2005 08:18:07 +0000 : stefan nagtegaal

I retested this patch and I can verify that it works the way it would
work on Safari..
Please apply..


BTW Ber: Macs just works! Only microsoft screws it!





--===============4907575929787988964==--


From drupal-devel@drupal.org Wed Aug 31 17:59:40 2005
From: =?utf-8?q?B=C3=A8r?= Kessels <drupal-devel@drupal.org>
To: development@drupal.org
Subject: [drupal-devel] [feature] JS-based file upload
Date: Wed, 31 Aug 2005 17:59:40 +0000
Message-ID: <type=project&nid=28483&cid=41906&host=@drupal.org>
In-Reply-To: <type=project&nid=28483&host=@drupal.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============0253693121414516133=="

--===============0253693121414516133==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit

Issue status update for 
http://drupal.org/node/28483
Post a follow up: 
http://drupal.org/project/comments/add/28483

 Project:      Drupal
 Version:      cvs
 Component:    upload.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Steven
 Reported by:  Steven
 Updated by:   Bèr Kessels
 Status:       patch (ready to be committed)

I don't want to be a nitpick, but IMO the progressbar should animate
from left to right. That just feels better.




Bèr Kessels



Previous comments:
------------------------------------------------------------------------

Tue, 09 Aug 2005 01:13:03 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload.patch (14.71 KB)

This patch adds JavaScript-based inline uploading (screenshot [1]).


It does this by redirecting the submission of the form to a hidden
<iframe> when you click "Attach" (we cannot submit data through Ajax
directly because you cannot read file contents from JS for security
reasons).
Once the file is submitted, the upload-section of the form is updated.


Things to note:



* The feature degrades back to the current behaviour without JS.
* If there are errors with the uploaded file (disallowed type, too big,
...), they are displayed at the top of the file attachments fieldset.
* Though the hidden-iframe method sounds dirty, it's quite compact and
is 100% implemented in .js files. The drupal.js api makes it a snap to
use.
* I included some minor improvements to the Drupal JS API and code.
* I added an API drupal_call_js() to bridge the PHP/JS gap: it takes a
function name and arguments, and outputs a <script> tag. The kicker is
that it preserves the structure and type of arguments, so e.g. PHP
associative arrays end up as objects in JS.
* I also included a progressbar widget [2] that I wrote for drumm's
ongoing update.php work. It includes Ajax status updating/monitoring,
but it is only used as a pure throbber in this patch. But as the code
was already written and is going to be used in the near future, I left
that part in. It's pretty small ;). If PHP supports ad-hoc upload info
in the future like Ruby on Rails, we can implement that in 5 minutes.

[1] http://www.acko.net/dumpx/jsupload.png
[2] http://www.acko.net/yay-progress




------------------------------------------------------------------------

Tue, 09 Aug 2005 01:39:42 +0000 : Steven

Attachment: http://drupal.org/files/issues/progress.gif (1.22 KB)

This image belongs in the misc directory.




------------------------------------------------------------------------

Tue, 09 Aug 2005 15:38:19 +0000 : m3avrck

This sounds like a great and much needed feature! I'm curious as to how
extendable this feature is because I would like to very soon create a
window that can be opened from a link (maybe even through TinyMCE) to
basically include links to documents on a Drupal website within the
content itself, instead of merely "attaching" them to the end of the
document. Taking this upload feature, along with better a file manager
could create quite an awesome webbased FTP like program that I know
many would find useful. I'll have to take a look at this patch much
more closely very soon.




------------------------------------------------------------------------

Wed, 10 Aug 2005 21:44:18 +0000 : drumm

+1


Works as expected and it degrades well (at first I didn't do a hard
refresh and my browser wasn't seeing the new js or css).




------------------------------------------------------------------------

Wed, 17 Aug 2005 00:41:14 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_0.patch (14.6 KB)

Keeping up to date. Still needs confirmed testing on KHTML browsers like
Konqueror and Safari, please.




------------------------------------------------------------------------

Wed, 17 Aug 2005 15:58:07 +0000 : Gabriel Radic

I'd be happy to test on Safari (1.2 and 2/1.3) and other WebKit browsers
and report the results here, but I can't apply the patch on any of my
installs (too risky) and don't have the time to set up a new Drupal
installation.


Steven, any chance for you to share a web page for me to test?


Thanks.




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:09:43 +0000 : stefan nagtegaal

Maybe it is me, but Steven why do you bother about the fact that this
will work on Safari? All other JS/AJAX we currently have in core
doesn't work with Safari...


Is it something special, or is t just curiocity?


Stefan




------------------------------------------------------------------------

Wed, 17 Aug 2005 16:35:43 +0000 : Steven

Mostly because I can't test it, and Javascript is a fidgety business.
But I have to admit I was surprised when I found out (recently!) that
Drupal JS doesn't work in Safari. The features we check for are all
basic W3C DOM stuff, I assumed Safari would support them.


Do we know why Safari doesn't work?




------------------------------------------------------------------------

Wed, 17 Aug 2005 19:44:23 +0000 : Crell

I'm not sure about Drupal in particular, but Safari is missing a LOT Of
W3C DOM "stuff".  Many functions are there, but stubbed out and do
nothing.  Others are badly implemented.  It's closer to the standard
than IE is, but with the missing functionality I'm not sure which is
really easier to code for.  I don't know if Konqueror is crippled in
the same way, but I would suspect it is.  (I've not gotten any decent
Ajax to work in Konqueror myself, although I freely admit to not being
an expert on the subject.)




------------------------------------------------------------------------

Wed, 17 Aug 2005 21:31:50 +0000 : stefan nagtegaal

Okay, I found out how to debug with Safari..


When I goto: '?q=node/add/$type', I get these errors:
- SyntaxError - Parse error (line 37)
  drupal.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 3)
  collapse.js
- TypeError - Object (result of expression isJsEnabled) does not allow
calls (line 4)
  autocomplete.js


Maybe you know what's wrong or what todo about it?




------------------------------------------------------------------------

Thu, 18 Aug 2005 08:49:00 +0000 : Thox

After successfully attaching a file, the user can navigate away from the
page without pressing submit. The attachment shows up in the table, but
in this case the attachment isn't linked to the node.




------------------------------------------------------------------------

Thu, 18 Aug 2005 14:17:01 +0000 : m3avrck

Now I haven't had time to play with this patch yet, but that last
comment gave me an idea.


Is there a JS check to see if the user is trying to navigate away from
the page? For example, they are uploading a file and click a link in
another app, all of a sudden that upload page they are on is replaced
by some other page. A simple JS check for this saying "Do you want to
leave this page and cancel the upload?" would certainly be a great
usability boost and prevent this from happening.




------------------------------------------------------------------------

Thu, 18 Aug 2005 14:53:58 +0000 : tostinni

Retaking this idea of controlling if user is living a page or not, I
recently saw the new IPB permission panel demo, it makes an intense use
of AJAX and provide a way to keep track if the user is living the page.
See the demo here [3].
[3] http://blog.mattmecham.com/archives/2005/07/ipb_21_acp_perm.html




------------------------------------------------------------------------

Thu, 18 Aug 2005 15:29:27 +0000 : Steven

Thox: this is by design, and no different from submitting a file by
doing a preview, and then navigating away from the edit page. "Changes
made to the attachments list will not be permanent until you save this
post."


The patch simply streamlines the workflow.




------------------------------------------------------------------------

Thu, 18 Aug 2005 22:02:17 +0000 : Dries

Patch doesn't apply.


(BTW, is it me or is the progress bar somewhat toy-ish and somewhat
flashy?)




------------------------------------------------------------------------

Fri, 19 Aug 2005 08:39:58 +0000 : Thox

When attaching two or more files with the same without pressing submit,
the original file is overwritten and the table shows two files with the
same name. After pressing submit, the table shows one file.




------------------------------------------------------------------------

Fri, 19 Aug 2005 13:07:56 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_1.patch (14.63 KB)

Thox: this is also an existing behaviour/bug in upload.module. This
patch does not affect any of the underlying file saving or storage
mechanisms.


It simply introduces an asynchronous form submission and result
display, but otherwise uses exactly the same code as the non-JS
version.


Attached is an up-to-date, but otherwise unchanged version.




------------------------------------------------------------------------

Fri, 19 Aug 2005 21:12:13 +0000 : Bèr Kessels

Steven: I will test this on KHTML, but might need some help, for I am
very limited in time (read: reallyy have no time for testing). HOwever:
in contrary to what Stefan says, the current ajax/js works well on konq.
AND on safari (and beleive me I have a hell of a testing environment :)
[4])
[4] http://bler.webschuur.com/big_bash




------------------------------------------------------------------------

Mon, 29 Aug 2005 19:17:20 +0000 : m3avrck

What's the status of this? Can we get this into core before the freeze,
seems to be working great for me!




------------------------------------------------------------------------

Wed, 31 Aug 2005 08:18:07 +0000 : stefan nagtegaal

I retested this patch and I can verify that it works the way it would
work on Safari..
Please apply..


BTW Ber: Macs just works! Only microsoft screws it!




------------------------------------------------------------------------

Wed, 31 Aug 2005 15:47:34 +0000 : Steven

Attachment: http://drupal.org/files/issues/jsupload_2.patch (14.63 KB)

Keeping up to date, setting to commit-ready given that there were no
bugs reported that are caused by this patch.





--===============0253693121414516133==--