libertyernie / brawltools

BrawlBox and BrawlLib
http://forums.kc-mm.com/index.php?topic=67847.0
142 stars 31 forks source link

Crash1 Collision Type #174

Closed soopercool101 closed 6 years ago

soopercool101 commented 6 years ago

The collision type currently labelled as "Crash1" has been found to be used as a Spikes Hazard collision on Target Test and Target Test based stages. As such, I propose that the collision names be updated to the following: Crash1->SpikesTargetTest (Or something similar, as it should be noted that the collision does crash on non Target Test based stages)

Crash2->Crash1 Crash3->Crash2 (Or they could stay the same. This doesn't matter so much)

libertyernie commented 6 years ago

Renamed Crash1 to SpikesTargetTestOnly in commit 1b9e2324a5fc9d2eca2cbbaf04a79732c1939bbf.

I think it's best to keep Crash2 and Crash3 as is, so we don't reuse names for one type that used to refer to another.