Skip to content

fix(config): fix "save as" and file extension filter#2719

Merged
JPSchellenberg merged 2 commits intoLumieducation:masterfrom
mindcrunch4u:master
Jan 15, 2026
Merged

fix(config): fix "save as" and file extension filter#2719
JPSchellenberg merged 2 commits intoLumieducation:masterfrom
mindcrunch4u:master

Conversation

@mindcrunch4u
Copy link
Copy Markdown
Contributor

Root cause:

  • There's a split second after confirming save location where Electron couldn't find a focused window, this causes an Exception and the file isn't saved.

Changes:

  • identify focused window before showing save dialogue.

Verified by user @nettkursxyz in this issue.

According to contributing guidelines, the reviewer should be:

cc @JPSchellenberg

mindcrunch4u and others added 2 commits January 7, 2026 17:35
There's a split second after confirming save location
where Electron couldn't find a focused window,
this causes an Exception and the file isn't saved.
According to Electron's doc: https://www.electronjs.org/docs/latest/api/dialog
"The extensions array should contain extensions without wildcards or dots"
@mindcrunch4u
Copy link
Copy Markdown
Contributor Author

mindcrunch4u commented Jan 9, 2026

Root cause:

  • The original code uses .h5p as file extension, which isn't what Electron wants. See document.

Changes:

  • change .h5p extension to h5p.

Related issue: link

I've verified the fix - it works on both Windows and Linux.

@mindcrunch4u mindcrunch4u changed the title fix(config): identify focused window before showing save diag fix(config): fix "save as" and file extension filter Jan 9, 2026
@JPSchellenberg JPSchellenberg merged commit a84ecd3 into Lumieducation:master Jan 15, 2026
1 check passed
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.0.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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