screenity icon indicating copy to clipboard operation
screenity copied to clipboard

undo redo

Open no-1ne opened this issue 4 years ago • 2 comments

Hello Alyssa, As many mentioned already Thank you for sharing your work. Its amazing..

Coming to the issue, It would be great if you can add undo redo in fabricjs this may help https://github.com/tbolis/react-sketch/blob/master/src/history.js

no-1ne avatar May 06 '21 12:05 no-1ne

Yeah, undo/redo would be easy to implement, I already have my own library I've used in other fabric.js projects of mine :) The issue is more UX-wise, I don't really want to add more buttons in there, plus if I used keyboard shortcuts this would be problematic in several websites, e.g. if you want to annotate on top of Figma for example, undoing/redoing would apply to both Screenity and Figma. So it's something that might need some thinking.

alyssaxuu avatar Jun 04 '21 09:06 alyssaxuu

Didn't think about keyboard shortcuts, one approach can be when pen mode enabled, handle redo and undo keypress combinations and not letting them pass thru to figma etc

no-1ne avatar Jun 08 '21 23:06 no-1ne

Added in the new MV3 version.

alyssaxuu avatar Dec 27 '23 17:12 alyssaxuu