Skip to content

Conversation

@willeastcott
Copy link
Contributor

@willeastcott willeastcott commented Jan 3, 2026

Fixes #1293

Description

When clicking on a script title in the entity inspector, the editor now navigates to the folder containing that script asset, matching the existing behavior for assets and templates.

select-script.mp4

Changes

  • Modified _onClickTitle in ScriptInspector to navigate to the containing folder after selecting the script asset
  • If the script is in a subfolder, the assets panel switches to that folder
  • If the script is in the root folder, the assets panel switches to the root

Before

Clicking a script title in the inspector would only select the script asset in the assets panel without navigating to its location.

After

Clicking a script title now both selects the asset and opens the folder containing it, consistent with how asset and template references work elsewhere in the editor.

@willeastcott willeastcott self-assigned this Jan 3, 2026
@willeastcott willeastcott added bug Something isn't working area: editor interface labels Jan 3, 2026
@willeastcott willeastcott marked this pull request as ready for review January 3, 2026 18:12
Copy link
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 fixes the behavior when clicking on a script title in the entity inspector. Previously, clicking would only select the script asset without navigating to its location in the assets panel. This change makes script references consistent with how asset and template references behave elsewhere in the editor.

Key Changes:

  • Added folder navigation logic to _onClickTitle method in ScriptInspector
  • Ensures the assets panel switches to the folder containing the script asset when the script title is clicked

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

@willeastcott willeastcott merged commit b5011be into main Jan 5, 2026
3 checks passed
@willeastcott willeastcott deleted the select-script branch January 5, 2026 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: editor interface bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clicking on script inside inspector should open a folder containing this script

3 participants