Skip to content

Introduce check icon #204

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

Merged
merged 1 commit into from
Dec 14, 2022
Merged

Introduce check icon #204

merged 1 commit into from
Dec 14, 2022

Conversation

jarolrod
Copy link
Member

@jarolrod jarolrod commented Dec 14, 2022

Introduces the check icon from the Bitcoin Icons set. The svg is taken from optimized/filled and the png has been ran through the optimization script.

The current production use-case of this icon will be for the check icon that is part of the current design for the OptionButton control.

The check icon was taken from the Bitcoin-Icons Repo. This was used to generate a PNG of 128x128 size using inkscape. The resulting PNG was then run through the optimize-pngs.py script. The output of the script is as follows:

optimizing check.png... done
summary:
+++++++++++++++++
check.png
  size diff from: 1291 to: 829
  old sha256: 4436036d05b03fb55ad86e684beb125aa4805b78c4b4d276e5a372ffe5ef2941
  new sha256: c8f95de9fa3c98fa318f0eaa55e658f61d6a19e78761b2ccd774f9f9ba6d8ee4

completed. Checksum stable: False. Total reduction: 462 bytes

You can demo this by testing #205

Windows
Intel macOS
Apple Silicon macOS
ARM64 Android

Copy link
Collaborator

@johnny9 johnny9 left a comment

Choose a reason for hiding this comment

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

ACK 76287e4

Looks like a pretty useful icon to have

@hebasto hebasto merged commit 1c865a1 into bitcoin-core:main Dec 14, 2022
hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 8, 2025
hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 9, 2025
hebasto pushed a commit to hebasto/gui-qml that referenced this pull request Jun 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants