-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[HOLD for 18997] [$1000] Android- infinite loading on open profile pic preview second time #22920
Comments
Triggered auto assignment to @adelekennedy ( |
Bug0 Triage Checklist (Main S/O)
|
ProposalPlease re-state the problem that we are trying to solve in this issue.Android- infinite loading on open profile pic preview second time What is the root cause of that problem?On second loading of ImageView, On first loading, App/src/components/ImageView/index.native.js Lines 211 to 227 in 405fcdf
App/src/components/ImageView/index.native.js Lines 142 to 148 in 405fcdf
When App/src/components/ImageView/index.native.js Lines 105 to 121 in 405fcdf
Once image is cached, What changes do you think we should make in order to solve the problem?In <Image
style={[
styles.w100,
styles.h100,
this.props.style,
// Hide image while loading so ImageZoom can get the image
// size before presenting - preventing visual glitches or shift
// due to ImageZoom
shouldShowLoadingIndicator ? styles.opacity0 : styles.opacity1,
]}
source={{uri: this.props.url}}
isAuthTokenRequired={this.props.isAuthTokenRequired}
resizeMode={Image.resizeMode.contain}
onLoadStart={this.imageLoadingStart}
onLoad={this.configureImageZoom}
onLoadEnd={() => this.setState({isLoading: false})}
/> And prevent to initialize image dimension in imageLoadingStart() {
if (this.state.isLoading) {
return;
}
this.resetImageZoom();
} ResultScreen.Recording.2023-07-15.at.7.03.08.AM.movWhat alternative solutions did you explore? (Optional)We can prevent to invoke imageLoadingStart() {
if (this.state.isLoading || (this.state.imgHeight !== 0 && this.state.imgWidth !== 0)) {
return;
}
this.resetImageZoom();
this.setState({imageHeight: 0, imageWidth: 0, isLoading: true});
} |
I'm not sure if it's dup with #16046 |
Job added to Upwork: https://www.upwork.com/jobs/~01d82661194c0b255b |
Current assignee @adelekennedy is eligible for the External assigner, not assigning anyone new. |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @aimane-chnaif ( |
I was too fast with making this external, walking it back since this may be related to a larger issue |
I think its better to use |
deprioritizing for a mo |
still holding here |
on hold |
The issue is reproducible on staging build 1.4.8.0, reopening @adelekennedy Android-Avatar-preview-endless-spinner-staging.mp4 |
we're still on hold here |
not reproducible |
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:
Expected Result:
Should show profile pic preview
Actual Result:
Infinite loading on open profile pic
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.39-11
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Screen_Recording_20230714_164616_New.Expensify.mp4
Screen_Recording_20230714_170233_Expensify.Chat.1.mp4
Expensify/Expensify Issue URL:
Issue reported by: @gadhiyamanan
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1689333392843659
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: