Skip to content

Conversation

@nickdoherty
Copy link
Contributor

Token address is inserted according to the EIP-55 format.

About RPL:
Rocket Pool smart contracts accept ETH deposits, assign them to node operators, and issue & track various tokens (RPL/rETH) and handle interactions with the beacon chain.

Etherscan: https://etherscan.io/token/0xD33526068D116cE69F19A9ee46F0bd304F21A51f
Official Website: https://rocketpool.net/
Official Forum: https://dao.rocketpool.net/
Contact: contact@rocketpool.net
Twitter: https://twitter.com/Rocket_Pool
CoinGecko: https://www.coingecko.com/en/coins/rocket-pool
CoinMarketCap: https://coinmarketcap.com/currencies/rocket-pool/

@nickdoherty nickdoherty requested review from a team and MRabenda as code owners March 29, 2022 02:16
},
"0xD33526068D116cE69F19A9ee46F0bd304F21A51f": {
"name": "Rocket Pool",
"logo": "rpl.png",
Copy link
Collaborator

Choose a reason for hiding this comment

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

SCG is required

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unfortunately the icon uses some complex gradients and subtle shadows which don't scale well using an SVG, which is why we've opted with a png (we don't have SVGs available because of the gradient scaling).

Is there any flexibility around using a png?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Have replaced with an SVG as requested.

@nickdoherty
Copy link
Contributor Author

Any update on this? Requested changes have been made.

@MRabenda
Copy link
Collaborator

SVG image size is still too big - regular on should have around 5kB - if you can decrease you 200kB to sth around 20-30kB would be great

@nickdoherty
Copy link
Contributor Author

Reduced file size to 16kb.

@MRabenda
Copy link
Collaborator

MRabenda commented May 5, 2022

Can you merge master and solve conflicts - I am not able to merge it

@nickdoherty
Copy link
Contributor Author

Fixed merge conflicts.

@MRabenda MRabenda merged commit d0b86a1 into MetaMask:master May 9, 2022
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