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

Loadout Customization - Items expanded & more... #7614

Draft
wants to merge 33 commits into
base: master
Choose a base branch
from

Conversation

Zenith00000
Copy link
Contributor

@Zenith00000 Zenith00000 commented Nov 18, 2024

About the pull request

Will update this soon. More stuff to come, ever evolving...

Explain why it's good for the game

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑 Crowford, Frans_Feiffer, VaNdU-jr, cuberound, armolitskiy

add: Added many new loadout items
/:cl:

@cmss13-ci cmss13-ci bot added Sprites Remove the soul from the game. Feature Feature coder badge labels Nov 18, 2024
@Wintermote
Copy link

Would be nice if the vast majority, or all, purely cosmetic items like scarves, berets, or goggles, only cost 1 point instead of 2-3.

Seems like it'll happen eventually anyway as the scope of cosmetic attachments and items increases. As it is right now though - TWE shoulder patch (1) + beret (3)? Then you can forget having your RMC bayonet (4) to go with it, even though it'd be thematically appropriate.

Just seems oddly and unnecessarily restrictive for stuff that doesn't influence gameplay much/at all.

@Blundir Blundir added the Sprites Frozen Merge this and you will make the spritetainers angry. label Nov 19, 2024
@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Nov 19, 2024
@cm13-github
Copy link
Contributor

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

@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Nov 19, 2024
@cm13-github
Copy link
Contributor

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

@Zenith00000
Copy link
Contributor Author

Would be nice if the vast majority, or all, purely cosmetic items like scarves, berets, or goggles, only cost 1 point instead of 2-3.

Seems like it'll happen eventually anyway as the scope of cosmetic attachments and items increases. As it is right now though - TWE shoulder patch (1) + beret (3)? Then you can forget having your RMC bayonet (4) to go with it, even though it'd be thematically appropriate.

Just seems oddly and unnecessarily restrictive for stuff that doesn't influence gameplay much/at all.

I'd agree, but there's probably a reason why the prices were set the way they are. I'm guessing because they don't want marines running around with every cosmetic, making it more about picking your favorites rather then everything possible. I do see both sides though and I'll consider it after getting some more feedback.

@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Nov 23, 2024
@cm13-github
Copy link
Contributor

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

@fluffe9911
Copy link
Contributor

Squad colored smock when?

@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 27, 2024
@cm13-github
Copy link
Contributor

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

@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Dec 4, 2024
@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Dec 14, 2024
@cm13-github
Copy link
Contributor

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

@cm13-github
Copy link
Contributor

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

@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Dec 15, 2024
@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Dec 15, 2024
@cm13-github
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Feature coder badge Sprites Frozen Merge this and you will make the spritetainers angry. Sprites Remove the soul from the game.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants