Skip to content

Pool table mechanics#675

Open
FalloutFalcon wants to merge 10 commits intoDarkPack13:masterfrom
FalloutFalcon:pool-table-mechanics
Open

Pool table mechanics#675
FalloutFalcon wants to merge 10 commits intoDarkPack13:masterfrom
FalloutFalcon:pool-table-mechanics

Conversation

@FalloutFalcon
Copy link
Member

@FalloutFalcon FalloutFalcon commented Feb 15, 2026

About The Pull Request

Functioning Pool!
Tables now spawn with a full set of balls and some pool cues. Attacking the table with a cue lets you choose a ball and then rolls your stats to see what you hit.

2026-02-15.09-44-05.mp4

Why It's Good For The Game

excellent barrp material.
A fun little use for streetwise as its missing a way to express it at the moment

Changelog

🆑
add: Pool tables has balls and cue's that creates a little 8-Ball mini-game.
/:cl:

@chazzyjazzy
Copy link
Contributor

Linter failure is pool cue missing worn icon state

Copy link
Contributor

@chazzyjazzy chazzyjazzy left a comment

Choose a reason for hiding this comment

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

really solid coding, this should probably be the gold standard as an intro to byond coding

Comment on lines 161 to 167
/obj/structure/table/wood/billiard/proc/sink_ball(mob/living/user, target_ball_type, accuracy_result, amount_to_hit_result, obj/item/pool_ball/sunk_ball)
if(!sunk_ball)
sunk_ball = random_ball(target_ball_type, accuracy_result)

if(!sunk_ball)
//user.visible_message(span_warning("[user] MISSED"), span_warning("You missed"))
return
Copy link
Contributor

Choose a reason for hiding this comment

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

correct me if im wrong but 165 166 and 167 will only ever run if there are literally zero balls on the table so the comment can probably be deleted but the conditional should most likely remain?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants