Skip to content

Add PKHeX installer (Silver)#477

Draft
Azevedoc wants to merge 2 commits intobottlesdevs:mainfrom
Azevedoc:add-pkhex-installer
Draft

Add PKHeX installer (Silver)#477
Azevedoc wants to merge 2 commits intobottlesdevs:mainfrom
Azevedoc:add-pkhex-installer

Conversation

@Azevedoc
Copy link
Copy Markdown

@Azevedoc Azevedoc commented Apr 11, 2026

Add PKHeX, a Pokémon core series save editor. Requires .NET Desktop Runtime 10 (installed automatically) and a DPI awareness registry workaround to prevent crashes on file dialogs under Wine.

Workarounds included:

  • Sets DPIUNAWARE AppCompatFlags registry key to avoid SetThreadDpiAwarenessContext crash in .NET 10 WinForms

Known cosmetic issues:

  • Window is not resizable and renders small (still usable)
  • Pokemon tooltip popups are inconsistent
  • Wine 11+ recommended as runner

Note: During testing, the installer's "Proceed" button for local resources was invisible on a system with libadwaita 1.9+. This seems to be an upstream Bottles bug — btn_proceed in dialog-installer.blp is a bare GtkButton child of AdwPreferencesPage, which may silently drop non-group children in newer libadwaita. Details documented in Reviews/bottles-libadwaita-1.9-proceed-button.md.

Type of change

  • New installer
  • Manifest fix
  • Other

Was This Tested Using a Local Repository?

  • Yes
  • No

PKHeX is a Pokémon core series save editor running on .NET Desktop
Runtime 10. Includes a DPI awareness workaround for Wine compatibility
and a review documenting current test results.
@Azevedoc Azevedoc marked this pull request as ready for review April 11, 2026 18:05
Move install directory from Program Files/PKHeX to PKHeX to avoid
path quoting issues with Bottles. Document the required env var
workaround (DOTNET_SYSTEM_GLOBALIZATION_USENLS=true) for .NET ICU
culture enumeration bug when running under Wine.
@jntesteves jntesteves marked this pull request as draft May 8, 2026 14:34
@jntesteves
Copy link
Copy Markdown
Member

jntesteves commented May 8, 2026

Hi, thanks for contributing. I think there are a lot of issues with it though:

  1. Bottles issues should be reported on the Bottles repository, not committed as a Markdown file here.
  2. We have a dependency installer that already includes dotnet10, it should be installed from there (by adding it to the Dependencies field, as it's done in other installers).
  3. The program doesn't seem to have an installer, so I don't really get why it needs an installer here. Running find on the filesystem and copying the file is not something we should do, the sandboxed filesystem often won't have access to it.

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.

2 participants