positron icon indicating copy to clipboard operation
positron copied to clipboard

Data Explorer: Summary panel reopens after being closed

Open kv9898 opened this issue 1 year ago • 1 comments

System details:

Windows 11

Positron and OS details:

Positron Version: 2024.10.0 (system setup) build 14 Code - OSS Version: 1.93.0 Commit: 901ab5d11d694212d32e53b97f771c5d601e428e Date: 2024-10-09T15:24:43.348Z Electron: 30.4.0 Chromium: 124.0.6367.243 Node.js: 20.15.1 V8: 12.4.254.20-electron.0 OS: Windows_NT x64 10.0.22631

Interpreter details:

R 4.4.1

Describe the issue:

After closing the summary panel for a dataset, it automatically reopens after switching back from another tab.

Steps to reproduce the issue:

  1. Run in R mtcars <- mtcars
  2. Open mtcars in data explorer
  3. Close the summary panel
  4. Turn to another tab (e.g. R script)
  5. Switch back to the data explorer of mtcars
  6. The summary panel is open again

Expected or desired behavior:

The summary panel stays closed after it's closed by the user.

Preferably, the user has an option to keep it closed by default and only open it on request.

Were there any error messages in the UI, Output panel, or Developer Tools console?

None

kv9898 avatar Oct 09 '24 22:10 kv9898

@kv9898 , thank you for your report! It does the same thing for me! I'll move it along into our triage process!

jonvanausdeln avatar Oct 10 '24 15:10 jonvanausdeln

I noted that the summary panel layout (left or right) is 'remembered' correctly, but I failed to find out the relevant code. It seems that summary panel-related code is scattered in multiple files. If someone can tell me the location of the code responsible storing the layout information I can try to write it for collapsing.

kv9898 avatar Jan 06 '25 02:01 kv9898

Verified Fixed

Positron Version(s) : Positron Dev Version: 2025.03.0 build 0, commit c854756bf OS Version(s) : MacOS

Test scenario(s)

Confirmed that expand/collapse summary persists when switching tabs.

Link(s) to test cases run or created:

n/a

midleman avatar Feb 11 '25 17:02 midleman