sharkening #949

Open
ichthyoacanthotoxism wants to merge 2 commits from (deleted):sharkening into master
First-time contributor

Short description

Adds several new markings to Feroxi:

  • Eight new body markings: Marbled, Encircled, Embraced, Centerpiece, Starry, Tailward, Harpoon, and Gills
  • Six new leg markings: Overside, Flank, Schism, Racing Stripe, Banded, and Underside
  • Three new foot markings: Claw, Stamped, Paw
  • Nine new tail markings: Saddle, Spatter, Speckle, Undertow, Crackle, Rimed, Spots, Banded, and Half-n-Half

Changes the Feroxi tail marking limitation from 10, up from 1.

Adds a base tail marking to the Feroxi species in order to permit layering of tail markings.

Due to the new capabilities for Feroxi tail marking layering, this update will remove the visible tail markings from most current Feroxi. As the affected markings have been changed to their own standalone pieces, it is still possible for an individual to reassemble these markings in the editor if they so desire.

Why we need to add this

Feroxi currently suffer from a lack of species-specific customization options, with most current markings being different mixtures of the same few individual patterns. This aims to fix this and give more versatility to the appearance of Feroxi.

Media (Video/Screenshots)

All new Feroxi markings.
image

Checklist

Breaking changes

Ingame Changelog

🆑

  • add: Twenty-six new Feroxi markings.
  • remove: Old Feroxi tail marking clutter.
  • tweak: Old Feroxi tail marking pieces are now available as standalone pieces.
<!-- Link definitions, do not edit --> [guidelines]: https://docs.spacestation14.com/en/general-development/codebase-info/pull-request-guidelines.html [ruleset]: https://rules.byrdstation.space/#h.vigrxrnffwji ## Short description <!-- What do you propose to change with your PR? --> Adds several new markings to Feroxi: - Eight new body markings: Marbled, Encircled, Embraced, Centerpiece, Starry, Tailward, Harpoon, and Gills - Six new leg markings: Overside, Flank, Schism, Racing Stripe, Banded, and Underside - Three new foot markings: Claw, Stamped, Paw - Nine new tail markings: Saddle, Spatter, Speckle, Undertow, Crackle, Rimed, Spots, Banded, and Half-n-Half Changes the Feroxi tail marking limitation from 10, up from 1. Adds a base tail marking to the Feroxi species in order to permit layering of tail markings. Due to the new capabilities for Feroxi tail marking layering, this update will remove the visible tail markings from most current Feroxi. As the affected markings have been changed to their own standalone pieces, it is still possible for an individual to reassemble these markings in the editor if they so desire. ## Why we need to add this <!-- What is the reason for adding these changes? Please post links to Discussions as well as Bug Reports here. Please describe how this will change the game balance. --> Feroxi currently suffer from a lack of species-specific customization options, with most current markings being different mixtures of the same few individual patterns. This aims to fix this and give more versatility to the appearance of Feroxi. ## Media (Video/Screenshots) <!--If your PR contains in-game changes you must provide screenshots/videos of the changes.--> All new Feroxi markings. ![image](/attachments/a78c422f-0ecb-41e9-abaf-41c9dd12b9ce) ## Checklist <!-- Check boxes for faster reviewing of your PR --> - [x] I do not require assistance to complete the PR. - [x] I have verified that the changes work before posting/requesting review of my PR. - [x] I have added screenshots/videos of the changes, or this PR does not change in-game mechanics. - [x] I have read the [Pull Request and Changelog Guidelines][guidelines] and [Byrd Station's Contributor Ruleset][ruleset]. ## Breaking changes <!-- List any breaking changes, including namespaces, public class/method/field changes, prototype renames; and provide instructions for fixing them. This will be posted in #codebase-changes. --> ### Ingame Changelog <!--If you want the players to know about changes made in this PR, specify them using the template outside the comment. Short and informative.--> <!--:cl: - add: Added fun! - remove: Removed fun! - tweak: Changed fun! - fix: Fixed fun!--> :cl: - add: Twenty-six new Feroxi markings. - remove: Old Feroxi tail marking clutter. - tweak: Old Feroxi tail marking pieces are now available as standalone pieces.
image.png
148 KiB
Added new Feroxi markings
Some checks failed
Close PRs on master / run (pull_request_target) Has been skipped
Labels: PR / labeler (pull_request_target) Successful in 7s
Labels: Size / size-label (pull_request_target) Failing after 5s
CRLF Check / CRLF Check (pull_request) Successful in 53s
RGA schema validator / YAML RGA schema validator (pull_request) Successful in 58s
RSI Validator / Validate RSIs (pull_request) Successful in 1m3s
Map file schema validator / YAML map schema validator (pull_request) Successful in 7m39s
YAML Linter / YAML Linter (pull_request) Successful in 7m46s
Test Packaging / Test Packaging (pull_request) Successful in 22m6s
757b5e76d8
Contributor

YIPEE

YIPEE
Contributor

shonk

shonk
@ -0,0 +21,4 @@
marking-FeroxiTailSpeckle-feroxi-tail-speckle = Speckle
marking-FeroxiTailSpots = Spots
marking-FeroxiTailSpots-feroxi-tail-Spots = Spots
Owner

Lower case spots.

Lower case spots.
@ -0,0 +60,7 @@
marking-FeroxiRightLegOverside = Overside
marking-FeroxiRightLegOverside-feroxi-right-leg-overside = Overside
marking-FeroxiLeftLegRacingstripe = Racing Stripe
marking-FeroxiLeftLegRacingstripe-feroxi-left-leg-racingstripe = Racing Stripe
marking-FeroxiRightLegRacingstripe = Racing Stripe
marking-FeroxiRightLegRacingstripe-feroxi-right-leg-racingstripe = Racing Stripe
Owner

FeroxiLeftLegRacingstripe related ftl strings are unused.

`FeroxiLeftLegRacingstripe` related ftl strings are unused.
@ -0,0 +83,4 @@
marking-FeroxiBodyEmbraced-feroxi-body-embraced = Embraced
marking-FeroxiBodyEncircled = Encircled
marking-FeroxiBodyEncircled-feroxi-body-centerpiece = Encircled
Owner

encircled instead of centerpiece.

`encircled` instead of `centerpiece`.
@ -594,4 +395,0 @@
- sprite: _Goobstation/Mobs/Customization/Feroxi/tail_markings.rsi
state: feroxi-dorsal
- sprite: _Goobstation/Mobs/Customization/Feroxi/tail_markings.rsi
state: feroxi-dorsal-tip
Owner

Would be better if you comment them out.

Would be better if you comment them out.
@ -302,4 +199,0 @@
- sprite: _Goobstation/Mobs/Customization/Feroxi/tail_markings.rsi #RedTerror Goob Resprite
state: feroxi-dorsal
- sprite: _Goobstation/Mobs/Customization/Feroxi/tail_markings.rsi #RedTerror Goob Resprite
state: feroxi-dorsal-tip
Owner

Would be better if you comment them out.

Would be better if you comment them out.
@ -108,0 +186,4 @@
speciesRestriction: [ Feroxi ]
sprites:
- sprite: _Goobstation/Mobs/Customization/Feroxi/tail_markings.rsi #RedTerror Goob Resprite
state: feroxi-tail-under
Owner

Those are Byrd additions basically, so just put them under _Byrd.

Those are Byrd additions basically, so just put them under _Byrd.
@ -0,0 +55,11 @@
"name": "feroxi-right-leg-overside",
"directions": 4
},
{
"name": "feroxi-left-leg-racingstripe",
"directions": 4
},
{
"name": "feroxi-right-leg-racingstripe",
"directions": 4
},
Owner

leg-racingstripes are unused.

`leg-racingstripe`s are unused.
Member

You need to re-create the PR since your repo got deleted.

Fork the repo again with the same name as before, do a git push, and then create a new PR.

You need to re-create the PR since your repo got deleted. Fork the repo again with the same name as before, do a `git push`, and then create a new PR.
Some checks failed
Close PRs on master / run (pull_request_target) Has been skipped
Labels: PR / labeler (pull_request_target) Successful in 7s
Labels: Size / size-label (pull_request_target) Failing after 5s
CRLF Check / CRLF Check (pull_request) Successful in 53s
RGA schema validator / YAML RGA schema validator (pull_request) Successful in 58s
RSI Validator / Validate RSIs (pull_request) Successful in 1m3s
Map file schema validator / YAML map schema validator (pull_request) Successful in 7m39s
YAML Linter / YAML Linter (pull_request) Successful in 7m46s
Test Packaging / Test Packaging (pull_request) Successful in 22m6s
This pull request is broken due to missing fork information.
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
5 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
Byrd-Station/Byrd!949
No description provided.