Skip to content

Fix packaged asset resolution for issue #78#99

Merged
thisislvca merged 4 commits intomasterfrom
fix-issue-78
Apr 21, 2026
Merged

Fix packaged asset resolution for issue #78#99
thisislvca merged 4 commits intomasterfrom
fix-issue-78

Conversation

@thisislvca
Copy link
Copy Markdown
Collaborator

@thisislvca thisislvca commented Apr 14, 2026

Summary

  • restore the packaged asset root fallback for Windows/Linux frozen builds so resources resolve from _internal/
  • build device image URLs with QUrl.fromLocalFile(...) instead of concatenating file:/// strings in QML
  • pass the app root into the backend so image asset resolution comes from a single source of truth
  • add a regression test covering a root path that contains spaces

Why

Issue #78 looks like the HID++ device-identification fix and the blank schema are separate problems. The screenshots show the device is identified correctly, but the mouse image never loads in the packaged app, which collapses the hotspots into the center.

A stronger suspect is packaged resource resolution: older frozen builds explicitly resolved assets from _internal/, while current code could fall back to the executable directory instead. This keeps the packaged fallback and also tightens the image URL handling in QML.

Validation

  • .venv/bin/python -m unittest tests.test_backend -q
  • manual sanity check of main_qml._resolve_root_dir() for dev, frozen fallback, and _MEIPASS cases

Refs #78

@thisislvca thisislvca changed the title Fix packaged device image URLs for issue #78 Fix packaged asset resolution for issue #78 Apr 14, 2026
@thisislvca thisislvca marked this pull request as ready for review April 14, 2026 19:41
@thisislvca thisislvca merged commit 5e8588a into master Apr 21, 2026
2 checks passed
@thisislvca thisislvca deleted the fix-issue-78 branch April 21, 2026 12:18
@thisislvca thisislvca linked an issue Apr 21, 2026 that may be closed by this pull request
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Mouse schema is not displayed anymore

1 participant