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

[MODULAR] adds intents #14

Merged
merged 8 commits into from
Jul 26, 2024
Merged

[MODULAR] adds intents #14

merged 8 commits into from
Jul 26, 2024

Conversation

NullDagaf
Copy link
Collaborator

About The Pull Request

adds intents
needs heavy testing

Why It's Good For The Game

soul

Changelog

🆑
add: adds intents
/:cl:

Copy link
Collaborator

@liz-lavenza liz-lavenza left a comment

Choose a reason for hiding this comment

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

Just some small changes (mostly comments) I'd like to request for maintainability reasons. Code-wise, this looks good, though I'd also like it if it were documented somewhere what exactly the non-help/harm intents do now—in that disarm intent makes you do right-click interactions by default, and grab intent makes you do ctrl-click interactions. These things may be obvious to you (and me, after I looked over the code), but we want contributors to be able to get up to speed on the new codebase as fast as possible.

code/__DEFINES/~~~splurt_defines/intents.dm Show resolved Hide resolved
modular_zzplurt/code/modules/client/click.dm Show resolved Hide resolved
Copy link
Collaborator

@liz-lavenza liz-lavenza left a comment

Choose a reason for hiding this comment

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

I think we can ignore non-mob intent interactions right now; the two important things are that it works for swapping when moving (and it does, or at least should), and that it works for basic mob interactions like shoving and grabbing. This should also be a good foundation for things like the interactions menu, etc.

All that said, looks good to merge for further playtesting, given that you've already done some non-exhaustive local playtesting to rule out obvious issues.

@MosleyTheMalO MosleyTheMalO merged commit ec3f767 into master Jul 26, 2024
25 of 26 checks passed
github-actions bot added a commit that referenced this pull request Jul 26, 2024
MosleyTheMalO pushed a commit that referenced this pull request Oct 25, 2024
# This is the 1st commit message:

goofsec hud

# This is the commit message #2:

the sensible tgui errors

# This is the commit message #3:

Gets it to compile

# This is the commit message #4:

Fixes all extra runtimes

# This is the commit message #5:

TGUI Fixes 1

# This is the commit message #6:

Removes open dream check

# This is the commit message #7:

Ticked file enforcement

# This is the commit message #8:

A very harsh way to fix potential missing icons

# This is the commit message #9:

Wipes all the errorneously included files

# This is the commit message #10:

fixes missing_icons fully

# This is the commit message #11:

some extra ci fixes before I forget

# This is the commit message #12:

WHAT THE FUCK

# This is the commit message #13:

really?

# This is the commit message #14:

bye waffles

# This is the commit message #15:

hooray
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants