-
-
Notifications
You must be signed in to change notification settings - Fork 385
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add auto-panning to the Artboard tool #1652
Add auto-panning to the Artboard tool #1652
Conversation
61d7243
to
121724e
Compare
Would it be reasonable to turn this PR into one which encompasses all the remaining tools, or do you think it requires too many changes and too much effort per tool that it'd be better to split it into one PR per tool? |
I stopped at one tool since I wanted confirmation on the way I have abstracted the code to I can combine it in one PR but it will take more than 2 weeks to complete it and diff size will definitely be above 500 lines. |
In that case, several PRs sounds preferable. Is this one ready for review and merging? |
Yes |
121724e
to
88816aa
Compare
!build |
|
Partly closes #1527
auto_panning.rs
.ResizingBounds
,Dragging
orDrawing
.AnimationFrame
as specified in https://discord.com/channels/731730685944922173/881073965047636018/1214107184200421396