Skip to content
This repository has been archived by the owner. It is now read-only.

[annotations] Add an Undo tool #4370

Closed
wresuolc opened this issue Apr 25, 2018 · 3 comments
Closed

[annotations] Add an Undo tool #4370

wresuolc opened this issue Apr 25, 2018 · 3 comments
Assignees

Comments

@wresuolc
Copy link
Contributor

@wresuolc wresuolc commented Apr 25, 2018

@chenba
Copy link
Collaborator

@chenba chenba commented May 21, 2018

I can apply a transform to the redo icon to get an undo icon though. 🤷‍♂️

@sevaan
Copy link
Collaborator

@sevaan sevaan commented May 22, 2018

Sorry for the delay responding. Yes, that does seem like an oversight. I think a transform is perfect.

chenba added a commit to chenba/screenshots that referenced this issue May 24, 2018
chenba added a commit to chenba/screenshots that referenced this issue May 25, 2018
jaredhirsch added a commit that referenced this issue May 29, 2018
* Add undo & redo for annotations. (#4370, #4371)

* Scale the drawing when saving history. (#4370)

* Scale once and disable reset button. (#4370, #4371, #4453)

* Replace switch with if-else. (#4370, #4371)

* Prevent decimals from zoom or DPI scaling. (#4370, #4371)

* Use more CSS and less svg files. (#4370, #4371)
testeaxeax pushed a commit to testeaxeax/screenshots that referenced this issue Jun 7, 2018
* Add undo & redo for annotations. (mozilla-services#4370, mozilla-services#4371)

* Scale the drawing when saving history. (mozilla-services#4370)

* Scale once and disable reset button. (mozilla-services#4370, mozilla-services#4371, mozilla-services#4453)

* Replace switch with if-else. (mozilla-services#4370, mozilla-services#4371)

* Prevent decimals from zoom or DPI scaling. (mozilla-services#4370, mozilla-services#4371)

* Use more CSS and less svg files. (mozilla-services#4370, mozilla-services#4371)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants