Skip to content
This repository has been archived by the owner on Dec 19, 2017. It is now read-only.

Drag and drop a photo from browser causes an error #3635

Closed
ktaki opened this issue Jan 28, 2016 · 18 comments
Closed

Drag and drop a photo from browser causes an error #3635

ktaki opened this issue Jan 28, 2016 · 18 comments

Comments

@ktaki
Copy link
Contributor

ktaki commented Jan 28, 2016

Action layers.addLayers failed: base element not found

Open Adobe Stock site in a browser. Drag and drop a photo onto a canvas.

screen shot 2016-01-28 at 2 03 32 pm

@iwehrman
Copy link
Contributor

iwehrman commented Feb 2, 2016

Probably happens for all images from browsers?

@ktaki
Copy link
Contributor Author

ktaki commented Feb 2, 2016

Yes, that is the case. Dragging any image from browsers on to canvas does.

@iwehrman iwehrman added this to the Sprint 3.4 milestone Feb 7, 2016
@iwehrman
Copy link
Contributor

iwehrman commented Feb 7, 2016

Hmm, I can't repro on Mac pgdev.1054 at 86ccaf6. Did you observe this on Mac or Win?

@ktaki
Copy link
Contributor Author

ktaki commented Feb 8, 2016

I tried with the Mac pgdev 1060 at 86ccaf6. It is and has been easy to repro for me. @chadrolfs Can you try it? Just drag any image from a browser to a canvas.

@ktaki ktaki assigned chadrolfs and unassigned ktaki Feb 8, 2016
@chadrolfs chadrolfs removed the CNR label Feb 8, 2016
@chadrolfs
Copy link
Contributor

yep, piece of 🍰, I can repro very easily

@chadrolfs chadrolfs assigned iwehrman and unassigned chadrolfs Feb 8, 2016
@iwehrman
Copy link
Contributor

I still can't repro, I guess it's a timing issue. Maybe some other engineer can try to repro? @DivyaPrabhakar?

@iwehrman iwehrman removed their assignment Feb 10, 2016
@iwehrman iwehrman changed the title Drag and drop a photo from Adobe Stock causes an error Drag and drop a photo from browser causes an error Feb 12, 2016
@iwehrman
Copy link
Contributor

@Volfied can you repro this?

@baaygun
Copy link
Contributor

baaygun commented Feb 17, 2016

I can repro, given that base element is not found error is coming from PS, I'll take a look.

@baaygun
Copy link
Contributor

baaygun commented Feb 17, 2016

Trick is to open a new document.

@baaygun
Copy link
Contributor

baaygun commented Feb 18, 2016

I believe the problem is dragging an image to canvas acts as a paste operation, and we get the "drag" event before Photoshop had the time to update it's own document.

https://jira.corp.adobe.com/browse/PS-1370

@iwehrman
Copy link
Contributor

Kicking this out of the milestone because we're blocked, but adding the Ship Blocker label.

@iwehrman iwehrman removed this from the Sprint 3.5 milestone Feb 19, 2016
@iwehrman
Copy link
Contributor

iwehrman commented Mar 9, 2016

Over to @ktaki to verify.

@iwehrman iwehrman assigned ktaki and unassigned shaoshing Mar 9, 2016
@ktaki ktaki assigned iwehrman and unassigned ktaki Mar 9, 2016
@ktaki
Copy link
Contributor Author

ktaki commented Mar 9, 2016

No, it did not fix the issue. Do we want to double the delay? It seems to me, this will not result in any bad state afterwards. What would happen if we leave it as is?

@iwehrman iwehrman assigned shaoshing and unassigned iwehrman Mar 9, 2016
@iwehrman
Copy link
Contributor

iwehrman commented Mar 9, 2016

Back to @shaoshing. Maybe the timeout wasn't long enough?

@iwehrman
Copy link
Contributor

iwehrman commented Mar 9, 2016

In theory the UI should reset itself after the unhandled error. I prefer not to rely on that though...

@shaoshing
Copy link
Contributor

@iwehrman @ktaki It turns out that the error has nothing to do with time delay. I've explains the actual cause in my PR #3797, and it only happens when dropping image from browser to an empty artboard.

Here's how to reproduce the bug:
1 create a new document
2 drag any image from browser to the art board
3 error base element not found

Same operation works if the artboard is not empty:
1 create a new document
2 draw a rectangular on the artboard
3 drag any image from browser to the art board

@iwehrman
Copy link
Contributor

iwehrman commented Mar 9, 2016

OK, let's try it again! FBNC to @ktaki.

@ktaki
Copy link
Contributor Author

ktaki commented Mar 10, 2016

Verified the fix.

@ktaki ktaki closed this as completed Mar 10, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants