HarbourMasters / Shipwright

3.21k stars 485 forks source link

[UI/UX] Indicate which inventory slots have alternative items you can cycle between #2605

Closed TheBeardOfTruth closed 1 year ago

TheBeardOfTruth commented 1 year ago

EDIT: You can apparently cycle the trade items, more of a UI/UX issue then. Updated title.

In the current seed I'm playing Anju gives child link cojiro after fetching cuccos, later in the same seed a business scrub sells pocket egg, overwriting cojiro resulting in the inability to get that check.

"_seed": "971059955",
"_version": "KHAN ALFA (6.1.0)",

I couldn't find a good way of exporting the randomizer settings so I screenshotted all panels where I've made changes.

Screenshot_2023-03-09_22-38-44 Screenshot_2023-03-09_22-38-29 Screenshot_2023-03-09_22-38-24 Screenshot_2023-03-09_22-38-20

In this specific case it doesn't softlock the sequence the way I progressed, because the spoiler log reveals that cojiro trades for a red rupee and pocket egg trades for shadow medallion.

However if I had checked impa's house -> three scrub grotto first and gotten pocket egg it would have softlocked the seed.

Obvious solutions include either locking later trade sequence items after earlier ones, or shuffling a random trade sequence item into the pool and then continuing the sequence from normal as there (this is the way the "normal" randomizer for the N64 rom does it, if memory serves), though there's probably a more elegant solution.

Archez commented 1 year ago

The tooltip for "Shuffle Adult Trade" sequence explains how it works in Ship of Harkinian

Adds all of the adult trade quest items into the pool, each of which can be traded for a unique reward.

You will be able to choose which of your owned adult trade items is visible in the inventory by selecting the item with A and using the control stick or D-pad.

It's not visually apparent from the pause menu, but you just press A when hovering over the trade item to cycle to other items you have collected.

TheBeardOfTruth commented 1 year ago

I see, I must have missed that tooltip then. Thanks for the explanation, I suppose it might be useful to add some UI hint (if I don't misremember, ALTTP randomizer has that functionality), perhaps an A button badge on the item?