Add keycard pickup, keycard reader, and unlockable door #38

noahg · 2026-03-14T00:19:39Z

noahg commented

2026-03-14 00:19:39 +00:00

Implements https://tasks.bugjam.dev/tasks/165

noahg requested reviews from xury, paco, a_kimb0, devonereynolds, TranquilMarmot

2026-03-14 00:20:08 +00:00

TranquilMarmot reviewed

2026-03-14 05:45:10 +00:00

interactable_objects/interaction_area.gd Outdated

					
				@ -5,0 +5,4 @@

				## The [b]local[/b] direction the player must be from this area to be able to interact with it.

				## [br]Must be set to one of [constant Vector3i.ZERO], [constant Vector3i.LEFT], [constant Vector3i.RIGHT], [constant Vector3i.UP], [constant Vector3i.DOWN], [constant Vector3i.FORWARD], or [constant Vector3i.BACK]

				## [br][b]Note:[/b] When set to [constant Vector3i.ZERO] this area can be interacted with from any direction

				@export var _interaction_direction: Vector3i = Vector3i.UP:

Would this work better as an enum?

enum InteractionDirection {
	ANY,
	LEFT,
	RIGHT,
	UP,
	DOWN,
	FORWARD,
	BACK,
}

@export var interaction_direction: InteractionDirection = InteractionDirection.ANY


func interaction_direction_to_vector(direction: InteractionDirection) -> Vector3:
	match direction:
		InteractionDirection.ANY:
			return Vector3.ZERO
		InteractionDirection.LEFT:
			return Vector3.LEFT
		InteractionDirection.RIGHT:
			return Vector3.RIGHT
		InteractionDirection.UP:
			return Vector3.UP
		InteractionDirection.DOWN:
			return Vector3.DOWN
		InteractionDirection.FORWARD:
			return Vector3.FORWARD
		InteractionDirection.BACK:
			return Vector3.BACK
		_:
			return Vector3.ZERO

Would this work better as an enum? ```gdscript enum InteractionDirection { ANY, LEFT, RIGHT, UP, DOWN, FORWARD, BACK, } @export var interaction_direction: InteractionDirection = InteractionDirection.ANY func interaction_direction_to_vector(direction: InteractionDirection) -> Vector3: match direction: InteractionDirection.ANY: return Vector3.ZERO InteractionDirection.LEFT: return Vector3.LEFT InteractionDirection.RIGHT: return Vector3.RIGHT InteractionDirection.UP: return Vector3.UP InteractionDirection.DOWN: return Vector3.DOWN InteractionDirection.FORWARD: return Vector3.FORWARD InteractionDirection.BACK: return Vector3.BACK _: return Vector3.ZERO ````

It definitely would! Thanks for the suggestion. I just made the change. I also added comments to each value to show which vector it is equivalent to in the editor hint. Hopefully it helps make it clearer how to deal with the two counter intuitive values of FORWARD and BACK because of Godot's coordinate system having Z be negative for forward.

It definitely would! Thanks for the suggestion. I just made the change. I also added comments to each value to show which vector it is equivalent to in the editor hint. Hopefully it helps make it clearer how to deal with the two counter intuitive values of `FORWARD` and `BACK` because of Godot's coordinate system having Z be negative for forward.

noahg marked this conversation as resolved