[Hold] Chat - Attachments thumbnail becomes blank for a moment in the preview when loading spinner appears

[kavimuru]

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!

---

Action Performed:

1. Go to https://staging.new.expensify.com/[](javascript:)
2. Log in with any account
3. Click on Plus button
4. Attach any picture

Expected Result:

Attachment is visible all the time while uploading

Actual Result:

Attachment is blank for a moment while uploading

Workaround:

Unknown

Platform:

Where is this issue occurring?

• Web
• iOS
• Android
• Mobile Web

Version Number: 1.1.51.0
Reproducible in staging?: y
Reproducible in production?: y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Bug5521419_Bug5521298_Screen_Recording_20220405-163952_Chrome.mp4

Expensify/Expensify Issue URL:
Issue reported by: Applause
Slack conversation:

View all open jobs on GitHub

[melvin-bot]

Triggered auto assignment to @johnmlee101 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[melvin-bot]

@johnmlee101 Eep! 4 days overdue now. Issues have feelings too...

[johnmlee101]

Whoops didn't mean to remove Overdue directly

[johnmlee101]

This seems like a minor issue that we could open up to be fixed. The issue is when we switch the local version of the photo to the uploaded version. We should make sure its ready to be fully rendered before removing the thumbnail and spinner.

[melvin-bot]

Triggered auto assignment to @JmillsExpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

[thesahindia]

I think we won't need this after #8590 gets fixed

[JmillsExpensify]

Let's go ahead and put a hold on it then, and if it can't be reproduced in testing once #8590 goes live, we can close.

[JmillsExpensify]

Keeping open until the related issue gets further along in the PR process.

[JmillsExpensify]

All the action is still in #8590

[JmillsExpensify]

Still working out linked issue above.

[JmillsExpensify]

Contributor hired in the linked issue above.

[JmillsExpensify]

Still working through the considerations in the linked issue.

[JmillsExpensify]

Same same in the linked issue. I'm still keeping a weekly on this one since we've landed at C+ PR review.

[JmillsExpensify]

Still discussing in the linked issue.

[JmillsExpensify]

Great discussion in the linked issue, still going.

[JmillsExpensify]

@michaelhaxhiu Looks like #8590 is in production now and nearly through the hold for the regression period. What do you think about taking this issue now? Did we end up resolving this, or should we take this issue off hold?

[michaelhaxhiu]

@JmillsExpensify nice timing 😃 .

Ok so - this is actually an obsolete bug report now, as far as I can see! The fix in #8590 changed the behavior of uploading images and now we don't show a blank white screen. This PR fix has some videos of uploads to demonstrate.

I think we can close this now!

[JmillsExpensify]

Nice to see we ended up resolving! I'll close.