Funkin icon indicating copy to clipboard operation
Funkin copied to clipboard

fix focuscamera CLASSIC breaking zooms and snapping

Open nebulazorua opened this issue 9 months ago • 3 comments

When using Classic on FocusCamera, you have to place any zooms AFTER it, because it breaks them. It also snaps the camera, creating for jerky movement when having to use many Classic camera movements This fixes that behaviour

nebulazorua avatar May 08 '24 13:05 nebulazorua

might update to add "Reset Camera" to the Focus Camera event schema. You might also want tweens to not reset for fast camera focuses, too, to prevent jerky movements.

nebulazorua avatar May 15 '24 12:05 nebulazorua

nevermind tweens dont even call Reset Camera until after the tween is done unless its instant what the FUUCK classic seems like it was just an oversight more than anything, then?? Still noticed an error though, so gonna fix that rq

nebulazorua avatar May 15 '24 12:05 nebulazorua

hopefully should be fixed now

nebulazorua avatar May 15 '24 12:05 nebulazorua

Me and Fabs reviewed this and concluded that the camera zooms work fine without this change. If you want to post the chart you're using and a before/after then that would help with review.

EliteMasterEric avatar May 29 '24 01:05 EliteMasterEric

Me and Fabs reviewed this and concluded that the camera zooms work fine without this change. If you want to post the chart you're using and a before/after then that would help with review.

Oh yeah hold on lemme get a video of it I was running into the issue when porting B-Sides Stress

nebulazorua avatar May 31 '24 09:05 nebulazorua

https://github.com/FunkinCrew/Funkin/assets/12458328/2bc0464f-091c-49b8-8af8-9fd7af5514b2

As the camera switching becomes faster, you can see it snap. It looks really bad, imo, and forces me to use an ease style when I'd want to use classic.

And here's the after:

https://github.com/FunkinCrew/Funkin/assets/12458328/4b41698e-5475-4748-a7d1-c2ce4a1fc93f

nebulazorua avatar May 31 '24 10:05 nebulazorua

OH YEAH also fixes a bug where camera focuses cancel camera zooms, like this (just a test on DadBattle)

Before:

https://github.com/FunkinCrew/Funkin/assets/12458328/95228810-e598-4fc9-9b41-b756df616180

After:

https://github.com/FunkinCrew/Funkin/assets/12458328/905cbcde-8377-44ca-aca9-57c8e387ea81

(Also shows the Classic snap) Chart: dadbattle-chart.json

nebulazorua avatar May 31 '24 10:05 nebulazorua

Maybe it's time for a merge? It was long enough that I didn’t even find it when I made my fix

PurSnake avatar Jun 09 '24 21:06 PurSnake

(Coming back to this to point out the issue this fixes can be seen in 0.5.0 with Erect Cocoa!)

nebulazorua avatar Sep 13 '24 15:09 nebulazorua

Can you update the merge conflicts on this branch? @nebulazorua

EliteMasterEric avatar Sep 22 '24 05:09 EliteMasterEric

Very good PR, needs to be conflict resolved immediately

Hundrec avatar Sep 26 '24 19:09 Hundrec