Skip to content

hide and rename _d files with same logic as _c files#10142

Closed
boxrocket6803 wants to merge 1 commit intoFacepunch:masterfrom
boxrocket6803:away-with-blob
Closed

hide and rename _d files with same logic as _c files#10142
boxrocket6803 wants to merge 1 commit intoFacepunch:masterfrom
boxrocket6803:away-with-blob

Conversation

@boxrocket6803
Copy link
Copy Markdown
Contributor

Summary

Hides blob data files if a source file is present, if the source file is renamed it will rename the blob data in addition to the compiled file.

Motivation & Context

Seemed messy to have them all out in the open like that.

image

Checklist

  • Code follows existing style and conventions
  • No unnecessary formatting or unrelated changes
  • Public APIs are documented (if applicable)
  • Unit tests added where applicable and all passing
  • I’m okay with this PR being rejected or requested to change 🙂

Copilot AI review requested due to automatic review settings March 7, 2026 19:58
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

Updates the editor asset browser/list behavior to treat “blob” data files (*_d) similarly to compiled files (*_c): they’re hidden when a corresponding source asset is present, and they’re renamed alongside compiled outputs when renaming the source.

Changes:

  • Rename *_d blob file alongside the compiled *_c file when renaming an asset.
  • Hide *_d blob files in the Asset Browser when a source asset exists (mirrors existing _c filtering behavior).
  • Prevent blobs from being treated as loadable assets/inspectable items in the Asset Browser UI.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
game/addons/tools/Code/Editor/AssetList/Entries/AssetEntry.cs Renames associated *_d blob file when renaming an asset (in addition to the compiled output).
game/addons/tools/Code/Editor/AssetBrowser/AssetBrowser.cs Filters blob files out of the Asset Browser list when a source asset exists, using the same approach as compiled-file filtering.

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

@handsomematt handsomematt added the triaged triaged pull-requests are replicated on the internal sbox repo label Mar 25, 2026
@sboxbot sboxbot added the accepted this pull request was accepted, hurrah! label Apr 7, 2026
@sboxbot
Copy link
Copy Markdown
Contributor

sboxbot commented Apr 7, 2026

This PR has been merged upstream.

@sboxbot sboxbot closed this Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accepted this pull request was accepted, hurrah! triaged triaged pull-requests are replicated on the internal sbox repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants