DDMAL / Pixel.js

🖌 A web-based pixel classification and correction platform
MIT License
9 stars 2 forks source link

Images are imported below erase paths on a same layer #232

Open PotassiumK opened 7 years ago

PotassiumK commented 7 years ago

If a user draws a path with the erase tool and then uploads an image onto that layer, the erase path will appear on top of the imported image.

This does not necessarily need to be "fixed" as it could be handy in a situation where a user has already erased some parts that definitely do not have pixels belonging to the layer.

zeyadhazem commented 7 years ago

The current implementation handles uploaded images as background images and this is why this happens. It was an intended functionality at that time but can be changed if needed

PotassiumK commented 7 years ago

The reason I pointed it out is that I uploaded the wrong layer image to a selected layer and I used the erase tool to erase everything and then tried to upload the correct image. This resulted in the correct image not appearing at all, evidently. The only option remaining was to delete the layer and create another one, which felt like an error because I was left with a layer that no longer supported imports.