Skip to content
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

Collapsible Platform Edges. #7096

Merged
merged 38 commits into from
Nov 9, 2024
Merged

Conversation

Venuska1117
Copy link
Contributor

@Venuska1117 Venuska1117 commented Sep 3, 2024

!!!DISCLAIMER!!!
I am very bad at spriting and i am not joking.

I recommend to put this PR in Testmerge, so i can find issues caused by swapping paths and find oversights with sprites.

About the pull request

I am working on rework of Kutjevo #6971

Fixes: #7049

Because i plan to add elevation by using large quantities of platform edges, i tough it will be too annoying for players, who are keep dying, like xenos and marines because edge stopped them for 1-2 seconds.

Edit: Its 70% done, i just lack sprites. (made only one platform sprite set for demonstration in videos below)

Edit(07.09): I was forced to change paths of platforms so they can play different sounds depending on what they are made of.

Edit(15.09): Finally done sprites, now time to wait.

Edit(21.09): thanks to Doubleumc | Casey Walton and cuberound for having time explain me some code related terms, thanks you both for that.

Edit(15.10): Added Stair variant of Platform with own sprites and broken states.

Explain why it's good for the game

Allow game to be more dynamic by adding little more "terrain destruction" that will benefit gameplay and mappers who like to keep Map elevation but they are worried that platform edges will make experience negative. (people complain a lot)

Testing Photographs and Procedure

Click HERE to see Videos

Platforms before destruction:
New1

Platforms after destruction:
New2

How it work and sound when xeno break them:

2024-09-15.12-04-01.mp4

How it will work for Explosions:

2024-09-15.12-00-33.mp4

Changelog

🆑 Venuska1117
code: Added ability to Destroy Platform Edges by xenos on "Disarm" intent, now depending on xenos "size" they can break Platform Edges quicker or Slower, Now Explosives have chance to destroy Platform Edges by depending on how strong explosion is.
imageadd: Added Sprites for Broken Platforms Edges.
soundadd: Added Sound for Breaking "Stone" Edges.
/:cl:

i lack that coding capability
@github-actions github-actions bot added Sprites Remove the soul from the game. Code Improvement Make the code longer labels Sep 3, 2024
- Instead of 9 seconds they take 30 seconds.
@Venuska1117 Venuska1117 marked this pull request as draft September 3, 2024 11:57
@Venuska1117 Venuska1117 changed the title [WIP] Collapsible Platform Edges. Collapsible Platform Edges. Sep 3, 2024
@Venuska1117 Venuska1117 changed the title Collapsible Platform Edges. [WIP] Collapsible Platform Edges. Sep 3, 2024
@github-actions github-actions bot added the Sound Blast 5 minutes of bass boosted music to our players label Sep 6, 2024
- Fixed Layers issues.
- Changed Explosion treshold for breaking.
- Added 2 types of platform breaking sound.
- Examinating broken platform give now feedback that its destroyed in description.
- Separated "Indestructable" and "Broken" to give two different feedbacks depending on platform status.
@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Sep 7, 2024
@cm13-github
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the Mapping did you remember to save in tgm format? label Sep 7, 2024
@cm13-github
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Sep 7, 2024
- Changed paths for all platforms, so they can play sound attached to their parent.
- Add "7096-platforms-paths.txt" to allow other people to update their platforms paths.
@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Sep 7, 2024
@cm13-github
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

- Fixed one wrong typo inside patforms.dm
- changed "7096-platforms-paths.txt" order of platform_cut, so var edited icon_states are replaced fist before changed to base type.
- Run BigRed trough updatepath to fix merge conflict.
- Removed old paths in platforms.dm used to check issues in previous update.
@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Sep 9, 2024
@cm13-github
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

- Changed text of attempting to destroy platform as xeno
@Venuska1117 Venuska1117 marked this pull request as ready for review November 6, 2024 20:55
@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Nov 7, 2024
@cm13-github
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@Drulikar Drulikar added Testmerge Candidate we'll test this while you're asleep and the server has 10 players Needs Testing Need to test it on the guinea pigs (production server) labels Nov 7, 2024
@cm13-github
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Nov 7, 2024
cm13-github added a commit that referenced this pull request Nov 7, 2024
cm13-github added a commit that referenced this pull request Nov 7, 2024
cm13-github added a commit that referenced this pull request Nov 7, 2024
cm13-github added a commit that referenced this pull request Nov 9, 2024
@Drulikar Drulikar added Mapping Approved adds 500 new dict keys and removed Needs Testing Need to test it on the guinea pigs (production server) labels Nov 9, 2024
@Drulikar Drulikar added this pull request to the merge queue Nov 9, 2024
Merged via the queue into cmss13-devs:master with commit c29c394 Nov 9, 2024
27 checks passed
cmss13-ci bot added a commit that referenced this pull request Nov 9, 2024
@Venuska1117 Venuska1117 deleted the breakable-edges branch November 9, 2024 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Improvement Make the code longer Mapping Approved adds 500 new dict keys Mapping did you remember to save in tgm format? Sound Blast 5 minutes of bass boosted music to our players Sprites Approved confirmed no stray pixels Sprites Remove the soul from the game. Testmerge Candidate we'll test this while you're asleep and the server has 10 players
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Platform Edges render on wrong layer.
6 participants