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

fix: update worldIdentifiers list in RemapWorlds #239

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Burrik
Copy link

@Burrik Burrik commented Feb 12, 2025

Description

Fixed an issue where World Browser was only showing the Default world because worldIdentifiers list wasn't being updated when new worlds were created.

Problem

When creating additional worlds (besides Default), they weren't visible in the World Browser dropdown because worldIdentifiers list in Hierarchy class wasn't being updated in RemapWorlds method.

Solution

Updated RemapWorlds method to clear and repopulate worldIdentifiers list with all existing worlds.

How to Test

  1. Create a new world besides Default
  2. Open World Browser window
  3. Click "W" button in the top toolbar
  4. Verify that all worlds are visible in the dropdown

Before Fix

Only Default world was visible in World Browser dropdown

After Fix

All worlds are visible in World Browser dropdown

The World Browser was only showing the Default world because worldIdentifiers
list wasn't being updated with new worlds. This fix ensures all worlds are
visible in the World Browser dropdown.
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.

1 participant