Skip to content

Clarify hint text for pots in interiors#5591

Merged
Malkierian merged 2 commits intoHarbourMasters:develop-blairfrom
Pepper0ni:PotHintArea
Jun 18, 2025
Merged

Clarify hint text for pots in interiors#5591
Malkierian merged 2 commits intoHarbourMasters:develop-blairfrom
Pepper0ni:PotHintArea

Conversation

@Pepper0ni
Copy link
Contributor

@Pepper0ni Pepper0ni commented Jun 16, 2025

Interiors are not always in their starting hint area, which can cause confusion when pot location hints refer to it. This PR resolves that by giving each interior and grotto it's own pot hint.

Build Artifacts

/*french*/ "Selon moi, une #jarre dans le Village de Cocorico# contient #[[1]]#.", {QM_RED, QM_GREEN}));

hintTextTable[RHT_POT_GRAVEYARD] = HintText(CustomMessage("They say that a #pot in Graveyard# contains #[[1]]#.",
hintTextTable[RHT_POT_DAMPE] = HintText(CustomMessage("They say that a #pot in Dampe's Tomb# contains #[[1]]#.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Game calls it Dampe's Grave.

Copy link
Contributor Author

@Pepper0ni Pepper0ni Jun 16, 2025

Choose a reason for hiding this comment

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

I was being a bit flowery, but I guess it's a weird midpoint between "Dampe's grave" and "the gravekeeper's tomb"

EDIT: went for the less literal, as I'm probably going to make a "literal hints" mode at some point to address people's issues knowing what check is being referred to sometimes

@Malkierian Malkierian merged commit b932b8f into HarbourMasters:develop-blair Jun 18, 2025
6 checks passed
krazyjakee pushed a commit to krazyjakee/OOT that referenced this pull request Sep 6, 2025
* Clarify hint text for pots in interors

* Make dampe's pots text less like a naming error
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