Skip to content

Improve tooltips - #60

Merged
flee135 merged 21 commits into
wafo:masterfrom
tanjo3:improve-tooltips
Jul 25, 2026
Merged

Improve tooltips#60
flee135 merged 21 commits into
wafo:masterfrom
tanjo3:improve-tooltips

Conversation

@tanjo3

@tanjo3 tanjo3 commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

Some changes to improve/correct item tooltips:

  • Support the new closed Door of Time & starting age tracker support.
  • Properly support dev-fork generator versions by resolving to the right fork/branch for logic files, and normalize fork settings before evaluation.
  • Broaden tooltip converter coverage to handle has_all_notes_for_song, region_has_shortcuts, bombchu bags, and other previously-unsupported settings. Should fix the logic for the SAWS tooltips.
  • Fix some bugs with bottle requirements, such as the logic not considering that the player must go adult to empty a Bottle with Big Poe.
  • Some performance changes. I found that I had a lot of latency upon the first hover (i.e., the first time tooltips get generated). Moved most of the computation to when the tracker is initialized and the starting age is set, so that the user doesn't have low responsiveness while they're actively using the tracker. Some memoing/caching stuff added and improved to keep latency at these points low.
  • Fix a bug with SoT notes and some missing regions/locations.

@flee135 flee135 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LG overall, just the one consideration that we discussed briefly offline about how we show songs in tooltips.

Comment thread src/utils/expression-converter.js
@flee135
flee135 merged commit 0aad58e into wafo:master Jul 25, 2026
1 check passed
@tanjo3
tanjo3 deleted the improve-tooltips branch July 26, 2026 00:54
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