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

New app icon #32

Closed
2 tasks done
patrickhlauke opened this issue May 15, 2017 · 7 comments
Closed
2 tasks done

New app icon #32

patrickhlauke opened this issue May 15, 2017 · 7 comments
Assignees

Comments

@patrickhlauke
Copy link
Contributor

patrickhlauke commented May 15, 2017

For consistency with the TPG brand (and to make it slightly more visually appealing), started work on a new icon in a separate branch https://github.com/ThePacielloGroup/aviewer/tree/new-app-icon

Work is not complete here, as I believe the following still need to be resolved:

  • convert all png files into a multi-image ico
  • recompile the 32bit and 64bit exe files to use the new icon
@patrickhlauke
Copy link
Contributor Author

@jun325 if you have any experience in the creation of appropriate multi-image ico files, give me a shout and we can coordinate

@patrickhlauke
Copy link
Contributor Author

For reference, adding the large version and 16x16 version here.

256x256

16x16

@patrickhlauke
Copy link
Contributor Author

seem to have managed to create a multi-image ico file, but not sure if it's correct 4d01995

@patrickhlauke
Copy link
Contributor Author

Tested the ico by creating a shortcut to an exe and manually assigning the ico to this shortcut, then resizing the icons in file explorer. At list size, it correctly shows the custom 16x16 version, so assuming the multi-image ico is correct. I've now merged the separate branch into Develop ... @jun325 could you recompile the 32bit and 64bit versions using the new Aviewer_Icon.ico file in the development branch? This should then hopefully result in the nice multi-image version being used for the executable at different sizes, and the custom 16x16 to appear in the application window title bar, I assume. Then I can package this up for next release :)

@patrickhlauke
Copy link
Contributor Author

(fair warning that I'll do a new icon for CCA-Win as well in the coming days)

ghost pushed a commit that referenced this issue May 29, 2017
@ghost
Copy link

ghost commented May 29, 2017

I changed it. 225fb63

@patrickhlauke
Copy link
Contributor Author

Perfect, thanks @jun325

@patrickhlauke patrickhlauke changed the title New installer/app icon New app icon May 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant