search

marianokit / magento-w2p

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

Image deletion #164

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
Add deletion function to user image library.

1. Add a little cross next to EDIT button to delete an image. 

2. Add Delete button to the image editor window. 

Remove from all scrolls upon confirmation.

Mockups?

Original issue reported on code.google.com by zetapri...@gmail.com on 21 Mar 2010 at 11:38

GoogleCodeExporter commented 8 years ago 
Image editing window preview - editing_window_delete_preview.png
Image editing window delete icon - delete.png

Image field scroll preview - image_field_scroll_delete_preview.png

I added the X icon at the end of the top row in the sprite image -
images-selector-sprites.png

Original comment by agur...@gmail.com on 23 Mar 2010 at 4:58

Attachments:

GoogleCodeExporter commented 8 years ago 
I don't see X icon at the end of the top row in the sprite image -
images-selector-sprites.png

Original comment by Anatoly....@gmail.com on 23 Mar 2010 at 5:25

GoogleCodeExporter commented 8 years ago 
It's in there(X-sprite_preview.png), you just don't see it because it is white 
on top
of a transparent background.
I made it this way just like the other two white icons at the end of the 
sprite. I
can give it to you with the blue background square if you prefer it that way.

Original comment by agur...@gmail.com on 23 Mar 2010 at 6:11

Attachments:

GoogleCodeExporter commented 8 years ago 
Heh, I haven't seen even other two icons before :-)

Original comment by Anatoly....@gmail.com on 23 Mar 2010 at 6:15

GoogleCodeExporter commented 8 years ago 
Edit icon contains blue background. Maybe it's better to make X icon with blue
background, too.
Please, add Edit icon to the sprite.

Original comment by Anatoly....@gmail.com on 23 Mar 2010 at 6:24

GoogleCodeExporter commented 8 years ago 
Is this ok?

Original comment by agur...@gmail.com on 23 Mar 2010 at 7:04

Attachments:

GoogleCodeExporter commented 8 years ago 
Looks OK for me

Original comment by Anatoly....@gmail.com on 24 Mar 2010 at 5:24

GoogleCodeExporter commented 8 years ago 
Please test on mageimage

Original comment by k.safro...@gmail.com on 26 Mar 2010 at 5:47

GoogleCodeExporter commented 8 years ago 
Delete icon on the image thumb in the scroll doesn't work.

Original comment by agur...@gmail.com on 29 Mar 2010 at 7:59

GoogleCodeExporter commented 8 years ago 
Can't replicate the problem. There is a slight chance I was working on another 
issue 
and code was broken at the moment you tested. Please retest and be more 
specific if you 
still can replicate the problem.

Original comment by k.safro...@gmail.com on 29 Mar 2010 at 8:54

GoogleCodeExporter commented 8 years ago 
Go to this product (in FF or IE)
http://www.zetaprints.com/mageimage/index.php/customized-for-testing/image-optio
ns.html
upload an image and click on the X icon in the scroll.

Original comment by agur...@gmail.com on 29 Mar 2010 at 9:11

GoogleCodeExporter commented 8 years ago 
Fixed

Original comment by k.safro...@gmail.com on 29 Mar 2010 at 10:10

GoogleCodeExporter commented 8 years ago 
1. Deleting an image from one of the scrolls does not remove the image from the 
other
scrolls. 

2. In FF I'm asked to click "OK" 3 times for a template with 3 image fields. In 
IE
I'm only asked to click "OK" 1 time but I get an err (cant_delete_image.png)

Tested on this product:
http://www.zetaprints.com/mageimage/index.php/customized-for-testing/multipage-t
esting-02.html

Original comment by agur...@gmail.com on 29 Mar 2010 at 10:36

Attachments:

GoogleCodeExporter commented 8 years ago

Original comment by k.safro...@gmail.com on 30 Mar 2010 at 6:28

GoogleCodeExporter commented 8 years ago

Original comment by agur...@gmail.com on 30 Mar 2010 at 8:18

GoogleCodeExporter commented 8 years ago

Original comment by zetapri...@gmail.com on 30 Mar 2010 at 10:11