Skip to content

feat: add HyperLend (HPL) icon for HyperEVM network#1749

Merged
MRabenda merged 1 commit intoMetaMask:masterfrom
fbslo:master
Mar 12, 2026
Merged

feat: add HyperLend (HPL) icon for HyperEVM network#1749
MRabenda merged 1 commit intoMetaMask:masterfrom
fbslo:master

Conversation

@fbslo
Copy link
Contributor

@fbslo fbslo commented Mar 10, 2026

Note

Low Risk
Low risk asset-only change: adds new ERC-20 metadata and an SVG logo file without touching any runtime code paths.

Overview
Adds a new ERC-20 token entry for HyperLend (HPL) on eip155:999 by introducing metadata/eip155:999/erc20:0xBD6D...C91B.json (name/symbol/decimals + logo pointer) and the corresponding icons/eip155:999/erc20:0xBD6D...C91B.svg logo asset.

Written by Cursor Bugbot for commit e27905d. This will update automatically on new commits. Configure here.

@fbslo fbslo requested review from a team and MRabenda as code owners March 10, 2026 11:04
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

@@ -0,0 +1 @@
<svg width='512' height='242' viewBox='0 0 512 242' fill='none' xmlns='http://www.w3.org/2000/svg'><path d='M511.964 62.451C511.964 78.9633 511.511 95.4755 511.964 111.988C512.418 121.514 508.562 124.478 498.808 123.843C489.055 123.208 479.301 123.843 469.321 123.843C456.391 123.843 446.184 128.5 437.792 137.603C409.665 167.452 381.538 197.09 353.185 226.727C345.472 234.772 335.265 239.006 323.243 239.429C286.043 240.276 281.961 237.735 283.095 202.805C283.775 180.366 283.095 157.714 283.321 135.274C283.321 126.383 279.919 123.208 270.619 123.843C258.144 124.478 248.39 128.924 240.224 137.603C213.005 166.605 185.332 195.608 157.885 224.398C144.729 238.159 116.603 245.145 98.6832 238.794C89.8369 235.618 85.5272 229.267 85.754 219.953C86.2076 192.432 85.754 165.124 85.754 137.603C85.754 124.69 85.3003 124.055 70.7833 124.055C51.0493 124.055 31.542 123.631 12.0348 124.055C2.50799 124.266 -0.213954 121.091 0.0128747 112.411C0.69336 91.2416 0.0128886 70.2837 0.239717 49.1141C0.466545 22.2287 23.1494 1.05907 51.7297 0.847376C64.2053 0.847376 76.6809 1.48245 89.1564 0.635667C98.6832 0.000578728 100.271 3.59943 100.271 11.4322C99.8174 39.5878 100.271 67.7433 100.271 95.8989C100.271 101.191 100.271 106.484 105.715 110.083C112.747 114.952 118.19 112.623 123.407 107.119C146.317 82.9855 170.588 59.6989 191.91 34.2954C218.449 3.17607 253.38 -2.11638 292.168 0.635667C299.426 1.27076 297.385 6.56316 297.612 10.162C297.612 37.2591 297.612 64.3562 297.612 91.4533C297.612 94.6288 297.612 97.8042 297.838 100.98C298.519 106.272 301.014 110.506 306.685 112.2C312.129 113.681 316.665 111.988 320.748 107.754C344.565 82.3504 369.063 57.582 392.653 31.9667C412.387 10.5854 436.204 -0.63451 466.599 0.635667C478.167 1.05906 489.962 1.05906 501.757 0.635667C509.242 0.423971 511.738 2.96436 511.738 9.73864C511.511 27.3094 511.738 44.8802 511.738 62.451H511.964Z' fill='#CAEAE5'/></svg> No newline at end of file
Copy link

Choose a reason for hiding this comment

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

Non-square SVG icon violates style guide requirements

Medium Severity

The SVG has a viewBox of 0 0 512 242 and dimensions of 512×242, which is a wide rectangle (~2:1 ratio). The STYLEGUIDE.md requires a square artboard (e.g., 256×256, 512×512), and CONTRIBUTING.md states icons must be square. Other icons in the same directory use square dimensions. This non-square icon will render poorly when cropped into a circle in the UI. Additionally, the icon uses only a #CAEAE5 fill with no background, making it nearly invisible in light mode.

Fix in Cursor Fix in Web

@MRabenda MRabenda merged commit 88ecea4 into MetaMask:master Mar 12, 2026
9 checks passed
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.

2 participants