Skip to content

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

Merged
ara4n merged 2 commits intodevelopfrom
t3chguy/electron_logo_windows_weirdness
Jun 4, 2017
Merged

to make the windows volume mixer not explode as it can't resize icons.#4183
ara4n merged 2 commits intodevelopfrom
t3chguy/electron_logo_windows_weirdness

Conversation

@t3chguy
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Member Author

t3chguy commented Jun 4, 2017

fixes #4185

@ara4n
Copy link
Copy Markdown
Member

ara4n commented Jun 4, 2017

that's hilarious. lgtm.

@ara4n ara4n merged commit 7b9ae01 into develop Jun 4, 2017
@t3chguy
Copy link
Copy Markdown
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