-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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] [$1000] mWeb - Contact gets highlighted on long press #19766
Comments
Triggered auto assignment to @puneetlath ( |
Bug0 Triage Checklist (Main S/O)
|
Job added to Upwork: https://www.upwork.com/jobs/~01458f36110327280c |
Current assignee @puneetlath is eligible for the External assigner, not assigning anyone new. |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @Santhosh-Sellavel ( |
Current assignee @puneetlath is eligible for the External assigner, not assigning anyone new. |
ProposalPlease re-state the problem that we are trying to solve in this issue.Contact gets highlighted on long press What is the root cause of that problem?On touch devices, long press sometimes fire hover event. its default behaviour can easily replicate and its not related to only search screen. bug videobug720p.movWhat changes do you think we should make in order to solve the problem?We can prevent it by checking App/src/components/Hoverable/index.js Lines 58 to 60 in 24381ed
We have two options.
// should allow hover only when device hasHoverSupport capability.
const isHovered = DeviceCapabilities.hasHoverSupport() && this.state.isHovered;
const child = _.isFunction(this.props.children) ? this.props.children(isHovered) : this.props.children;
render() {
if(!DeviceCapabilities.hasHoverSupport()) {
const childrenWithHoverState = _.isFunction(props.children) ? props.children(false) : props.children;
return <View style={props.containerStyles}>{childrenWithHoverState}</View>;
}
.....
} both will solve our problem globally. What alternative solutions did you explore? (Optional)None |
@puneetlath, @Santhosh-Sellavel Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@Santhosh-Sellavel thoughts on the proposal? |
@kushu7's proposal Looks good to me! @puneetlath |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
@puneetlath Sorry Yeah 2nd option looks good to me! |
Any update? |
📣 @kushu7 You have been assigned to this job by @puneetlath! |
Triggered auto assignment to @tylerkaraszewski ( |
Thanks, @amyevans! |
Ah ok - got it! |
@Santhosh-Sellavel it looks like #20585 is done. Does this still need to be held? |
@puneetlath This issue is what we discussed yesterday! |
@Santhosh-Sellavel @kushu7 it looks like this was made obsolete by another PR. However, since you were hired and already did the work, we'll still go ahead and pay you out (no bonus) for the effort. I'll go ahead and close the PR and I've sent you both hiring offers. Ping me here once you've accepted and I'll pay them out. Thanks for the hard work! |
Alright. @Santhosh-Sellavel will be paid via this. @kushu7 has been paid via Upwork. Thanks everyone! |
Hey @puneetlath, I think it is eligible for the reporting compensation. |
@thesahindia sorry about that. Sent you an offer. @Santhosh-Sellavel do you need me to leave the issue open for tracking your payment? Or am I good to close once I pay sahil? |
Let's just keep it open till next week, if it's not affecting anything! |
Ok sounds good. @thesahindia please accept the contract when you have a chance. |
Accepted, thanks! |
Ok, @thesahindia has been paid. So we can close this once @Santhosh-Sellavel has confirmed he's been paid via Expensify. |
@Santhosh-Sellavel did you receive payment yet? |
Everyone has been paid. Closing! |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Action Performed:
Expected Result:
It shouldn't get highlighted
Actual Result:
The contact gets highlighted
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.19.2
Reproducible in staging?: n/a
Reproducible in production?: n/a
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
video_20230525_164732_edit.mp4
Expensify/Expensify Issue URL:
Issue reported by: @thesahindia
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1685013467217839
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: