Opened 10 years ago
Last modified 3 years ago
#2461 new enhancement
[PATCH] Ship Sinking Refinement — at Version 6
Reported by: | brian | Owned by: | |
---|---|---|---|
Priority: | Nice to Have | Milestone: | Backlog |
Component: | Simulation | Keywords: | patch, simple |
Cc: | Patch: |
Description (last modified by )
It would be nice if sunken ships were visible after a naval battle. The ships would sink quickly utilizing the current behavior until they reach the bottom. Upon reaching the bottom, they would sink at a slow rate like regular units.
This will be somewhat tricky as the sink rate (SinkRate) and sink acceleration (SinkAccel) are defined on the basic ship template. It would need to be determined where to get the slower decay values.
The relevant C++ file is: source/simulation2/components/CCmpDecay.cpp
This ticked is somewhat related to: http://trac.wildfiregames.com/ticket/2986
Change History (7)
comment:1 by , 10 years ago
Keywords: | simple added |
---|
by , 10 years ago
Attachment: | ShipSinkingRefinements.patch added |
---|
comment:2 by , 10 years ago
Keywords: | review added |
---|---|
Milestone: | Backlog → Alpha 17 |
Summary: | Ship Sinking Refinement → [PATCH] Ship Sinking Refinement |
comment:3 by , 10 years ago
Keywords: | patch added; ship boat sink sinking decay review removed |
---|---|
Milestone: | Alpha 17 → Alpha 18 |
comment:4 by , 10 years ago
Thanks for the feedback and sorry for the late reply. I'll get on this and have something for you at the weekend.
comment:5 by , 9 years ago
Description: | modified (diff) |
---|
comment:6 by , 9 years ago
Description: | modified (diff) |
---|
Hello Demiguise, and thanks for that patch!
I tested it and I must say I find the result really neat. :) However I find the slow sinking phase really long, maybe you could tweak this to make it a bit quicker.
About the code, I think you should change the way you reorganized the template. Indeed, what you call the "slow sinking" is actually the normal "sinking" for land units, so you should better add into
<SinkingAnim/>
a "rapid sinking" for more consistency. That should make your code in the Decay component cleaner too.About the style, just fix the indentation in the xml, and avoid putting comments just after brackets, better separate this into two lines. I also see little mistakes, just read thoroughly wiki:Coding_Conventions if you didn't already.
I'll push this ticket to the next milestone because we're close to feature freeze, so I'm afraid we won't add your code to Alpha 17. However, we're looking forward your contribution for the next release! Thanks again for working on this.