Skip to content

entrance shuffle: sign hints#5335

Merged
Malkierian merged 9 commits intoHarbourMasters:developfrom
serprex:sign-hint
Sep 17, 2025
Merged

entrance shuffle: sign hints#5335
Malkierian merged 9 commits intoHarbourMasters:developfrom
serprex:sign-hint

Conversation

@serprex
Copy link
Contributor

@serprex serprex commented Apr 6, 2025

This makes it so signs by entrances will say where the exit leads to rather than where it leads to in vanilla

Particularly useful with decoupled entrances

Build Artifacts

@serprex serprex marked this pull request as draft April 6, 2025 04:02
@Malkierian
Copy link
Contributor

The spoiled state of an entrance should already be handled somewhere with the entrance tracker.

@serprex
Copy link
Contributor Author

serprex commented Apr 6, 2025

The spoiled state of an entrance should already be handled somewhere with the entrance tracker.

No, this PR makes it so when you talk to a sign (mostly non-cuttable arrow signs) it'll hint the entrance to you. Marking it spoiled would be similar to #4568

@Malkierian
Copy link
Contributor

Ah, I see.

@serprex
Copy link
Contributor Author

serprex commented Apr 11, 2025

waiting on #5363 to update this PR with tracker integration

@serprex serprex marked this pull request as ready for review May 24, 2025 01:46
@serprex
Copy link
Contributor Author

serprex commented May 24, 2025

This is now in a state where it could be merged

Copy link
Contributor

@Malkierian Malkierian left a comment

Choose a reason for hiding this comment

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

I agree that this should just be automatic and not be a setting.

@serprex
Copy link
Contributor Author

serprex commented Jun 29, 2025

Need to test how this handles mixed entrances where destination is grotto because grotto entrance IDs are weird

@serprex
Copy link
Contributor Author

serprex commented Jun 30, 2025

Tested with grottos. Works

Copy link
Contributor

@Pepe20129 Pepe20129 left a comment

Choose a reason for hiding this comment

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

The text names could be more descriptive

@Malkierian
Copy link
Contributor

Pablo has a point, some of those text enum names are too vague, like the TEXT_DMT_DMT one. Make them a bit more descriptive, and I'll get this merged.

@serprex
Copy link
Contributor Author

serprex commented Aug 25, 2025

Thanks, will look into when I'm home next week

@serprex
Copy link
Contributor Author

serprex commented Sep 1, 2025

TEXT_DMT_DMT is a little weird since it's a sign halfway up DMT that just says "Death Mountain Trail". Adding _SIGN at end of names, TEXT_DMT_SIGN feels kinda generic compared to TEXT_DMT_DMT_SIGN pointing out this oddity, but DMT_DMT pretty easy to come off as a typo

going with TEXT_DMT_SIGN

@Malkierian Malkierian merged commit e6663a1 into HarbourMasters:develop Sep 17, 2025
6 checks passed
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