Skip to content

Add slingshot animation#329

Merged
freezy merged 15 commits into
masterfrom
feature/slingshot
Oct 3, 2021
Merged

Add slingshot animation#329
freezy merged 15 commits into
masterfrom
feature/slingshot

Conversation

@freezy
Copy link
Copy Markdown
Owner

@freezy freezy commented Oct 2, 2021

This PR adds support for animating the slingshot rubbers. For more info check the documentation.

slingshot2.mp4

TODO

  • There is a bug that sets control point 0 to (0,0) after first play.

@freezy freezy self-assigned this Oct 2, 2021
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 2, 2021

Codecov Report

Merging #329 (8bce409) into master (1749b9a) will increase coverage by 0.09%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #329      +/-   ##
==========================================
+ Coverage   83.05%   83.14%   +0.09%     
==========================================
  Files         124      124              
  Lines        6708     6718      +10     
==========================================
+ Hits         5571     5586      +15     
+ Misses       1137     1132       -5     
Impacted Files Coverage Δ
VisualPinball.Engine/Math/DragPointData.cs 70.00% <100.00%> (+23.12%) ⬆️
VisualPinball.Engine/VPT/Surface/SurfaceData.cs 82.00% <100.00%> (+0.75%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b7a447d...8bce409. Read the comment docs.

@freezy freezy force-pushed the feature/slingshot branch from 40e8142 to b555d19 Compare October 2, 2021 11:58
@xantari
Copy link
Copy Markdown

xantari commented Oct 2, 2021

@freezy Thats a whole lot of sexy! Looks great!

@freezy freezy merged commit fefb44a into master Oct 3, 2021
@freezy freezy deleted the feature/slingshot branch October 3, 2021 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants