-
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
Web - Workspace - "More" tooltip is displayed when hovering over the three dots after clicking on them #24300
Comments
Triggered auto assignment to @zanyrenney ( |
Bug0 Triage Checklist (Main S/O)
|
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open
|
Triggered auto assignment to @AndrewGable ( |
Confirmed I can only reproduce this on staging |
ProposalPlease re-state the problem that we are trying to solve in this issue.Web - Workspace - "More" tooltip is displayed when hovering over the three dots after clicking on them What is the root cause of that problem?We are not hiding the tooltip when popover is open. What changes do you think we should make in order to solve the problem?Pass another prop to toolTip App/src/components/ThreeDotsMenu/index.js Line 88 in c70a619
shouldDisplay={!this.state.isPopupMenuVisible} and then in Tooltip conditionally render it What alternative solutions did you explore? (Optional)We can also hide tooltip by passing prop isPopupMenuVisible |
I don't think this needs to be a deploy blocker. It also seems similar to #24284 |
I think the fact that it still show the tooltip when you re-hover is fine. The FAB also does that. I think instead the problem is that re-clicking on it doesn't do anything. |
I agree @puneetlath , may be clicking on it again should hide the dropdown. |
📣 @samueltuyizere! 📣
|
Contributor details |
✅ Contributor details stored successfully. Thank you for contributing to Expensify! |
ProposalI agree with Puneet, that we should hide the modal if we click three dot again. Please re-state the problem that we are trying to solve in this issue.Threedot button does nothing when popup is already visible. What is the root cause of that problem?In our onPress function of threedot icon, we only have logic of showing it here: App/src/components/ThreeDotsMenu/index.js Lines 91 to 94 in 0f83752
That's why once it opened and we clicked threedot icon, we won't see anything happened. What changes do you think we should make in order to solve the problem?We can change this to: onPress={() => {
if (this.state.isPopupMenuVisible) {
this.hidePopoverMenu();
} else {
this.showPopoverMenu();
}
if (this.props.onIconPress) {
this.props.onIconPress();
}
}} What alternative solutions did you explore? (Optional)N/A |
Dupe of #24296 |
@aimane-chnaif, I would like to bring to your attention that I have previously reported an issue regarding confusion in creating GitHub issues. Kindly refer to the following Slack conversation for further details: https://expensify.slack.com/archives/C049HHMV9SM/p1691501020584879." |
@tewodrosGirmaA we already planned to fix as a follow-up, so no bug reporting bonus here. |
@aimane-chnaif , this is not fair , just without any reason, I reported first |
Huh? How come you can report bug even if PR was not merged yet? |
That is my magic hhhh , I mean I reported first before this #24296, |
@tewodrosGirmaA saying again, it's firstly reported by QA team while PR testing before merge. @zanyrenney let's close this as dupe |
Closing as dupe |
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:
After opening the details, hovering over the three dots should not display the "More" tooltip again and the Cursor should be Changed
Actual Result:
After opening the details, hovering over the three dots displays the "More" tooltip again.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: v1.3.51-2
Reproducible in staging?: Y
Reproducible in production?: N
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
Recording.1136.mp4
screen-capture.-.2023-08-08T062550.532.mp4
Expensify/Expensify Issue URL:
Issue reported by: @tewodrosGirmaA
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1691501020584879
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: