Skip to content

Conversation

@JoryAnderson
Copy link

Hides cargo hold options while in a Barracuda Trial and at helm, preventing unintended stop-nav when setting/trimming sails.
Adds config information to the README, as well as missing information for HideStopNavigatingDuringTrials

@JoryAnderson
Copy link
Author

JoryAnderson commented Dec 4, 2025

Hoping to add another option or two over the weekend for non-wind/extractor facilities - will make a separate PR for those.

@LlemonDuck
Copy link
Owner

This might be better just merged into HideStopNavigatingDuringTrials as a broader LockToHelmDuringTrials?

@JoryAnderson
Copy link
Author

Good point, bringing back to the lab.

@JoryAnderson JoryAnderson marked this pull request as draft December 5, 2025 07:20
@JoryAnderson JoryAnderson marked this pull request as ready for review December 5, 2025 07:25
@JoryAnderson
Copy link
Author

JoryAnderson commented Dec 5, 2025

That was quick and easy to do. I will add to this soon to include non-navigational facilities, but for now I think this is a good start. My biggest problem was clicking through the sail when setting/trimming and clicking cargo hold instead.

@JoryAnderson JoryAnderson changed the title Added: HideCargoHoldDuringBT menu-entry swap Added: LockToHelmDuringTrials menu-entry swap Dec 5, 2025
@JoryAnderson
Copy link
Author

Unless you think this should wait until facilities are included this should be good for review now.

@JoryAnderson
Copy link
Author

JoryAnderson commented Dec 6, 2025

This is now complete - With this config enabled, all non-Wind Facilities are de-prioritized while the following criteria is met:

  • Player is in a Barracuda Trial, AND
  • Player is at the helm of their ship.

Interacting with the wind facilitates doesn't force you away from the helm, unlike the other facilities. Players can click their catcher/extractor to activate their effects, or use the Sailing menu. Players can always right-click helm/other facilities to access the de-prioritized interactions during a trial. No more botched trials from clicking through the sail when setting/trimming 😄

@JoryAnderson JoryAnderson force-pushed the DeprioCargoHoldDuringBT branch from a405a54 to 1fb7f36 Compare December 6, 2025 14:29
@JoryAnderson
Copy link
Author

Rebased on top of latest.

@JoryAnderson JoryAnderson force-pushed the DeprioCargoHoldDuringBT branch from 621decd to 594e83f Compare December 14, 2025 02:48
@JoryAnderson
Copy link
Author

Rebased. If there's anything you'd like me to touch on or review please let me know :)

@JoryAnderson JoryAnderson marked this pull request as draft December 23, 2025 01:15
@JoryAnderson
Copy link
Author

Trimming down the changes, slight overhaul

@JoryAnderson JoryAnderson force-pushed the DeprioCargoHoldDuringBT branch from aba29e6 to 2ae3a13 Compare December 23, 2025 04:21
JoryAnderson added a commit to JoryAnderson/sailing that referenced this pull request Dec 23, 2025
@JoryAnderson JoryAnderson marked this pull request as ready for review December 23, 2025 04:22
@JoryAnderson
Copy link
Author

Fixed codestyle and commits squashed.

@JoryAnderson JoryAnderson force-pushed the DeprioCargoHoldDuringBT branch from 2ae3a13 to 40e0aae Compare December 23, 2025 04:28
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.

2 participants