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

Reduces taj/vulp darksight to 3 tiles again #28324

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Bm0n
Copy link
Contributor

@Bm0n Bm0n commented Feb 10, 2025

What Does This PR Do

Lowers taj/vulp darksight to 3 tiles again.

Why It's Good For The Game

Taj/vulp darksight was originally lowered to 3 in #20035 but was raised up to 4 in #20275 with the addition of a malus.

This malus was removed in #23642 but darksight remained at 4.

Taj/vulps have very minor maluses, they should not be getting 4 darksight for what they currently have.

Images of changes

#20035 Has pictures of what 3 darksight looks like. This is one extra tile over human eyes.

Testing

It compiles.


Declaration

  • I confirm that I either do not require pre-approval for this PR, or I have obtained such approval and have included a screenshot to demonstrate this below.

Changelog

🆑
tweak: Taj/vulp darksight lowered to 3 again.
/:cl:

@ParadiseSS13-Bot ParadiseSS13-Bot added the -Status: Awaiting approval This PR is waiting for approval internally label Feb 10, 2025
@1080pCat
Copy link
Contributor

1080pCat commented Feb 10, 2025

hope you have approval for this?
or would it fall under a oversight fix since it wasn't removed then the malus was

@Bm0n
Copy link
Contributor Author

Bm0n commented Feb 11, 2025

hope you have approval for this? or would it fall under a oversight fix since it wasn't removed then the malus was

Had a word with contra about it last night and he said it was okay to keep up (the answer is no)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-Status: Awaiting approval This PR is waiting for approval internally
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants