search

microsoft / AzureStorageExplorer

Easily manage the contents of your storage account with Azure Storage Explorer. Upload, download, and manage blobs, files, queues, tables, and Cosmos DB entities. Gain easy access to manage your virtual machine disks. Work with either Azure Resource Manager or classic storage accounts, plus manage and configure cross-origin resource sharing (CORS) rules.
Creative Commons Attribution 4.0 International
378 stars 87 forks source link

The 'EXPLORER' panel is opened if close one split tab #7925

Open v-xianya opened 6 months ago

v-xianya commented 6 months ago

Storage Explorer Version: 1.34.0-dev (98) Build Number: 20240508.1 Branch: rel/1.34.0 Platform/OS: Windows 10/Windows 11/Linux Ubuntu 22.04/MacOS Sonoma 14.4.1(Apple M1 Pro) Architecture: x64/arm64/x64/arm64 How Found: Exploratory testing Regression From: Previous release (1.33.1)

Steps to Reproduce

  1. Expand one storage account -> Blob Containers.
  2. Open one blob container -> Split the blob container tab.
  3. Toggle sidebar by pressing Ctrl+B or clicking the first icon on the vertical toolbar.
  4. Close one of the blob container tabs.
  5. Check whether the EXPLORER panel is not opened.

Expected Experience

The EXPLORER panel is not opened.

Actual Experience

The EXPLORER panel is opened.

Additional Context

If switch to 'ACCOUNT MANAGEMENT' panel -> then close one split tab -> It will navigate to 'EXPLORER' panel.

craxal commented 6 months ago

This only happens if the last tab of a split panel is closed. The shell does not take into account that other tabs in other splits might be available to focus before focusing on the explorer panel.

Not the easiest fix, because the tab closing code is spread across several classes, and the code that adjusts focus isn't in a convenient spot. It's not a breaking change, and a subtle behavior, so I think this can wait till 1.35.0.

Bug was introduced here when fixing #7702: https://devdiv.visualstudio.com/DefaultCollection/DevDiv/_git/StorageExplorer/pullrequest/531330

MRayermannMSFT commented 6 months ago

Revsisit if a user complains.