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

[PORT] RPD QOL improvements #7708

Merged
merged 13 commits into from
Oct 28, 2022
Merged

Conversation

mystery3525
Copy link
Contributor

@mystery3525 mystery3525 commented Sep 19, 2022

About The Pull Request

Ports:

various RPD quality life improvements from an entire two years that we missed out on

Why It's Good For The Game

holy shit where have you been all my life

Testing Photographs and Procedure

Screenshots&Videos
dreamseeker_vS65yEkOhF.mp4

image

Changelog

🆑 Skoglol, Dennok, TheChosenEvilOne, Timberpoes, Anturk, DuffCreeper, vincentiusvin, ported by mystery3525
tweak: RPD subcategories and preview icons reorganized.
add: RPD now starts with painting turned off, hitting pipes with build and no paint will target the turf underneath instead. Bye bye turf pixelhunting.
add: Reverse wrench mode upgrade to the RPD. Attention, due to budget cuts, the mode is hard linked to the destroy mode control button.
add: You can now change piping layers on the RPD by scrolling with the mouse wheel.
fix: Various cumulative fixes to RPDs
add: RPD can destroy broken disposal pipe
balance: Securing a disposal pipe over a floor tile with a crowbar in your offhand will remove the floor tile before trying to secure it
/:cl:

@itsmeow
Copy link
Member

itsmeow commented Sep 19, 2022

holy shit

@CydiaLamiales
Copy link
Contributor

Yess, triple layering the space loop will now be nearly not as painful, thank you.

Copy link
Member

@Crossedfall Crossedfall left a comment

Choose a reason for hiding this comment

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

You must show testing results for your PR. You can't outsource testing to the parent PRs, it needs to be shown that it works here.

code/__DEFINES/construction.dm Show resolved Hide resolved
code/game/objects/items/RPD.dm Show resolved Hide resolved
code/game/objects/items/RPD.dm Outdated Show resolved Hide resolved
code/game/objects/items/RPD.dm Show resolved Hide resolved
* * user - mob that use upgrade on RPD
*/
/obj/item/pipe_dispenser/proc/install_upgrade(obj/item/rpd_upgrade/rpd_up, mob/user)
if(rpd_up.upgrade_flags& upgrade_flags)
Copy link
Member

Choose a reason for hiding this comment

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

this is extremely bothersome

Copy link
Contributor Author

Choose a reason for hiding this comment

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

allows for future upgrades I guess? direct port from /tg/, though it interferes with /obj/item/pipe_dispenser/plumbing so I added an override...

Copy link
Member

Choose a reason for hiding this comment

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

i meant the syntax around &, I hate it

code/game/objects/items/RPD.dm Outdated Show resolved Hide resolved
code/game/objects/items/RPD.dm Outdated Show resolved Hide resolved
code/game/objects/items/RPD.dm Outdated Show resolved Hide resolved
code/game/objects/items/RPD.dm Outdated Show resolved Hide resolved
code/modules/research/designs/tool_designs.dm Outdated Show resolved Hide resolved
@PowerfulBacon
Copy link
Member

Unaddressed reviews

@github-actions github-actions bot closed this Oct 5, 2022
@mystery3525
Copy link
Contributor Author

@itsmeow can you reopen this pr reviews are fully addressed in latest

@github-actions
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@itsmeow
Copy link
Member

itsmeow commented Oct 27, 2022

screwed up your master merge, you need to re add the signal

 code/_onclick/click.dm:495:error: COMSIG_MOUSE_SCROLL_ON: undefined var
code/_onclick/click.dm:495:error: COMSIG_MOUSE_SCROLL_ON: undefined var
code/game/objects/items/RPD.dm:264:error: COMSIG_MOUSE_SCROLL_ON: undefined var
code/game/objects/items/RPD.dm:266:error: COMSIG_MOUSE_SCROLL_ON: undefined var
code/game/objects/items/RPD.dm:269:error: COMSIG_MOUSE_SCROLL_ON: undefined var
code/game/objects/items/RPD.dm:273:error: COMSIG_MOUSE_SCROLL_ON: undefined var
code/game/objects/items/RPD.dm:594:error: COMSIG_MOUSE_SCROLL_ON: undefined var

signals have different files for stuff now

@mystery3525 mystery3525 requested review from itsmeow and removed request for Crossedfall October 27, 2022 16:33
@itsmeow
Copy link
Member

itsmeow commented Oct 27, 2022

"Various cumulative fixes" in CL is very unclear

@mystery3525
Copy link
Contributor Author

copy pasted from /tg/

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.

None yet

5 participants