Skip to content

Consolidated bugfixes for version 2.0#1230

Open
Lwmte wants to merge 12 commits into
developfrom
lwmte/bugfixes_v20
Open

Consolidated bugfixes for version 2.0#1230
Lwmte wants to merge 12 commits into
developfrom
lwmte/bugfixes_v20

Conversation

@Lwmte
Copy link
Copy Markdown
Collaborator

@Lwmte Lwmte commented May 17, 2026

No description provided.

@Lwmte Lwmte requested a review from Copilot May 17, 2026 17:53
@Lwmte Lwmte changed the title Consolidated bugfixes Consolidated bugfixes for version 2.0 May 17, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR bundles several small fixes/improvements across Tomb Editor, WadTool, and the TEN node catalog, primarily targeting UI stability and metadata correctness.

Changes:

  • Improve WadTool skeleton editor display by including mesh names in the bone tree.
  • Harden UI components and event teardown paths (null-safe texture path check, safer node dispose).
  • Fix/adjust editor behaviors and node-catalog metadata (Content Browser add action routing, Level Settings checkbox behavior, inventory node argument filtering metadata, changelog updates).

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
WadTool/Forms/FormSkeletonEditor.cs Adds helper for displaying bone + mesh names in the skeleton tree and uses it in node labels / rename path.
WadTool/Controls/PanelTextureMap.cs Prevents a null dereference when checking for missing external textures.
TombLib/TombLib/Catalogs/TEN Node Catalogs/Inventory.lua Updates inventory node argument enumeration filters so additional items appear in pickers.
TombLib/TombLib.Forms/Controls/VisualScripting/VisibleNodeBase.cs Moves UI disposal into the managed-dispose branch to avoid unsafe cleanup patterns.
TombEditor/ToolWindows/ContentBrowser.cs Routes “add imported geometry” through the command system instead of directly setting an editor action.
TombEditor/Forms/FormLevelSettings.cs Stops forcibly overriding the “Map animated textures” checkbox state during dialog updates.
Installer/Changes.txt Updates release notes to reflect the bundled fixes and additions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread WadTool/Forms/FormSkeletonEditor.cs Outdated
Comment thread WadTool/Forms/FormSkeletonEditor.cs
Comment thread TombLib/TombLib/Catalogs/TEN Node Catalogs/Inventory.lua Outdated
Comment thread Installer/Changes.txt Outdated
@Lwmte Lwmte added this to the Version 2.0 milestone May 22, 2026
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.

3 participants