Skip to content

Fix weight updates from grid view#261

Open
mikec316 wants to merge 5 commits intoPCGEx:mainfrom
mikec316:MikeFilters
Open

Fix weight updates from grid view#261
mikec316 wants to merge 5 commits intoPCGEx:mainfrom
mikec316:MikeFilters

Conversation

@mikec316
Copy link
Contributor

@mikec316 mikec316 commented Mar 8, 2026

Problem: When editing the weight via the tile's spinbox (either dragging the slider or typing a value and pressing Enter/clicking away), the bIsBatchOperation flag suppresses OnObjectModified, so the detail panel's FStructOnScope copy is never refreshed. The details only update when navigating to a different record (which calls UpdateDetailForSelection()).

Fix: Added an OnTilePropertyChanged delegate to SPCGExCollectionGridTile that fires after tile-level property edits complete (weight slider end, weight text commit, and subcollection checkbox toggle). The grid view binds this delegate to call UpdateDetailForSelection(), which re-copies the entry data into the detail panel's struct, keeping it in sync immediately.

mikec316 and others added 5 commits March 8, 2026 20:13
Fix: Added an OnTilePropertyChanged delegate to SPCGExCollectionGridTile that fires after tile-level property edits complete (weight slider end, weight text commit, and subcollection checkbox toggle). The grid view binds this delegate to call UpdateDetailForSelection(), which re-copies the entry data into the detail panel's struct, keeping it in sync immediately.
… seed

- Material variant selection in PCGExAssetStaging and PCGExStagingSplineMesh always produced the same variant for a given entry because the material pick used the same Seed as the entry pick. Both create a FRandomStream(Seed) independently, and since the stream is deterministic, the first FRand() from each is identical — making variant selection perfectly correlated with entry selection.

- Fixed by deriving a decorrelated seed for material variant picking: PCGHelpers::ComputeSeed(Seed, Staging.InternalIndex) mixes in the entry's internal index, breaking the correlation while remaining deterministic.
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