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

Clearer hint text for bridge condition #2105

Merged
merged 2 commits into from
Nov 29, 2023

Conversation

fenhl
Copy link
Collaborator

@fenhl fenhl commented Sep 30, 2023

This makes 2 changes to the text box that gives the rainbow bridge condition as part of the message shown when reading the Temple of Time altar as adult:

  • If the “Clearer Hints” setting is enabled, the text is changed from “The awakened ones will await for the Hero to collect” to “The rainbow bridge will be built once the Hero collects”.
    • The text for open bridge is unchanged, since it already mentions the bridge: “The awakened ones will have already created a bridge to the castle where the evil dwells.”
  • Regardless of settings, the light arrow icon, which has misled some people into thinking the text was hinting the location of the light arrows, is removed. This is similar to the change made to the child altar text in Rework the ToT altar text to not be misleading and provide more information #1102.

Fixes #1473.

image

There's something weird going on with the indentation in this text box. Maybe our line wrapping algorithm is getting confused by the previous boxes in this message having icons?

@fenhl fenhl added the Component: Hints related to how we help the player label Sep 30, 2023
@r0bd0g
Copy link

r0bd0g commented Sep 30, 2023

I wonder if that indent is forced by the game, and that's why there was randomly a picture of the light arrows there before. Like I don't think it was put there b/c a picture of the light arrows would be nice, but because some icon had to be used to avoid the weird indent.

@GSKirox
Copy link
Collaborator

GSKirox commented Sep 30, 2023

I don't think this is caused by the line wrap function, i suspect this is a vanilla issue when drawing consecutive textboxes, since i don't see any vanilla textbox that has an icon at first then remove it, they always keep an icon on the left.
So we could either :

@r0bd0g
Copy link

r0bd0g commented Sep 30, 2023

I feel like I remember a discussion about this back when the icons of the medallions were added, long time ago now.

@ETR-BTF
Copy link

ETR-BTF commented Sep 30, 2023

I don't think this is caused by the line wrap function, i suspect this is a vanilla issue when drawing consecutive textboxes, since i don't see any vanilla textbox that has an icon at first then remove it, they always keep an icon on the left. So we could either :

How about an "empty" icon that's just a transparent square? Would that be easier to implement?

@GSKirox
Copy link
Collaborator

GSKirox commented Oct 1, 2023

I don't see an easy way to do that, might be easier to just fix it.

@fenhl
Copy link
Collaborator Author

fenhl commented Oct 26, 2023

I've added the Master Sword icon for now. I guess we'll have to fix this properly before Master Sword shuffle.

image

@cjohnson57 cjohnson57 added Type: Enhancement New feature or request Status: Needs Review Someone should be looking at it Status: Waiting for Release This PR is ready for merge, but we're holding off on it until after the next release and removed Status: Needs Review Someone should be looking at it Status: Waiting for Release This PR is ready for merge, but we're holding off on it until after the next release labels Nov 11, 2023
Copy link
Collaborator

@cjohnson57 cjohnson57 left a comment

Choose a reason for hiding this comment

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

Looks good

@cjohnson57 cjohnson57 merged commit b821966 into OoTRandomizer:Dev Nov 29, 2023
3 checks passed
@fenhl fenhl deleted the rainbow-bridge-hint branch November 30, 2023 01:53
@fenhl fenhl added this to the 8.1 milestone Mar 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Hints related to how we help the player Type: Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clearer Hints Text Needed
5 participants