fix: prevent 'Go one folder up' from changing editor selection

[willeastcott]

Summary

• Fix the "Go one folder up" button in the Assets panel so it only moves the active/current folder highlight to the parent folder without changing the editor's asset selection.

Details

navigateBack() calls _focusAssetItem() after filtering completes to scroll the previous folder into view. However, _focusAssetItem also sets selected = true on the grid/table item, which triggers _onSelectAssetElement → editor.call('selector:add', ...), modifying the editor's selection as a side effect.

The fix wraps the _focusAssetItem call with _suspendSelectEvents = true so the item is still focused and scrolled into view, but the selector is not modified.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
editor	Building	Preview, Comment	Mar 3, 2026 7:29pm

[Copilot]

Pull request overview

Fixes Assets panel “Go one folder up” navigation so the parent folder is focused/scrolled into view without mutating the editor’s asset selection via selection event side effects.

Changes:

• Wrap _focusAssetItem(prevFolderId) in a temporary _suspendSelectEvents block when filter:end fires during navigateBack().

---

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

[Copilot]

This temporarily forces _suspendSelectEvents to false after focusing, which can break callers that already had selection events suspended when filter:end fires (nested/overlapping suspensions). Capture the previous value and restore it (ideally in a try/finally) so this callback doesn’t inadvertently re-enable select events.

Suggested change

	this._suspendSelectEvents = true;
	this._focusAssetItem(prevFolderId);
	this._suspendSelectEvents = false;
	const prevSuspend = this._suspendSelectEvents;
	this._suspendSelectEvents = true;
	try {
	this._focusAssetItem(prevFolderId);
	} finally {
	this._suspendSelectEvents = prevSuspend;
	}