marianokit / magento-w2p

Automatically exported from code.google.com/p/magento-w2p
0 stars 0 forks source link

Upload file on selection #242

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
It appears users don't always click on upload file button, go to update preview 
and expect the file to upload.
We don't want to upload them all at once in one go.
It will be easier to start file uploading as soon as they selected the file.

Original issue reported on code.google.com by zetapri...@gmail.com on 24 Jun 2010 at 1:11

GoogleCodeExporter commented 8 years ago
Add cancel button.

Original comment by zetapri...@gmail.com on 8 Jul 2010 at 10:38

GoogleCodeExporter commented 8 years ago

Original comment by zetapri...@gmail.com on 18 Aug 2010 at 10:49

GoogleCodeExporter commented 8 years ago

Original comment by trilobit.alex@gmail.com on 23 Aug 2010 at 6:27

GoogleCodeExporter commented 8 years ago
We figured out there is a problem with multiple file uploading.
It was possible to select more than one file one after the other and they would 
all upload in parallel, but there was no hint they were all uploading.
It may look like a feature, but in fact it's a bug.
Just shouldn't be there any longer. One file at time per field.

Original comment by zetapri...@gmail.com on 27 Aug 2010 at 10:59

GoogleCodeExporter commented 8 years ago
I need a design for CANCEL button in place of UPLOAD button

Original comment by trilobit.alex@gmail.com on 27 Aug 2010 at 11:44

Attachments:

GoogleCodeExporter commented 8 years ago
and I need a grey design for BROWSE

Original comment by trilobit.alex@gmail.com on 27 Aug 2010 at 12:14

GoogleCodeExporter commented 8 years ago

Original comment by agur...@gmail.com on 27 Aug 2010 at 1:49

Attachments:

GoogleCodeExporter commented 8 years ago

Original comment by zetapri...@gmail.com on 10 Sep 2010 at 12:48

GoogleCodeExporter commented 8 years ago
SVN release:
http://code.google.com/p/magento-w2p/source/detail?r=1040

to test:
http://ec2-174-129-95-130.compute-1.amazonaws.com/magedev/index.php/prod-2.html

Original comment by trilobit.alex@gmail.com on 13 Sep 2010 at 11:24

GoogleCodeExporter commented 8 years ago
1. Out of 20 tries, I got 3 pop-up err (upload_err.png). Can't replicate it 
deliberately, though. Could be local or smth, not sure. It only happened when I 
stacked up the queue (selected 3 images at once).

2. The file name should disappear from the input field after a user clicks 
"Cancel". Browse for an image and select it. As soon as the image starts 
uploading, click "Cancel". The image name is still visible in the field 
(file_name.png).

3. Change the cursor to a pointer when hovering the "Browse" button.

Original comment by agur...@gmail.com on 13 Sep 2010 at 3:22

Attachments:

GoogleCodeExporter commented 8 years ago
I'd be worried about the failure rate.

Original comment by zetapri...@gmail.com on 13 Sep 2010 at 9:56

GoogleCodeExporter commented 8 years ago
#2 - fixed
SVN release
http://code.google.com/p/magento-w2p/source/detail?r=1051
to test
http://ec2-174-129-95-130.compute-1.amazonaws.com/magedev/index.php/prod-2.html

#3 - I can't. The problem is that Firefox does not support setting cursor on 
the input file element.

Original comment by trilobit.alex@gmail.com on 15 Sep 2010 at 7:30

GoogleCodeExporter commented 8 years ago
1. Wasn't able to replicate the err again. Uploaded a new template with 5 image 
fields in it:
http://ec2-174-129-95-130.compute-1.amazonaws.com/magedev/index.php/multi-image.
html
Stacked up the queue, all images got uploaded OK.

2. Test OK

3. Ignore it then.

Original comment by agur...@gmail.com on 15 Sep 2010 at 12:24

GoogleCodeExporter commented 8 years ago
I did notice, however, that the uploading process is a bit different than the 
one on w2p site.
Images start uploading as soon as I select them without stacking up in a queue. 
In other words, they do not upload one by one (latest selected waiting for the 
previous image to upload first). Isn't it safer to stack them up and upload one 
by one, just like on w2p site?

Original comment by agur...@gmail.com on 15 Sep 2010 at 12:29

GoogleCodeExporter commented 8 years ago

Original comment by zetapri...@gmail.com on 10 Oct 2010 at 7:08

GoogleCodeExporter commented 8 years ago
Please, test it with latest release.

Original comment by Anatoly....@gmail.com on 25 Nov 2010 at 6:21

GoogleCodeExporter commented 8 years ago
That last issue from my previous comment remains. Uploader works fine but 
images are not queued like they were before. All images start uploading at the 
same time.

Original comment by agur...@gmail.com on 26 Nov 2010 at 9:51

GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
But they are never queued in the extension.

Original comment by Anatoly....@gmail.com on 26 Nov 2010 at 11:25

GoogleCodeExporter commented 8 years ago
We have separate issue for opera created by you, see issue 342.

Original comment by Anatoly....@gmail.com on 26 Nov 2010 at 11:47

GoogleCodeExporter commented 8 years ago
Yep, I deleted the Opera comment after I remembered about the separate issue.

Well, auto uploader works across browsers, I'm flagging this issue as Test OK. 
We'll deal with queue order separately if needed.

I guess it will be Test OK after we implement it in the next stable release.

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

GoogleCodeExporter commented 8 years ago
Is this issue still valid?

Original comment by zetapri...@gmail.com on 12 Jan 2011 at 6:38

GoogleCodeExporter commented 8 years ago
From @21:
> Well, auto uploader works across browsers, I'm flagging this issue as Test OK.

It's Test OK

> We'll deal with queue order separately if needed.

Will we or not? :-) If yes, then we have to create new issue.

Original comment by Anatoly....@gmail.com on 12 Jan 2011 at 7:42

GoogleCodeExporter commented 8 years ago
What is happening with this issue ?
Is it in the trunk?
Included into any release?

Original comment by zetapri...@gmail.com on 17 Feb 2011 at 7:29

GoogleCodeExporter commented 8 years ago

Original comment by zetapri...@gmail.com on 17 Feb 2011 at 7:29

GoogleCodeExporter commented 8 years ago
We already have it in releases.

But last question was to deal with queue or not?

Original comment by Anatoly....@gmail.com on 18 Feb 2011 at 3:48

GoogleCodeExporter commented 8 years ago

Original comment by Anatoly....@gmail.com on 18 Feb 2011 at 3:48

GoogleCodeExporter commented 8 years ago
Never mind the queue. Let it upload at the same time for now. We'll open a new 
issue if it proves to be a problem.

Original comment by zetapri...@gmail.com on 18 Feb 2011 at 3:54

GoogleCodeExporter commented 8 years ago
Then it's Test OK

Original comment by Anatoly....@gmail.com on 18 Feb 2011 at 4:49

GoogleCodeExporter commented 8 years ago
No need to re-test, but just check we have it next time you test something else.

Original comment by zetapri...@gmail.com on 18 Feb 2011 at 4:56

GoogleCodeExporter commented 8 years ago

Original comment by zetapri...@gmail.com on 3 Mar 2011 at 10:49

GoogleCodeExporter commented 8 years ago

Original comment by zetapri...@gmail.com on 3 Mar 2011 at 10:49