Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

The Quick Access node is selected after clicking 'Load more' under storage accounts tree view #6311

Closed
v-xianya opened this issue Nov 29, 2022 · 2 comments
Closed

The Quick Access node is selected after clicking 'Load more' under storage accounts tree view #6311

v-xianya opened this issue Nov 29, 2022 · 2 comments
Assignees
@MRayermannMSFT
Labels
🧪 testing Found through regular testing ✅ merged A fix for this issue has been merged
Milestone
1.29.0

Comments

@v-xianya
Copy link
Member

Storage Explorer Version: 1.27.0
Build Number: 20221128.3
Branch: rel/1.27.0
Platform/OS: Windows 10/Linux Ubuntu 22.04/MacOS Ventura 13.0.1 (Apple M1 Pro)
Architecture: ia32/x64
How Found: Ad-hoc testing
Regression From: Not a regression

Steps to Reproduce

  1. Expand Storage Accounts node -> Make sure there is 'Load more' under storage accounts tree view.
  2. Click 'Load more'.
  3. Check whether the loaded storage account is selected.

Expected Experience

The loaded storage account is selected.

Actual Experience

The Quick Access node is selected.

Addition Context

Here is the record:
load
@v-xianya v-xianya added the 🧪 testing Found through regular testing label Nov 29, 2022
@JasonYeMSFT
Copy link
Contributor

This will occur when loading the last batch of nodes. The current code selects the load more placeholder node which is soon removed. The tree view moves the selection to the first node (Quick Access) on its removal.

@MRayermannMSFT MRayermannMSFT self-assigned this Dec 5, 2022
@MRayermannMSFT MRayermannMSFT added this to the 1.28.0 milestone Dec 5, 2022
@MRayermannMSFT MRayermannMSFT modified the milestones: 1.28.0, 1.29.0 Jan 18, 2023
@MRayermannMSFT MRayermannMSFT added the ✅ merged A fix for this issue has been merged label Mar 4, 2023
@v-xianya
Copy link
Member Author

v-xianya commented Mar 6, 2023

Verified this issue on the build main 20230304.8. Fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MRayermannMSFT MRayermannMSFT

Labels
🧪 testing Found through regular testing ✅ merged A fix for this issue has been merged
Projects
None yet
Milestone
1.29.0
Development

No branches or pull requests
3 participants
@MRayermannMSFT @JasonYeMSFT @v-xianya