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

[$125] Web - Onboarding - Onboarding modal's corner become rectangular when hovering over the back button #48684

Closed
1 of 6 tasks
IuliiaHerets opened this issue Sep 6, 2024 · 39 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause

Comments

@IuliiaHerets
Copy link

IuliiaHerets commented Sep 6, 2024

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Version Number: 9.0.30-7
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers): nathanmulugetatesting+1383@gmail.com
Issue reported by: Applause Internal Team

Action Performed:

  1. Navigate to staging.new.expensify.com
  2. Sign up with a new account
  3. When the onboarding modal shows up click on something else
  4. Hover over the back button and observe the corner of the modal

Expected Result:

Nothing unusual happens to the corners of the onboarding modal

Actual Result:

Corners of the onboarding modal become rectangular first until the tooltip shows up and reverts back to being curved

Workaround:

Unknown

Platforms:

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Bug6595105_1725610381650.bandicam_2024-09-06_11-08-18-975.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021832074357963924539
  • Upwork Job ID: 1832074357963924539
  • Last Price Increase: 2024-09-27
@IuliiaHerets IuliiaHerets added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Sep 6, 2024
Copy link

melvin-bot bot commented Sep 6, 2024

Triggered auto assignment to @sakluger (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@IuliiaHerets
Copy link
Author

We think that this bug might be related to #wave-collect - Release 1

@IuliiaHerets
Copy link
Author

@sakluger FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

@sakluger
Copy link
Contributor

sakluger commented Sep 6, 2024

This seems like a basic styling issue. I'm going to set the price at $125.

@sakluger sakluger added the External Added to denote the issue can be worked on by a contributor label Sep 6, 2024
Copy link

melvin-bot bot commented Sep 6, 2024

Job added to Upwork: https://www.upwork.com/jobs/~021832074357963924539

@melvin-bot melvin-bot bot changed the title Web - Onboarding - Onboarding modal's corner become rectangular when hovering over the back button [$250] Web - Onboarding - Onboarding modal's corner become rectangular when hovering over the back button Sep 6, 2024
@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Sep 6, 2024
Copy link

melvin-bot bot commented Sep 6, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @akinwale (External)

@sakluger sakluger changed the title [$250] Web - Onboarding - Onboarding modal's corner become rectangular when hovering over the back button [$125] Web - Onboarding - Onboarding modal's corner become rectangular when hovering over the back button Sep 6, 2024
Copy link

melvin-bot bot commented Sep 6, 2024

Upwork job price has been updated to $125

@rlmorrison74
Copy link

rlmorrison74 commented Sep 6, 2024

Just tried to reproduce on mac-chrome, but I'm not experiencing this issue.

Edit: just tried safari as well, looks all good to me.

Copy link

melvin-bot bot commented Sep 6, 2024

📣 @rlmorrison74! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

  1. Make sure you've read and understood the contributing guidelines.
  2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
  3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
  4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
    Screen Shot 2022-11-16 at 4 42 54 PM
    Format:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

@rlmorrison74
Copy link

Contributor details
Your Expensify account email: rlmorrison.74@gmail.com
Upwork Profile Link: https://www.upwork.com/freelancers/~010522cfcacf6a249a

Copy link

melvin-bot bot commented Sep 6, 2024

✅ Contributor details stored successfully. Thank you for contributing to Expensify!

@sandeeppal007
Copy link

This issue is caused by a state re-render. When we hover over the back button, the card initially displays the default CSS, which lacks the border-radius property. Once the original CSS loads, the card becomes rounded as intended.

Copy link

melvin-bot bot commented Sep 7, 2024

📣 @sandeeppal007! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

  1. Make sure you've read and understood the contributing guidelines.
  2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
  3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
  4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
    Screen Shot 2022-11-16 at 4 42 54 PM
    Format:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

Copy link

melvin-bot bot commented Sep 7, 2024

✅ Contributor details stored successfully. Thank you for contributing to Expensify!

@shubham1206agra
Copy link
Contributor

@sakluger Please add retest label. #46788 might have fixed this problem

Copy link

melvin-bot bot commented Sep 9, 2024

@akinwale, @sakluger Whoops! This issue is 2 days overdue. Let's get this updated quick!

@melvin-bot melvin-bot bot added the Overdue label Sep 9, 2024
@sakluger sakluger added the retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause label Sep 9, 2024
@sakluger
Copy link
Contributor

sakluger commented Sep 9, 2024

Added the retest-weekly label.

@sakluger sakluger added Weekly KSv2 and removed Daily KSv2 labels Sep 9, 2024
Copy link

melvin-bot bot commented Sep 10, 2024

⚠️ Missing/invalid email or upwork profile link. Please make sure you add both your Expensify email and Upwork profile link in the format specified.

Copy link

melvin-bot bot commented Sep 13, 2024

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

@Bakhtiar85
Copy link

Proposal

Please re-state the problem that we are trying to solve in this issue.

The corners of the onboarding modal initially appear rectangular until the tooltip displays, at which point they return to their curved state. Additionally, if the user lightly touches the edge of the back button without pressing further, the corners remain rectangular.

What is the root cause of that problem?

The issue seems to stem from improper styling in the BaseOnboardingPersonalDetails component. This inconsistency causes the modal’s border-radius to temporarily reset.

What changes do you think we should make in order to solve the problem?

I will adjust the CSS for the BaseOnboardingPersonalDetails component, ensuring the border-radius remains consistent, even when the tooltip appears or when hovering near the back button.

What alternative solutions did you explore? (Optional)

One alternative approach is applying overflow: hidden either to the component itself or to its parent container to control any visual overflow that may be contributing to the issue.

@melvin-bot melvin-bot bot added Daily KSv2 Overdue and removed Weekly KSv2 labels Sep 13, 2024
@akinwale
Copy link
Contributor

Still looking for proposals that explain the exact root cause with an appropriate fix suggested.

@melvin-bot melvin-bot bot removed the Overdue label Sep 16, 2024
@ra-md
Copy link
Contributor

ra-md commented Sep 18, 2024

Proposal

Please re-state the problem that we are trying to solve in this issue.

Onboarding modal's corner become rectangular when hovering over the back button

What is the root cause of that problem?

The issue is likely due to a bug in Chromium-based browsers, as it only occurs there. The ScreenWrapper component escapes the boundary despite the parent element having overflow: hidden.

What changes do you think we should make in order to solve the problem?

Add a border radius (styles.br4) to the ScreenWrapper component in here and in here to help prevent it from visually escaping its parent container.

style={[styles.defaultModalContainer, shouldUseNativeStyles && styles.pt8, styles.br4]}

What alternative solutions did you explore? (Optional)

@melvin-bot melvin-bot bot added the Overdue label Sep 18, 2024
@sakluger
Copy link
Contributor

Hmm, we never got a retest by QA.

@AttockOffice85 @Bakhtiar85 @ra-md were you all able to reproduce this issue after Sept 9?

@melvin-bot melvin-bot bot removed the Overdue label Sep 19, 2024
@ra-md
Copy link
Contributor

ra-md commented Sep 19, 2024

@sakluger Yes, I can still reproduce the issue.

Copy link

melvin-bot bot commented Sep 20, 2024

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

Copy link

melvin-bot bot commented Sep 20, 2024

@akinwale @sakluger this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@melvin-bot melvin-bot bot added the Overdue label Sep 23, 2024
@akinwale
Copy link
Contributor

@ra-md Do you have a reference or an explanation for the cited bug? Is there an existing bug report filed with Chromium?

@melvin-bot melvin-bot bot removed the Overdue label Sep 23, 2024
@wildan-m
Copy link
Contributor

Issue not replicated in Chrome Version 128.0.6613.139 on macOS. The original poster was using Windows; could this be a Chrome bug specific to Windows? Which Chrome version was utilized by the tester?

@melvin-bot melvin-bot bot added the Overdue label Sep 25, 2024
@sakluger
Copy link
Contributor

@ra-md could you please clarify which operating system and version of the app you were using when you reproduced the issue? And could you please answer @akinwale's question here? Thanks!

@melvin-bot melvin-bot bot removed the Overdue label Sep 25, 2024
@ra-md
Copy link
Contributor

ra-md commented Sep 26, 2024

@akinwale Currently, I don’t have a detailed explanation for the cited bug as reproducing it has been quite difficult. I haven’t found an existing bug report filed with Chromium.

@ra-md
Copy link
Contributor

ra-md commented Sep 26, 2024

I was using Ubuntu 22.04 with Chrome version 127.0.6533.119 and Windows 11 23H2 with Chrome version 129.0.6668.59 when I reproduced the issue.

Copy link

melvin-bot bot commented Sep 27, 2024

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

@melvin-bot melvin-bot bot added the Overdue label Sep 30, 2024
@sakluger
Copy link
Contributor

I can't reproduce this one in Mac Chrome (Version 128.0.6613.138). Given the low impact on users and the difficulty in reproduction, I'm going to close this one for now so we can focus on higher impact bugs.

@github-project-automation github-project-automation bot moved this from Polish to Done in [#whatsnext] #wave-collect Sep 30, 2024
@melvin-bot melvin-bot bot removed the Overdue label Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause
Projects
No open projects
Status: Done
Development

No branches or pull requests

10 participants