Skip to content

Conversation

caspermeijn
Copy link
Contributor

@caspermeijn caspermeijn commented Aug 30, 2024

winapi has not been updated for four years. windows-sys seems to have a good maintainer. winit also uses windows-sys.

The Windows API calls are not changed. It just uses a different bindings library.

  • I have followed the instructions in the PR template

Alternative

I tried to remove the Windows API calls from this crate. As stated in a comment, winit can do some of the work, but I was unable to fix winit enough to use that. My poor knowledge of Windows API doesn't help with that.

// We would get fairly far already with winit's `set_window_icon` (which is exposed to eframe) actually!
// However, it only sets ICON_SMALL, i.e. doesn't allow us to set a higher resolution icon for the task bar.
// Also, there is scaling issues, detailed below.

`winapi` has not been updated for four years. `windows-sys` seems to have a good maintainer. `winit` also uses `windows-sys`.

The Windows API calls are not changed. It just uses a different bindings library.
@caspermeijn caspermeijn marked this pull request as ready for review September 11, 2024 08:18
@emilk emilk added eframe Relates to epi and eframe dependencies Pull requests that update a dependency file labels Sep 16, 2024
@emilk emilk closed this in #7416 Aug 12, 2025
emilk pushed a commit that referenced this pull request Aug 12, 2025
This the second take of <#5038>,
adjusted for newer version of `windows-sys`, and with merge conflicts
resolved. I added the original author to `Co-authored-by:`.

* Closes <#7334>
* Closes <#5038>
* [x] I have followed the instructions in the PR template

Tested on Windows 10:
<img width="322" height="272" alt="image"
src="https://github.com/user-attachments/assets/15ddf0bf-1f30-452f-8764-07ccfce2b00c"
/>
<img width="802" height="632" alt="image"
src="https://github.com/user-attachments/assets/db6f4586-cfa5-4f7f-996c-06f7e2489df6"
/>

Action run result:
https://github.com/sola-contrib/egui/actions/runs/16748810761

---------

Co-authored-by: Casper Meijn <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file eframe Relates to epi and eframe

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants