-
Notifications
You must be signed in to change notification settings - Fork 74
Drag and drop a photo from browser causes an error #3635
Comments
Probably happens for all images from browsers? |
Yes, that is the case. Dragging any image from browsers on to canvas does. |
Hmm, I can't repro on Mac pgdev.1054 at 86ccaf6. Did you observe this on Mac or Win? |
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. |
yep, piece of 🍰, I can repro very easily |
I still can't repro, I guess it's a timing issue. Maybe some other engineer can try to repro? @DivyaPrabhakar? |
@Volfied can you repro this? |
I can repro, given that |
Trick is to open a new document. |
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. |
Kicking this out of the milestone because we're blocked, but adding the Ship Blocker label. |
Over to @ktaki to verify. |
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? |
Back to @shaoshing. Maybe the timeout wasn't long enough? |
In theory the UI should reset itself after the unhandled error. I prefer not to rely on that though... |
@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: Same operation works if the artboard is not empty: |
OK, let's try it again! FBNC to @ktaki. |
Verified the fix. |
Action layers.addLayers failed: base element not found
Open Adobe Stock site in a browser. Drag and drop a photo onto a canvas.
The text was updated successfully, but these errors were encountered: