pikers
/
piker
4
1
Fork 0
Code Issues 16 Pull Requests 15 Packages Projects Releases Wiki Activity

fix DpiAwareFont default size calculation #48

Open
momo wants to merge 105 commits from dpi-font-auto-calc into how_to_show_ur_pp
pull from: dpi-font-auto-calc
merge into: pikers:how_to_show_ur_pp
Conversation 8 Commits 105 Files Changed 80
momo commented 2025-12-03 21:56:56 +00:00
Collaborator

DPI Calculation Fix

Previously, configure_to_dpi() used logical DPI or _font_sizes presets, which produced small fonts (~8px) on high-DPI displays.

Changes:

  • Changed font size calculation to use physicalDotsPerInch(): font_size = round(inches * pdpi)
  • Ensures consistent, readable default font size based on actual screen DPI

UI Zoom Feature

From @dnks, refined by @goodboy. Added global UI font-size zoom functionality for dynamic adjustment of all UI text sizes.

Keyboard shortcuts:

  • Ctrl+Shift+Plus - Zoom in
  • Ctrl+Shift+Minus - Zoom out
  • Ctrl+Shift+0 - Reset to default

Implementation:

  • GlobalZoomEventFilter at QApplication level intercepts zoom shortcuts
  • DpiAwareFont extended with zoom_level multiplier
  • All UI widgets updated with update_fonts() methods
  • Chart axes properly recalculate on zoom changes

UX Improvements

Better defaults:

  • Start at 100% zoom (was 400%)
  • Smoother 20% steps (was 100% jumps)
  • Max zoom 300% (was 1000%, which caused UI cropping)

Keyboard compatibility:

  • Support for international keyboard layouts (Shift+Minus → _, Shift+0 → ))

Layout fixes:

  • Search widget and completer view resize properly during zoom
## DPI Calculation Fix Previously, `configure_to_dpi()` used logical DPI or `_font_sizes` presets, which produced small fonts (~8px) on high-DPI displays. **Changes:** - Changed font size calculation to use `physicalDotsPerInch()`: `font_size = round(inches * pdpi)` - Ensures consistent, readable default font size based on actual screen DPI ## UI Zoom Feature From @dnks, refined by @goodboy. Added global UI font-size zoom functionality for dynamic adjustment of all UI text sizes. **Keyboard shortcuts:** - `Ctrl+Shift+Plus` - Zoom in - `Ctrl+Shift+Minus` - Zoom out - `Ctrl+Shift+0` - Reset to default **Implementation:** - `GlobalZoomEventFilter` at `QApplication` level intercepts zoom shortcuts - `DpiAwareFont` extended with `zoom_level` multiplier - All UI widgets updated with `update_fonts()` methods - Chart axes properly recalculate on zoom changes ## UX Improvements **Better defaults:** - Start at 100% zoom (was 400%) - Smoother 20% steps (was 100% jumps) - Max zoom 300% (was 1000%, which caused UI cropping) **Keyboard compatibility:** - Support for international keyboard layouts (`Shift+Minus` → `_`, `Shift+0` → `)`) **Layout fixes:** - Search widget and completer view resize properly during zoom
momo added 1 commit 2025-12-03 21:56:57 +00:00
momo requested review from goodboy 2025-12-03 21:57:08 +00:00
goodboy reviewed 2025-12-03 22:10:29 +00:00
pyproject.toml Outdated
@ -77,6 +77,9 @@ dependencies = [
"trio-typing>=0.10.0",
"numba>=0.61.0",
"pyvnc",
"pyqt6>=6.8.0",
goodboy commented 2025-12-03 22:10:29 +00:00
Owner

@momo let’s not add these deps to the core set and instead add them to the dev set since we only need them for running this new test suite 🙏

i also wonder if we might want to even consider making a new testing = [] dependency group eventually?

I think for now tossing it into dev is sufficient. Also not sure if uv has a way for deps-groups to depend on others?

Maybe do a quick check for that in their docs too?

@momo let's not add these deps to the core set and instead add them to the `dev` set since we only need them for running this new test suite 🙏 i also wonder if we might want to even consider making a new `testing = []` dependency group eventually? I think for now tossing it into `dev` is sufficient. Also not sure if `uv` has a way for deps-groups to depend on others? Maybe do a quick check for that in their docs too?
goodboy commented 2025-12-03 22:12:02 +00:00
Owner

Also not sure if uv has a way for deps-groups to depend on others?

looks like nope says gemini Xp

uv does not currently support nesting dependency groups directly within the pyproject.toml file in a way that one group explicitly “includes” another in the same manner as, for example, extras can be defined to include other extras. However, you can achieve a similar effect by managing your uv commands to include multiple groups when performing operations like sync or pip install.

> Also not sure if uv has a way for deps-groups to depend on others? looks like nope says gemini Xp > uv does not currently support nesting dependency groups directly within the pyproject.toml file in a way that one group explicitly "includes" another in the same manner as, for example, extras can be defined to include other extras. However, you can achieve a similar effect by managing your uv commands to include multiple groups when performing operations like sync or pip install.
goodboy reviewed 2025-12-03 22:14:29 +00:00
tests/test_dpi_font.py Outdated
@ -0,0 +15,4 @@
# pass the mock screen to configure_to_dpi
font.configure_to_dpi(screen=MockScreen())
print("Computed pixel size:", font.px_size)
goodboy commented 2025-12-03 22:14:29 +00:00
Owner

woo nice!

this is a good start but we should wrap this in a pytest test-fn (function) so that it will report like the rest of the test suites ;)

https://docs.pytest.org/en/stable/getting-started.html#create-your-first-test

woo nice! this is a good start but we should wrap this in a `pytest` test-fn (function) so that it will report like the rest of the test suites ;) https://docs.pytest.org/en/stable/getting-started.html#create-your-first-test
goodboy reviewed 2025-12-03 22:19:22 +00:00
piker/ui/_style.py Outdated
@ -183,3 +183,3 @@
# float to px size: int.
self._font_inches = inches
font_size = math.floor(inches * dpi)
font_size = math.floor(inches * pdpi)
goodboy commented 2025-12-03 22:19:22 +00:00
Owner

Yeah, it’s funny that this worked for you since the non-physical DPI is supposed to be the one that “you perceive on screen given DE scaling compensation” ..

so it does indeed look like we might always want/need to do from-scratch analysis of every display’s reported DPI and totally ignore OS-dependent metrix/reports including those with supposed auto-scaling.

Yeah, it's funny that this worked for you since the non-physical DPI is supposed to be the one that "you perceive on screen given DE scaling compensation" .. so it does indeed look like we might **always** want/need to do from-scratch analysis of every display's reported DPI and totally ignore OS-dependent metrix/reports including those with supposed auto-scaling.
momo added 1 commit 2025-12-08 20:30:01 +00:00
momo added 1 commit 2025-12-09 03:17:05 +00:00
momo added 1 commit 2025-12-09 03:22:37 +00:00
momo added 5 commits 2025-12-22 21:47:59 +00:00
35744f666f Add some Qt DPI extras to `qt_screen_info.py` 
- set `QT_USE_PHYSICAL_DPI='1'` env var for Qt6 high-DPI
  * we likely want to do this in `piker.ui` as well!
- move `pxr` calc from widget to per-screen in loop.
- add `unscaled_size` calc using `pxr * size`.
- switch from `.availableGeometry()` to `.geometry()` for full
  bounds.
- shorten output labels, add `!r` repr formatting
- add Qt6 DPI rounding policy TODO with doc links

(this patch was generated in some part by [`claude-code`][claude-code-gh])
[claude-code-gh]: https://github.com/anthropics/claude-code
49bedb4912 Reorder imports in `qt_screen_info.py` ?? 
For wtv reason on nixos importing `pyqtgraph` first is causing `numpy`
to fail import?? No idea, but likely something to do with recent
`flake.nix`'s ld-lib-linking with `<nixpkgs>` marlarky?
e63cffaf53 Add `.xsh` script mentioned in gitea #50 
Note since it's actually `xonsh` code run with either,
- most pedantically: `xonsh ./snippets/calc_ppi.xsh`
- or relying on how shebang: `./snippets/calc_ppi.xsh`
  * an sheboom.
goodboy commented 2026-01-12 01:04:39 +00:00
Owner

Hey @momo sorry about the delay getting you a rebase cmd!

Looking at the old history the rebase cmd you need should be something like,

git rebase 3751140f dpi-font-auto-calc --onto=main

where commit 3751140f should have a message like,

ib: bump docker/ib/README.rst

i believe, and it should still be the first commit from before you created your new dev branch dpi-font-auto-calc.

During this rebase you will hit some conflicts on pyproject.toml and uv.lock because it looks like you changed those but they’ve already been better re-orged and set upstream on main now (also i know about the conflicts bc i already tested the git rebase cmd i provide above ;).

The commits that touch these i think are,

so when you hit conflicts what i’d recommend is doing,

git status/diff  # just if you want to see the conflicted files
git reset uv.lock pyproject.toml
git checkout --ours uv.lock pyproject.toml
git add uv.lock pyproject.toml
git rebase --continue

i believe the 2nd commit actually changes uv.lock without conflict, but i’d still like to have the main version of it be what you base your work on before changing it; pretty sure you won’t need to change it either bc now we use [dependency-groups] more correctly and with the --dev group actually nesting them nicely.

if you run into any issues you can always do,

git rebase --abort

to start over from the beginning.

once you’ve successfully rebased the way i would want it the history should look like the image i’ve uploaded,

Hint you can see something similar on console by using the cmd,

git log --graph --oneline --decorate --all ( i alias this in my global .gitconfig as logtree)

as always if you have an questions or what to go into more detail about all this just hit me up on chat ;)

Hey @momo sorry about the delay getting you a rebase cmd! Looking at the old history the rebase cmd you need should be something like, `git rebase 3751140f dpi-font-auto-calc --onto=main` where commit 3751140f should have a message like, > ib: bump `docker/ib/README.rst` i believe, and it should still be the first commit from *before* you created your new dev branch `dpi-font-auto-calc`. During this rebase you will hit some conflicts on `pyproject.toml` and `uv.lock` because it looks like you changed those but they've already been better re-orged and set upstream on `main` now (also i know about the conflicts bc i already tested the `git rebase` cmd i provide above ;). The commits that touch these i think are, - c6c768e - 7ff4f5d so when you hit conflicts what i'd recommend is doing, ```sh git status/diff # just if you want to see the conflicted files git reset uv.lock pyproject.toml git checkout --ours uv.lock pyproject.toml git add uv.lock pyproject.toml git rebase --continue ``` i believe the 2nd commit actually changes `uv.lock` *without* conflict, but i'd still like to have the `main` version of it be what you base your work on before changing it; pretty sure you won't need to change it either bc now we use `[dependency-groups]` more correctly and with the `--dev` group actually nesting them nicely. if you run into any issues you can always do, `git rebase --abort` to start over from the beginning. --- once you've successfully rebased the way i would want it the history should look like the image i've uploaded, Hint you can see something similar on console by using the cmd, `git log --graph --oneline --decorate --all` ( i alias this in my global `.gitconfig` as `logtree`) --- as always if you have an questions or what to go into more detail about all this just hit me up on chat ;)
2026-01-11T19:50:33,748162158-05:00.png
279 KiB
2026-01-11T19:50:33,748162158-05:00.png
momo force-pushed dpi-font-auto-calc from 6ee0d86548 to e1a81fa433 2026-01-13 03:58:42 +00:00 Compare
momo commented 2026-01-13 04:01:18 +00:00
Poster
Collaborator

@goodboy how it looks? there is no conflicts now

@goodboy how it looks? there is no conflicts now
goodboy commented 2026-03-03 19:25:16 +00:00
Owner

@momo i’d say this is fine to land but it’d be nice to get some feedback on another OS if possible.

@dnks maybe once we land #78 (and any friends in tractor) we can try this patch on top and see if defaults are any better?

I’d also like some (AI genned?) docs around all this malarky including how to run the,

python snippets/qt_screen_info.py

and grok output; maybe we should also just bundle this into a piker utils/ui screen_info or something CLI ep?

@momo i'd say this is fine to land but it'd *be nice* to get some feedback on another OS if possible. @dnks maybe once we land https://pikers.dev/pikers/piker/pulls/78 (and any friends in `tractor`) we can try this patch on top and see if defaults are any better? I'd also like some (AI genned?) docs around all this malarky including how to run the, `python snippets/qt_screen_info.py` and grok output; maybe we should also just bundle this into a `piker utils/ui screen_info` or something CLI ep?
goodboy commented 2026-03-03 19:31:10 +00:00
Owner

Ah right another thing is we should pull the “manually adjustable font size” approach stuff that @dnks drafted as part of #64 / #65?

Not sure if the first one helps all that much, likely some weird vibed thing by claude as an initial attempt @dnks ?

Ah right another thing is we should pull the "manually adjustable font size" approach stuff that @dnks drafted as part of #64 / #65? - window geo "session saving"? * not sure if we actually need/will-find-this-useful? @dnks needs to clarify what it's for. * https://pikers.dev/pikers/piker/pulls/64/files#diff-6f0c22d01e2f9be0c2ed10e50eba29c13f105bfd - manual font-size-driving-widget sizing from #65, * https://pikers.dev/pikers/piker/pulls/65/files#issuecomment-1197 * to clarify, we **only need** the changes to files under `piker.ui` **EXCEPT FOR** the changes to `piker.ui._display` which is already integrated (and only for) the macos support coming in https://pikers.dev/pikers/piker/pulls/78. Not sure if the first one helps all that much, likely some weird vibed thing by `claude` as an initial attempt @dnks ?
momo added 3 commits 2026-03-05 17:56:45 +00:00
78531cd573 🔴 No files deleted. 
🟢 a/piker/ui/_exec.py for adding global keyboard shortcuts for UI zoom.

🛠️ a/piker/ui/_forms.py -> Added methods to update fonts and sizes dynamically.

🛠️ a/piker/ui/_label.py -> Added method to update font and color after zoom.

🛠️ a/piker/ui/_position.py -> Included update_fonts method for labels and bars.

🛠️ a/piker/ui/_search.py -> Added update_fonts method for search view components.

🛠️ a/piker/ui/_style.py -> Enhanced configure_to_dpi() to support zoom level multipliers.

🛠️ a/piker/ui/_window.py -> Implemented application-wide zoom with event filter and zoom methods, including signals to update UI elements' font sizes and styles.

🔴 No files deleted.
62982a41fc 🟢 .gitignore 
🛠️ piker/ui/_axes.py -> Enhance axis font and size handling
🛠️ piker/ui/_window.py -> Improve zoom key detection and event handling
🛠️ piker/ui/_window.py -> Update axes fonts and layout after zoom events
momo commented 2026-03-05 18:03:19 +00:00
Poster
Collaborator

@goodboy

  • Fixed startup zoom level: Changed from 4.0 (400%, too large) to 1.0 (100%, normal size)
  • Made zoom increments smoother: Changed from 100% jumps to 20% increments
  • Reduced max zoom: Changed from 10.0 to 3.0 to prevent severe UI cropping issues
  • Added keyboard layout compatibility:
    • Zoom out now works with both - and _ (Ctrl+Shift+Minus on some keyboards)
    • Reset now works with both 0 and ) (Ctrl+Shift+0 on some keyboards)
  • Fixed search box not resizing during zoom by adding updateGeometry() calls

Also included: Fixed Binance AggTrade validation (made nq field optional)

Known Issues

  • Chart splitter proportions don’t perfectly reset after zoom changes
  • Multi-display setups may calculate different default font sizes based on each screen’s DPI (expected behavior, not a bug in these changes)
  • Some extreme zoom levels may still cause minor layout quirks

Feedback Needed

  • Does zoom feel smooth and responsive on your setup?
  • Do the keyboard shortcuts work on your keyboard layout?
  • Any UI elements that don’t resize properly?
  • Issues with multi-display setups?
@goodboy - Fixed startup zoom level: Changed from 4.0 (400%, too large) to 1.0 (100%, normal size) - Made zoom increments smoother: Changed from 100% jumps to 20% increments - Reduced max zoom: Changed from 10.0 to 3.0 to prevent severe UI cropping issues - Added keyboard layout compatibility: - Zoom out now works with both - and _ (Ctrl+Shift+Minus on some keyboards) - Reset now works with both 0 and ) (Ctrl+Shift+0 on some keyboards) - Fixed search box not resizing during zoom by adding updateGeometry() calls Also included: Fixed Binance AggTrade validation (made nq field optional) Known Issues - Chart splitter proportions don't perfectly reset after zoom changes - Multi-display setups may calculate different default font sizes based on each screen's DPI (expected behavior, not a bug in these changes) - Some extreme zoom levels may still cause minor layout quirks Feedback Needed - Does zoom feel smooth and responsive on your setup? - Do the keyboard shortcuts work on your keyboard layout? - Any UI elements that don't resize properly? - Issues with multi-display setups?
goodboy commented 2026-03-05 19:14:33 +00:00
Owner

@momo sweet!

going to try and test shortly 👍

I left some notes (in chat) about how we should prolly swap the 2 commits you have from @dnks to the new re-written versions i’ve got in the history of #78.

It’d be ideal to use the newly piker.ui-only isolated commit-message-rewritten versions,

so these commits from this reworked #78 history,

(which i generated using our new .claude/skills/commit-msg/ setup btw from #69 😎)

and use those instead of the origs (also attached snap of what they look like on the commits tab of this PR):

@momo sweet! going to try and test shortly 👍 --- I left some notes (in chat) about how we should prolly swap the 2 commits you have from @dnks to the new re-written versions i've got in the history of https://pikers.dev/pikers/piker/pulls/78. It'd be ideal to use the newly `piker.ui`-only isolated commit-message-rewritten versions, so these commits from this reworked #78 history, - https://pikers.dev/pikers/piker/commit/98ab543f95abfe90d904f07b4e198b5efb1d359e - https://pikers.dev/pikers/piker/commit/4cfc2b695f09e220b3cd3ddf938eb4dacd0a6548 (which i generated using our new `.claude/skills/commit-msg/` setup btw from #69 😎) and use those instead of the origs (also attached snap of what they look like on the commits tab of this PR): - 62982a41fcacf3bad54b4ff247f64ba428dbafe4 - 78531cd57355c7b92a1908fb5dc9b21f70786bc8
2026-03-05T14:12:08,533800177-05:00.png
36 KiB
2026-03-05T14:12:08,533800177-05:00.png
momo force-pushed dpi-font-auto-calc from 1376b4667e to cf97a70b2c 2026-03-06 06:04:19 +00:00 Compare
goodboy commented 2026-03-06 21:51:03 +00:00
Owner

I just changed the base branch (what we’ll merge this into) to main as well since the original is already merged.

@momo so actually try to do a git rebase main and see how it goes. i think it should be a clean operation.

I just changed the base branch (what we'll merge this into) to `main` as well since the original is already merged. @momo so actually try to do a `git rebase main` and see how it goes. i think it should be a clean operation.
goodboy requested review from dnks 2026-03-06 21:52:22 +00:00
goodboy requested review from guille 2026-03-06 21:52:29 +00:00
momo force-pushed dpi-font-auto-calc from cf97a70b2c to 416a121de1 2026-03-09 00:49:22 +00:00 Compare
momo commented 2026-03-09 00:54:14 +00:00
Poster
Collaborator

I just changed the base branch (what we’ll merge this into) to main as well since the original is already merged.

@momo so actually try to do a git rebase main and see how it goes. i think it should be a clean operation.

@goodboy there was a conflict with binance feed file, i removed it, you already fixed the issue in main.

> I just changed the base branch (what we'll merge this into) to `main` as well since the original is already merged. > > @momo so actually try to do a `git rebase main` and see how it goes. i think it should be a clean operation. @goodboy there was a conflict with binance feed file, i removed it, you already fixed the issue in main.
This pull request can be merged automatically.
You are not authorized to merge this pull request.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b dpi-font-auto-calc how_to_show_ur_pp
git pull origin dpi-font-auto-calc

Step 2:

Merge the changes and update on Gitea.
git checkout how_to_show_ur_pp
git merge --no-ff dpi-font-auto-calc
git push origin how_to_show_ur_pp
Sign in to join this conversation.
Reviewers
No reviewers
goodboy
dnks
guille
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: pikers/piker#48
There is no content yet.
Delete Branch "dpi-font-auto-calc"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?