Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix window icon on Windows #790

Merged
merged 3 commits into from
Jul 3, 2022
Merged

Conversation

SteffenL
Copy link
Collaborator

@SteffenL SteffenL commented Jun 30, 2022

See commit messages.

Fixes issues raised in #781.

Fixes an issue where the window icon looks blurry.
Fixes window icon by loading a correctly sized icon and setting it as the
hIcon member of WNDCLASSEXW. Previously a small icon was always loaded and
set for the hIcon member.

If we do not have a small icon then we do not need to specify the hIconSm
member as the system will search hIcon for an appropriately sized icon.
@SteffenL SteffenL changed the title Fix window icon Fix window icon on Windows Jun 30, 2022
Copy link

@precisionpete precisionpete left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That makes sense...

@SteffenL SteffenL merged commit fffd6d0 into webview:master Jul 3, 2022
@SteffenL SteffenL deleted the up/fix-window-icon branch July 3, 2022 15:56
@SteffenL SteffenL added this to the v0.10.0 milestone Sep 1, 2023
@SteffenL SteffenL added the type: bug report Issue is a bug report label Sep 1, 2023
@SteffenL SteffenL added type: fix Pull request is a fix and removed type: bug report Issue is a bug report labels Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: fix Pull request is a fix
Development

Successfully merging this pull request may close these issues.

3 participants