Skip to content

Don't display silver rupee counter if dungeon info is displayed - #2610

Merged
fenhl merged 1 commit into
OoTRandomizer:Devfrom
GSKirox:fix_silver_counter_overlap
Jul 24, 2026
Merged

Don't display silver rupee counter if dungeon info is displayed#2610
fenhl merged 1 commit into
OoTRandomizer:Devfrom
GSKirox:fix_silver_counter_overlap

Conversation

@GSKirox

@GSKirox GSKirox commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

If you enable everything that makes the dungeon info screen larger (silver rupees + MQ + shuffle CMG keys), it goes left to the rupee/keys/silver rupee counter area, and since the silver rupee counter is drawn after the dungeon info screen, they overlap.
image

An easy fix is just to not display the silver rupee counter is any dungeon info screen is on.
You could also display the counter first, then the info screen on top of it, but imo the information is not really important to keep on screen while the player is looking at the other dungeon info, and two of the dungeon info screens display the silver rupee count anyway.

Testing

Tested on Retroarch that the silver rupee counter is not displayed anymore if any dungeon info screen is displayed.

@GSKirox GSKirox added Type: Bug Something isn't working Component: ASM/C Changes some internals of the ASM/C libraries Status: Needs Review Someone should be looking at it labels Jul 24, 2026
@fenhl fenhl removed the Status: Needs Review Someone should be looking at it label Jul 24, 2026
@fenhl fenhl added this to the next milestone Jul 24, 2026
@fenhl
fenhl merged commit 8c230ff into OoTRandomizer:Dev Jul 24, 2026
6 checks passed
@r0bd0g

r0bd0g commented Jul 24, 2026

Copy link
Copy Markdown

Aw I'd hoped it would go behind the info the way that rupees and keys do, like it was always part of the game. :p

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component: ASM/C Changes some internals of the ASM/C libraries Type: Bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants