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

Allow inspector window to fit in low resolution displays using high DPI #1033

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

Cuperino
Copy link
Member

@Cuperino Cuperino commented Jan 8, 2025

Allows the following sections to shrink better:

  • Object tab (PropertiesTab)
  • Messages tab (MessageHandlerWidget)
  • Font browser (FontBrowserWidget)
  • Models tab (ModelInspectorWidget)

Fixes value field in PropertiesTab not being accessible at small pane widths.

Improves Models tab by allowing data that was previously drawn on labels to be copied.

Solves #358

@Cuperino Cuperino added the bug label Jan 8, 2025
@Cuperino Cuperino force-pushed the fix-358 branch 2 times, most recently from 92da7db to 00dffae Compare January 11, 2025 05:24
Javier O. Cordero Pérez added 5 commits January 11, 2025 01:27
This prevents dynamically generated items from having 0 width,
allowing users to realize when new options have appeared on screen.
Refactor to free horizontal space and make better use of vertical space.
This helps GammaRay be used on low resolution screen with high DPI.

Helps solve #358
Allows GammaRay to be used on low resolution screen with high DPI.

Helps solve #358
By nesting buttons into their own scroll area we can use
the model view's scroll bars separately, which use stepped increments.

This allows GammaRay to be used on low resolution screen with high DPI.

Helps solve #358
By turning a few labels into lineEdits we can not only make more
more efficient use of space, but we also allow people to copy
information more easily.

Allows GammaRay to be used on low resolution screen with high DPI.

Helps solve #358
@Cuperino
Copy link
Member Author

Cuperino commented Jan 11, 2025

Models

Before, min window size:
Screenshot_20250111_013147
After, min window size:
Screenshot_20250111_012556
After, default window size:
Screenshot_20250111_015615

Messages

Before, min window size:
Screenshot_20250111_013124
After, min window size:
Screenshot_20250111_012642
After, default window size:
Screenshot_20250111_015537

Font Browser

Before, min window size:
Screenshot_20250111_013102
After, min window size:
Screenshot_20250111_014105
After, default window size:
Screenshot_20250111_015438

Properties Tab

Before, min window size:
Screenshot_20250111_013012
After, min window size:
Screenshot_20250111_012544
After, default window size:
Screenshot_20250111_015301

@Cuperino Cuperino self-assigned this Jan 11, 2025
@Cuperino Cuperino requested a review from pasnox January 16, 2025 01:19
@@ -26,7 +26,7 @@
<item>
<widget class="QSplitter" name="mainSplitter">
<property name="orientation">
<enum>Qt::Horizontal</enum>
<enum>Qt::Orientation::Horizontal</enum>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This work on Qt5 / Qt6?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants