chore: collapsed transaction info #998

[0xeye]

#998

x	x

Changes:

• collapsed state for deposit info
• hide vault share info when 0, to stop 0 = 0 state
• hide usd on 0
• adjust text to match Withdraw section, lower case text, upper case start.

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
yearnfi	Ready	Preview, Comment	Mar 3, 2026 3:18pm

[rossgalloway]

I think I may have led us astray in my comment in #998 where we are now in an uncomfortable middle ground. Probably better to show a single line, like you mentioned initially in the issue, and allow expand to show all, or just show all. Now we are in a weird situation where we show a bit and hide a bit.

[0xeye]

I think I may have led us astray in my comment in #998 where we are now in an uncomfortable middle ground. Probably better to show a single line, like you mentioned initially in the issue, and allow expand to show all, or just show all. Now we are in a weird situation where we show a bit and hide a bit.

Good point, I've updated it. Let me know what you think

[rossgalloway]

Good point, I've updated it. Let me know what you think

yep better. You will also need to revert the compressed state for that component that I created when the page gets short. There is one condition where the info section gets collapsed that we don't need with this change. see it by adding dev tools at the bottom of the browser.

I'd also say that this is something we should do some user testing around. Can we leave both options in? or have a branch that persists with both options?