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

Open default advanced inventory layout doesn't clear container summary #75122

Closed
alef opened this issue Jul 20, 2024 · 0 comments · Fixed by #75289
Closed

Open default advanced inventory layout doesn't clear container summary #75122

alef opened this issue Jul 20, 2024 · 0 comments · Fixed by #75289
Labels
(S1 - Need confirmation) Report waiting on confirmation of reproducibility

Comments

@alef
Copy link
Contributor

alef commented Jul 20, 2024

Describe the bug

Described in review in #62547

Attach save file

N/A

Steps to reproduce

  1. Make sure Open default advanced inventory layout is enabled
  2. Save a layout with one pane [W] (Worn Items)
  3. Open AIM, scroll to a container, open it with c
  4. Close and reopen AIM
  5. Both <X> and >W< are selected, the container summary is shown

Expected behavior

Reset container summary and select only >W<.

Screenshots

Screenshot in #62547

Versions and configuration

  • OS: Windows
    • OS Version: 10.0.19045.4651 (22H2)
  • Game Version: 0.H-6420-g07bcddcd3 [64-bit]
  • Graphics Version: Tiles
  • Game Language: System language []
  • Mods loaded: [
    Dark Days Ahead [dda],
    Disable NPC Needs [no_npc_food],
    Portal Storms Ignore NPCs [personal_portal_storms],
    Slowdown Fungal Growth [no_fungal_growth]
    ]

Additional context

With the debugger I just skipped the one line container = save_state->container at advanced_inv_pane.cpp:85 and seems to work (Set Next Statement in Visual Studio)

@alef alef added the (S1 - Need confirmation) Report waiting on confirmation of reproducibility label Jul 20, 2024
@alef alef changed the title Open default advanced inventory layout do not clear container summary Open default advanced inventory layout doesn't clear container summary Jul 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
(S1 - Need confirmation) Report waiting on confirmation of reproducibility
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant