Closed DryCreations closed 3 years ago
There are foreseeable compatibility issues with this pull request: https://github.com/DryCreations/pdfproject/pull/36
I will close this until I resolve them.
Changes have been made to be compatible with https://github.com/DryCreations/pdfproject/pull/36
Currently the save function does work with all tools, except the eraser.
I see some issues arising form design decisions that we have made already, that will make implementing this erase tool very difficult.
Currently I have just made the eraser draw in white, however this is not consistent with our javafx canvas; The eraser does not affect the background pdf when drawing on the javafx canvas. However with the pdfcanvas, the text that's in the background can be overwritten.
It might make sense to change the eraser tools functionality. I see a few possible changes:
select
then delete
command everytime the user needs to delete something.if anyone else is able to figure out a good way to erase things on the pdfcanvas, feel free to say so. I'm currently messing with masks, and shading to see if I can get it to work.
clicking save will prompt the user to select a location to save the file to, then a new pdf document with the changes is created.