Skip to content

fix(firered): match bag, item PC, and field UI - #72

Merged
UNDERdecoded merged 3 commits into
UNDERdecoded:mainfrom
Lokesh-shiva:firered-visual-bugs-03
Oct 19, 2026
Merged

UNDERdecoded merged 3 commits into
UNDERdecoded:mainfrom
Lokesh-shiva:firered-visual-bugs-03

Conversation

@Lokesh-shiva

Copy link
Copy Markdown
Contributor

Summary

  • match FireRed Bag and Player PC item-storage visuals to retail, including CANCEL/return rows, item descriptions, quantity/result windows, and the dedicated WITHDRAW screen
  • add FireRed Item-PC submenu/message/quantity rendering with the correct standard frames, tile flips, item icon/description layout, and no edge bleed
  • carry FireRed field-message context into the renderer so Mom uses the female red face, HELP uses the signpost frame, and the empty MAILBOX message removes/restores the top PC menu
  • extract/use the dedicated FRLG male/female font faces and correct field text/signpost metrics

Validation

  • FireRed visual driver: RESULT pass=24 fail=0
  • production Player PC deposit and withdraw mutations pass
  • production Mom, HELP sign, and MAILBOX interactions captured at exact 4x integer scale
  • git diff --check clean before commit
  • fresh FireRed cache bumped to rom-cache-v365

Stacking

This branch is based on the still-open #71 (firered-visual-bugs-02), so GitHub will temporarily show those earlier commits in this PR until #71 merges.

Lokesh-shiva and others added 3 commits October 12, 2026 18:53
Font.lua: keep this PR's prepareFrameStrip/drawDialogueRecord split and add
main's Font.dialogueBoxBounds alongside it.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Gen3ItemPcQuantity: keep the Deposit/Withdraw verb and dark style, add main's
strings-gate marker on the line-break glue.
Gen3PlayerPC: route the withdrew fallback through Strings() so
gate_strings_coverage passes.
RomImporter: v367 for this PR's extraction on top of main's v366.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@UNDERdecoded
UNDERdecoded merged commit e0af342 into UNDERdecoded:main Oct 19, 2026
3 of 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.

2 participants