fix(simulation): scale token logo fallback and handle image load errors

[andrewwahid]

Summary by CodeRabbit

• Bug Fixes
  • Improved fallback handling for token logo display when logos are unavailable or fail to load, ensuring consistent visual presentation across all scenarios.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e2903d99-8853-4c7e-8894-afa2c798c5de

📥 Commits

Reviewing files that changed from the base of the PR and between 196fa4f and 7ed8b9c.

📒 Files selected for processing (1)

• lib/features/verify_safe_transaction/simulation/safe_tx_simulation_screen.dart

---

📝 Walkthrough

Walkthrough

Updated the _tokenLogo widget in the safe transaction simulation screen to improve error resilience. The change extracts a reusable fallback widget (grey "?" with scaled styling) that is now applied both when the logo URI is explicitly "unknown" and when network image loading fails via an errorBuilder callback.

Changes

Token Logo Fallback Handling

Layer / File(s)	Summary
Token logo fallback widget lib/features/verify_safe_transaction/simulation/safe_tx_simulation_screen.dart	_tokenLogo method now defines a shared fallback widget and uses it for both the "unknown" logo URI case and when Image.network fails to load, adding robust error handling via errorBuilder.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• candidelabs/safe-opensig#84: Both PRs modify the _tokenLogo behavior in safe_tx_simulation_screen.dart—this PR adds error handling and a shared fallback widget, while the other PR introduces or expands usage of the token logo placeholder in token detail and tappable UI components.

Poem

🐰 A logo fell into the net, a grey question mark rose instead,
Now reused, reliable, and shared—no fallback left unspread.
From "unknown" to error, one path appears so neat,
Making token screens resilient, the fix is sweet! ✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main changes: scaling the token logo fallback and adding image load error handling.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}