Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix broken image fallback #12679

Merged
merged 3 commits into from
Jun 26, 2023
Merged

Fix broken image fallback #12679

merged 3 commits into from
Jun 26, 2023

Conversation

Jibz-Mysten
Copy link
Contributor

Description

Whenever browsing between pages/screens in Explorer and Wallet, the fallback avatar sometime does not show. This addresses it by checking for the URL string before rendering the image

Fixes this
Screenshot 2023-06-25 at 6 54 00 PM

@Jibz-Mysten Jibz-Mysten requested a review from a team as a code owner June 25, 2023 22:57
@Jibz-Mysten Jibz-Mysten requested review from Nikhil-Mysten and hayes-mysten and removed request for a team June 25, 2023 22:57
@vercel
Copy link

vercel bot commented Jun 25, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
explorer ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 26, 2023 0:45am
explorer-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 26, 2023 0:45am
4 Ignored Deployments
Name Status Preview Comments Updated (UTC)
multisig-toolkit ⬜️ Ignored (Inspect) Jun 26, 2023 0:45am
sui-kiosk ⬜️ Ignored (Inspect) Jun 26, 2023 0:45am
sui-wallet-kit ⬜️ Ignored (Inspect) Jun 26, 2023 0:45am
wallet-adapter ⬜️ Ignored (Inspect) Jun 26, 2023 0:45am

@Jibz-Mysten Jibz-Mysten merged commit 1de4a64 into main Jun 26, 2023
@Jibz-Mysten Jibz-Mysten deleted the gj--fix-broken-image-fallback branch June 26, 2023 18:21
ebmifa pushed a commit that referenced this pull request Jul 12, 2023
## Description 

Whenever browsing between pages/screens in Explorer and Wallet, the
fallback avatar sometime does not show. This addresses it by checking
for the URL string before rendering the image

Fixes this 
<img width="1477" alt="Screenshot 2023-06-25 at 6 54 00 PM"
src="https://github.com/MystenLabs/sui/assets/126525197/b8706f58-88d0-4277-8464-6983e2b6b193">
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.

4 participants