-
Notifications
You must be signed in to change notification settings - Fork 65
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
Update CyberChef to use CyberChef icon #1054
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the changes @emtuls! After some more testing, I am not sure that converting the image to icon is the best approach. The current code make the shortcut looks like:
But we could download and use directly the icon in https://raw.githubusercontent.com/gchq/CyberChef/18159ce806e63286019a5a6828422a992fbb253b/src/web/static/images/favicon.ico which I think looks a bit better and removes the need for the extra code:
It is also confusing and inconsistent with other apps (like capa or fakenet) that the icon in the taskbar is a different one. This is related to #1021 and #822, but I think we should address then in the same PR so that the icon change takes place everywhere.
@emtuls 👀 |
221b105
to
1b8dcd8
Compare
@d35ha can you test this locally and share your opinion on this? 😄 |
@stevemk14ebr @mr-tz @binjo @d35ha any preference on the icon? The icons look a bit different (see screenshots in the previous comments). @emtuls prefers to generate Robot-Chef icon (requires some additional code to convert an image file to a |
I don't mind either, I vote for whichever is easiest to maintain |
I second this. FWIW taskbars may be configured in other colors (mine is black). I agree that gray on gray is a bit hard to see. |
After googling CyberChef and seeing most websites/videos use the Robot-Chef icon, I'm inclined to keep that style for now. :) |
9dfdcd4
to
81433e7
Compare
81433e7
to
8d127df
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 👍
I find it confusing to have the CyberChef icon in the taskbar and that when clicking it, it opens Chrome as a different taskbar icon:
But I think this is already part of #1100 and we can continue to discuss how to address it there.
This converts the CyberChef PNG file that is provided with the download and converts it to a
.ico
so that Chocolatey can use it for an icon for the shortcut.Closes #966