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

Using a multitool to find an APC now highlights the APC #7020

Merged
merged 4 commits into from
Aug 24, 2024

Conversation

TheGamerdk
Copy link
Contributor

@TheGamerdk TheGamerdk commented Aug 23, 2024

About the pull request

Makes the find-APC function more useful.

Explain why it's good for the game

Very convenient. Lasts for 1.4 seconds (scan cooldown is 1.5 seconds)
Clientside, respects darkness.
You still get the 'X units in Y direction' thing

image

Testing Photographs and Procedure

Spawned in, tried to use multitool, saw green.
Turned off lights, did not see green.
Joined with second client, used multitool. Only saw it on 1 client.
Left after clicking multitool, rejoined to make sure highlight is gone. (Client gets deleted when disconnecting)

Changelog

🆑
add: APCs are now highlighted when found via a Multitool/Security Access Tuner.
/:cl:

@github-actions github-actions bot added the Feature Feature coder badge label Aug 23, 2024
Copy link
Member

@harryob harryob left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same thing

code/game/objects/items/devices/multitool.dm Outdated Show resolved Hide resolved
@TheGamerdk
Copy link
Contributor Author

same thing

true that

@harryob harryob added this pull request to the merge queue Aug 24, 2024
Merged via the queue into cmss13-devs:master with commit 116b8f0 Aug 24, 2024
27 checks passed
cm13-github added a commit that referenced this pull request Aug 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Feature coder badge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants