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

Can not select columns in Running Queries #1140

Open
aachaemenes opened this issue Nov 20, 2024 · 8 comments
Open

Can not select columns in Running Queries #1140

aachaemenes opened this issue Nov 20, 2024 · 8 comments
Assignees
Labels
DBA Dash GUI system client (web, mobile, CLI, 3rd party) defect Something is off and needs to be addressed

Comments

@aachaemenes
Copy link

There is an issue with choosing columns to display in Running queries tab. You can not see witch items(columns) are selected

running queries

DavidWiseman added a commit to DavidWiseman/dba-dash that referenced this issue Nov 20, 2024
If the column height is too short, checkboxes won't be visible.  A height of 24 causes problems in some systems.
trimble-oss#1140
@DavidWiseman
Copy link
Collaborator

This is related to #996, #938. The checkbox won't be visible if the row height is too small. Increasing the height manually is a workaround.

It's odd as this issue impacts some users and not others. The default row height should be 29, but it's set to 24 in many places. This still looks OK on my computer and the checkboxes display normally. On your computer - I can see the height is too small - even the running queries grid in the background. Maybe some graphics or font setting makes a difference, but either way, it's worth changing the height back to the default of 29. I've done a bulk fix for this.

I'm not sure how they got set to 24 as it's unlikely I did this intentionally - probably a designer issue at some point.

@DavidWiseman DavidWiseman self-assigned this Nov 20, 2024
@DavidWiseman DavidWiseman added DBA Dash GUI system client (web, mobile, CLI, 3rd party) defect Something is off and needs to be addressed labels Nov 20, 2024
DavidWiseman added a commit that referenced this issue Nov 20, 2024
If the column height is too short, checkboxes won't be visible.  A height of 24 causes problems in some systems.
#1140
@DavidWiseman
Copy link
Collaborator

DavidWiseman commented Nov 20, 2024

Fixed in 3.15 🎉

@FuriousDBA
Copy link

FuriousDBA commented Nov 23, 2024

Hi @DavidWiseman, the issue is still there even after update to 3.15. The workaround you mention (change row height size) works.
If I work on big external screen all is fine. If I work on 15" laptop screen, then I face mentioned issue. Additionally summary page looks weird then. Columns on bottom grid are too narrow.
image

Same case for a few other tabs:
image

If it matters, I use DBA Dash on RDP session.

@DavidWiseman DavidWiseman reopened this Nov 25, 2024
@DavidWiseman
Copy link
Collaborator

I'm not able to reproduce this issue. It works OK for me on a 15" laptop screen via RDP. The checkboxes are still visible and the summary columns are sized appropriately with horizontal scrollbars on the smaller screen when needed.
What is the resolution of the laptop screen and scaling used? I've tried messing about with resolution and scaling but I still can't reproduce the issue.
What RDP client is used? What version of Windows?

@FuriousDBA
Copy link

Hi, thanks for response.
I'm on Windows 11, using Windows Remote Desktop Connection
image
image

The remote server is Windows Server 2022

@DavidWiseman
Copy link
Collaborator

Thanks - I can replicate this. The scale at 250% is causing the issue and it doesn't need to be a RDP session. Setting the resolution lower and reducing the scale would fix the issue but won't make best use of the high PPI display.
I'll investigate how I can fix the issue - at least I can reproduce it now.

@DavidWiseman
Copy link
Collaborator

This is a better workaround:

  • Right-click DBADash.exe
  • Click change high DPI settings on Compatibility tab
  • Check the Override high DPI scaling behaviour checkbox.
  • Select System (Enhanced)

image

It's not a perfect solution. Some text might not be as crisp as it should be - which probably won't be that noticeable on a small screen.

@FuriousDBA
Copy link

Workaround works, thanks a lot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DBA Dash GUI system client (web, mobile, CLI, 3rd party) defect Something is off and needs to be addressed
Projects
None yet
Development

No branches or pull requests

3 participants