AzureStorageExplorer icon indicating copy to clipboard operation
AzureStorageExplorer copied to clipboard

The layout of one table editor is unfriendly if clicking 'Refresh' then clicking 'Query' with no empty space for its editor

Open v-xianya opened this issue 3 years ago • 4 comments

Storage Explorer Version: 1.25.0-dev Build Number: 20220608.1 Branch: main Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Monterey 12.4 (Apple M1 Pro) Architecture ia32\x64 How Found: From running test cases Regression From: Previous release (1.24.2)

Steps to Reproduce

  1. Expand one storage account -> Tables.
  2. Create a table -> Import the test.csv file (contains 25 entities) to the table.
  3. Click 'Refresh' -> Click 'Query'.
  4. Check whether the layout is friendly.

Expected Experience

The layout is friendly.

Actual Experience

The layout is unfriendly. image

Additional Context

  1. This issue reproduces if there is no empty space for the table editor. image
  2. Here is the record: 06081

v-xianya avatar Jun 08 '22 06:06 v-xianya

Verified this issue on the build main/20220622.1. Fixed.

Update: Follow the steps below. the issue reproduces.

  1. Expand one storage account -> Tables.
  2. Create a table -> Import the test.csv file (contains 25 entities) to the table.
  3. Click 'Refresh' -> Click 'Query' -> Click 'Execute query'.
  4. Click 'Refresh' -> Click 'Query'.

5806

v-xianya avatar Jun 22 '22 02:06 v-xianya

This is a known issue related to the height calculations made for the table when a layout occurs. We're not sure what causes this yet. We added a small delay to the redraw to mitigate most occurrences of this issue. Does resizing the panel or window resolve the issue?

craxal avatar Sep 12 '22 18:09 craxal

@craxal Verified this issue on the build 20220913.2. This issue can be resolved by resizing the panel.

v-xianya avatar Sep 13 '22 02:09 v-xianya

We're currently working on a re-haul of the table explorer using React. Considering the complexity of the explorer as of now, and that the issue has an easy workaround, we may wait for the re-hauled explorer to be finished in order to resolve this issue.

craxal avatar Sep 13 '22 17:09 craxal

React version of the explorer will take some time. We are currently investigating a quicker fix for the current Knockout/jQuery version that we could potentially provide for 1.27.0.

craxal avatar Oct 18 '22 17:10 craxal

workaround: resize('click and shake back and forth) the left hand explorer pane, until the right side top menu reappears. ugly but works.

OzBob avatar Oct 19 '22 08:10 OzBob

I am going to make a stylesheet update to mitigate this issue.

JasonYeMSFT avatar Oct 20 '22 20:10 JasonYeMSFT

Verified this issue on the main build 20221025.2. Fixed. So, we are going to close it.

v-xianya avatar Oct 25 '22 02:10 v-xianya

@v-xianya i'm not sure this is fixed, i've just upgraded(v1.27.0 build 20221201.1) and am getting total banner dissapears in Queues image

OzBob avatar Dec 09 '22 01:12 OzBob

Hi @OzBob , yes, this issue is fixed, and it is about the table explorer.

I am not able to reproduce the queue issue you met on the (v1.27.0 build 20221201.1). Are there any extra steps for this issue to reproduce? thanks

v-xianya avatar Dec 09 '22 02:12 v-xianya

@OzBob The original issue was for Table explorer. We haven't done anything in Queue explorer for this type of problem. If you have additional information on how to repro it, please open a new issue and we can investigate.

JasonYeMSFT avatar Dec 09 '22 17:12 JasonYeMSFT