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

to make the windows volume mixer not explode as it can't resize icons. #4183

Merged
merged 2 commits into from
Jun 4, 2017

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented Jun 4, 2017

fixes
image

Might not be the best way, probably worth @dbkr looking it over.

I hate Windows.

Signed-off-by: Michael Telatynski 7t3chguy@gmail.com

               I hate Windows.

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@t3chguy t3chguy force-pushed the t3chguy/electron_logo_windows_weirdness branch from 90ae7bb to 25a0af6 Compare June 4, 2017 10:05
@t3chguy
Copy link
Member Author

t3chguy commented Jun 4, 2017

Alternative is extracting the red circle badge component of the image and using the actual overlay function, that would mean another lot of canvas etc in the renderer though

@t3chguy
Copy link
Member Author

t3chguy commented Jun 4, 2017

Oh actually this won't work at all and only works because of my monitor being at 125% DPI Scale.
### DO NOT MERGE

@t3chguy t3chguy force-pushed the t3chguy/electron_logo_windows_weirdness branch from 5e0fa0a to 685d45b Compare June 4, 2017 13:26
…delay

(not noticeably more than the existing delay)

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@t3chguy t3chguy force-pushed the t3chguy/electron_logo_windows_weirdness branch from 685d45b to 1e5a742 Compare June 4, 2017 13:27
@t3chguy
Copy link
Member Author

t3chguy commented Jun 4, 2017

This should fix it, converts png to a proper ico file, have to save it as nativeImage has no proper support for icos and can only load them form a file, can't read from buf etc

@t3chguy
Copy link
Member Author

t3chguy commented Jun 4, 2017

fixes #4185

@ara4n
Copy link
Member

ara4n commented Jun 4, 2017

that's hilarious. lgtm.

@ara4n ara4n merged commit 7b9ae01 into develop Jun 4, 2017
@t3chguy
Copy link
Member Author

t3chguy commented Jun 4, 2017

Agreed, gotta love Windows ... right!?

@t3chguy t3chguy deleted the t3chguy/electron_logo_windows_weirdness branch October 29, 2017 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants