Commit e34218d
authored
Fix mobile vault external link
## Why
The primary external vault CTA was hidden on mobile vault detail pages. On the YieldNest Max Vaults page this meant the Euler link was present on desktop, but unavailable in the mobile header.
## Lessons learnt
Vault detail header CTAs need explicit mobile coverage because the Technical Details fallback link can hide a header visibility regression.
## Summary
- Keep the external vault CTA visible in the vault detail header on mobile.
- Add an integration test covering mobile visibility for an external vault link.1 parent df85d34 commit e34218d
3 files changed
Lines changed: 16 additions & 11 deletions
File tree
- src/routes/trading-view/vaults/[vault=slug]
- tests
- integration/trading-view/vaults
- mocks/top-vaults
Lines changed: 3 additions & 11 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
27 | 27 | | |
28 | 28 | | |
29 | 29 | | |
30 | | - | |
31 | | - | |
32 | | - | |
33 | | - | |
34 | | - | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
35 | 33 | | |
36 | 34 | | |
37 | 35 | | |
| |||
50 | 48 | | |
51 | 49 | | |
52 | 50 | | |
53 | | - | |
54 | | - | |
55 | | - | |
56 | | - | |
57 | | - | |
58 | | - | |
59 | 51 | | |
60 | 52 | | |
61 | 53 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
339 | 339 | | |
340 | 340 | | |
341 | 341 | | |
| 342 | + | |
| 343 | + | |
| 344 | + | |
| 345 | + | |
| 346 | + | |
| 347 | + | |
| 348 | + | |
| 349 | + | |
| 350 | + | |
| 351 | + | |
| 352 | + | |
| 353 | + | |
342 | 354 | | |
343 | 355 | | |
344 | 356 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
315 | 315 | | |
316 | 316 | | |
317 | 317 | | |
| 318 | + | |
318 | 319 | | |
319 | 320 | | |
320 | 321 | | |
| |||
0 commit comments