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.

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

Select all rows in results grid should allow copy #8185

Open
DanielPiersonLN opened this issue Nov 27, 2024 · 1 comment
Open

Select all rows in results grid should allow copy #8185

DanielPiersonLN opened this issue Nov 27, 2024 · 1 comment
Labels

Comments

@DanielPiersonLN
Copy link

Describe the bug

Assuming I have a result set with more than the configured "Data result rows per page", I click the upper left corner of the grid to select all rows. This selects just the rows in the batch, but offers a button labeled "Select All N Rows". If I click that button, a message is shown stating "All N rows are selected". However, the Copy button is disabled, so I can't actually copy the rows.

To Reproduce

Steps to reproduce the behavior:

  1. Rows per page is set to 1000
  2. I run a query that has 24945 results
  3. Click on the empty space above the row numbers to select all rows
  4. Click on the button labeled "Select All 24945 Rows"

Expected behavior
Copy button should be enabled, allowing me to copy these rows.

Screenshots
image

image

Desktop (please complete the following information):

  • OS: Windows-11-10.0.22631-SP0
  • Version: 8.13
  • Mode: Desktop

Additional context

My employer manages our PGAdmin install. Not sure what version I had previously but it didn't have this paging system. In that version, I could just click the space above the row numbers twice and the entire result set would be loaded.

I can work around this new issue by editing the range but I find this less convenient than the way it worked before.

@adityatoshniwal
Copy link
Contributor

Hi @DanielPiersonLN,
Previously, when select all was clicked the UI fetched all the rows from server. This is not the case now. For performance reasons, only page data will be loaded in UI. What can be done is when copy is clicked (with select all) - change the page range to fetch all the rows and then copy the data.
As a workaround, you can either increase the page size from preferences or change the range from data output.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: No status
Development

No branches or pull requests

3 participants