search

beefyfinance / beefy-v2

Beefy is a Decentralized, Multichain Yield Optimizer that allows its users to earn compound interest on their crypto holdings. Beefy earns you the highest APYs with safety and efficiency in mind.
https://app.beefy.com
MIT License
49 stars 96 forks source link

Some asset images are excessively large #2226

Open ReflectiveChimp opened 2 weeks ago

ReflectiveChimp commented 2 weeks ago

PNGs can be resized down to 128x128 if larger and losslessly compressed SVGs can be losslessly compressed (e.g. with SVGO)

128K ./src/images/single-assets/wBaseDOGE.png
131K ./src/images/single-assets/RETRO.svg
131K ./src/images/single-assets/oRETRO.svg
145K ./src/images/single-assets/crvUSDBTCETH3.svg
169K ./src/images/single-assets/RPL.png
177K ./src/images/single-assets/NORMUS.png
222K ./src/images/single-assets/PEARL.png
377K ./src/images/single-assets/hDAI.png
379K ./src/images/single-assets/hUSDT.png
383K ./src/images/single-assets/hSNX.png
396K ./src/images/single-assets/hETH.png
401K ./src/images/single-assets/hUSDC.png
471K ./src/images/single-assets/AERO.svg
559K ./src/images/single-assets/weETH.png
DefiDebauchery commented 1 week ago

Should it be added to pre-commit hooks, or is it good enough to do once-off ops?

Similarly, should we use 384 instead (128 x3 for hi-dpi screens)? I was able to get weETH to 55K at 384, for instance. Not sure where we're even using this asset, though; I only see the SVG in use.