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

[READY] Activation and Reload of specific items without picking them up #24732

Merged
merged 12 commits into from Aug 24, 2018

Conversation

Projects
None yet
8 participants
@Robik81
Copy link
Contributor

commented Aug 9, 2018

Closes #24686

What it does

This PR is mainly meant to save the player unnecessary tool shuffling inside of his base / crafting area.

Why I did it

There are various crafting tools that are necessary for crafting and which require batteries or charcoal to properly function. Under normal circumstances, player does not want to move such tools, just interact with them, but current implementation always picked the tool into player inventory, before any interaction could take place. Player then had to drop the tool back on the original tile. In some cases, as some of these tools are quite bulky, player had to first make more space in his or her inventory or manually wield the tool, which is even more unnecessary steps that needlessly break the flow of the game.

Similar problem is with light sources, which player place in his crafting area, to allow crafting in the evening hours... or sometimes even in the night shift. Player does not usually want to pick the lantern up, just to lit it up.

How it works

Crafting tools and light sources can be flagged to allow interaction from adjacent tile, without picking them up.

I added the flag to following tools:

Light sources

  • atomic lamp
  • atomic nightlight
  • gasoline_lantern
  • oil lamp
  • electric lantern

Tools

  • brick kiln
  • electric kiln
  • electric forge
  • charcoal forge
  • charcoal smoker
  • filled charcoal kiln (to lit it)
  • lit charcoal kiln (to check, if it is finished)
  • water purifier
  • charcoal water purifier
  • food dehydrator
  • food processor
  • welder
  • makeshift welder
  • vacuum sealer
  • makeshift vacuum sealer

image

Testing Area 51

@cainiaowu

This comment has been minimized.

Copy link
Contributor

commented Aug 9, 2018

With this one implemented, will the player clever enough to auto active the lighting during a long crafting session to avoid speed penalty/interupt?

@Robik81

This comment has been minimized.

Copy link
Contributor Author

commented Aug 9, 2018

With this one implemented, will the player clever enough to auto active the lighting during a long crafting session to avoid speed penalty/interupt?

No, the craft will still get interrupted. It's up to the player whether he then starts a fire or lit a lantern.

@cainiaowu

This comment has been minimized.

Copy link
Contributor

commented Aug 9, 2018

Well that is bad, most player dont even know that they can resume crafting without losing progress by "repeat last craft".

@Robik81

This comment has been minimized.

Copy link
Contributor Author

commented Aug 9, 2018

Well that is bad, most player dont even know that they can resume crafting without losing progress by "repeat last craft".

Could probably be better solved by expanding the "You can no longer see well enough to keep crafting." message to "You can no longer see well enough to keep crafting. Once you obtain a source of lighting, you can resume crafting by pressing x key." or something like that.

@Lorith

This comment has been minimized.

Copy link
Contributor

commented Aug 9, 2018

How about activating folded vehicles to unfold them so you no longer have an issue of being able to fold vehicles you can never unfold since they are too heavy to pick up?

@Robik81

This comment has been minimized.

Copy link
Contributor Author

commented Aug 9, 2018

How about activating folded vehicles to unfold them so you no longer have an issue of being able to fold vehicles you can never unfold since they are too heavy to pick up?

This issue is mentioned in #22771

I agree that unfolding makes more sense while folded vehicle lies on the ground. But it is out of the scope for this PR.

@Robik81

This comment has been minimized.

Copy link
Contributor Author

commented Aug 11, 2018

I expanded the list of tools that can be used remotely with following tools and updated the OP:

  • charcoal water purifier
  • food dehydrator
  • food processor
  • welder
  • makeshift welder
  • vacuum sealer
  • makeshift vacuum sealer

@Robik81 Robik81 force-pushed the Robik81:activate branch Aug 13, 2018

@Robik81 Robik81 changed the title Activation and Reload of specific items without picking them up [READY] Activation and Reload of specific items without picking them up Aug 14, 2018

@Robik81 Robik81 changed the title [READY] Activation and Reload of specific items without picking them up [WIP] Activation and Reload of specific items without picking them up Aug 14, 2018

@Robik81 Robik81 changed the title [WIP] Activation and Reload of specific items without picking them up [READY] Activation and Reload of specific items without picking them up Aug 14, 2018

@Robik81 Robik81 force-pushed the Robik81:activate branch to 0c37046 Aug 16, 2018

@Break-Off

This comment has been minimized.

Copy link

commented Aug 18, 2018

Does this fix #24413?

@Robik81

This comment has been minimized.

Copy link
Contributor Author

commented Aug 18, 2018

Does this fix #24413?

I would guess so, but only for tools mentioned here, not for every tool.

@ZhilkinSerg ZhilkinSerg self-assigned this Aug 24, 2018

@ZhilkinSerg ZhilkinSerg merged commit 53566d9 into CleverRaven:master Aug 24, 2018

4 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.01%) to 24.15%
Details
gorgon-ghprb Build finished.
Details

@ZhilkinSerg ZhilkinSerg removed their assignment Aug 24, 2018

@Robik81 Robik81 deleted the Robik81:activate branch Aug 25, 2018

@KurzedMetal

This comment has been minimized.

Copy link
Contributor

commented Aug 31, 2018

I really appreciate this PR

I really like being able to active stuff from the floor.
Sometimes fixing stuff like Storage Battery + Welder with Battery Compartment Mod that has a Car Battery because obnoxiously heavy and I don't see a reason why the player character needs to have both in their hands/inventory to make them work (could fix the battery directly on the floor only grabbing the welder gun)

@KurzedMetal

This comment has been minimized.

Copy link
Contributor

commented Aug 31, 2018

I was just checking the list of items that can be used to make a "meat pie" (just a random food I picked), and saw that several of the tools listed there don't have this flag but they arguably deserve it.

image

Like:

  • hotplate
  • mess kit
  • stove kit
  • multi cooker

There's probably a lot more tools that could include this new flag.
Are there any plans to expand this to other tools?
Or any guideline one what tools should receive it? Maybe I can submit a PR for other tools that follow that guideline.

@Robik81

This comment has been minimized.

Copy link
Contributor Author

commented Sep 2, 2018

Only 'guidelines' are

  1. has to make sense for given tool
  2. tool must work correctly with the new flag

Adding the flag might not be enough for some tools and additional changes in C++ might be necessary.

I do not plan to add the flag to other tools right now, but anyone can make such PR. Just make sure the tool works.

@Theundyingcode

This comment has been minimized.

Copy link
Contributor

commented Sep 11, 2018

With this one implemented, will the player clever enough to auto active the lighting during a long crafting session to avoid speed penalty/interupt?

No, the craft will still get interrupted. It's up to the player whether he then starts a fire or lit a lantern.

How difficult would it be to change the interrupt to a prompt to turn on a light in a subsequent PR?

@Robik81

This comment has been minimized.

Copy link
Contributor Author

commented Sep 11, 2018

I am not sure, such change has more to do with crafting code which I am not familiar with.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.