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

[$250] iOS Split - Submit button appears then disappears when keyboard is open #52828

Closed
4 of 8 tasks
IuliiaHerets opened this issue Nov 20, 2024 · 5 comments
Closed
4 of 8 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. 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 Reviewing Has a PR in review Weekly KSv2

Comments

@IuliiaHerets
Copy link

IuliiaHerets commented Nov 20, 2024

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


Version Number: 9.0.64-4
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team

Action Performed:

  1. Launch ND or hybrid app.
  2. Open chat.
  3. Tap + > Split expense > Manual.
  4. Enter amount > Next.
  5. Tap on the split input.

Expected Result:

Submit button will not appear and disappear when keyboard is open.

Actual Result:

Submit button appears then disappears when keyboard is open.

Workaround:

Unknown

Platforms:

  • Android: Standalone
  • Android: HybridApp
  • Android: mWeb Chrome
  • iOS: Standalone
  • iOS: HybridApp
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Bug6670779_1732114874878.ScreenRecording_11-20-2024_22-57-28_1.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021860016281597788500
  • Upwork Job ID: 1860016281597788500
  • Last Price Increase: 2024-11-22
Issue OwnerCurrent Issue Owner: @mollfpr
@IuliiaHerets IuliiaHerets added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Nov 20, 2024
Copy link

melvin-bot bot commented Nov 20, 2024

Triggered auto assignment to @isabelastisser (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.

@QichenZhu
Copy link
Contributor

QichenZhu commented Nov 22, 2024

Edited by proposal-police: This proposal was edited at 2024-11-22 04:05:07 UTC.

Proposal

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

The submit button appears and disappears when the keyboard opens on the money request confirmation page.

What is the root cause of that problem?

The submit button is hidden on purpose after the keyboard opens to resolve #50663.

const footerContent = useMemo(() => {
if (isReadOnly || isKeyboardShown || isWindowHeightReducedByKeyboard) {
return;

The problem is that the keyboard opens with an animation, but the button is hidden only after the keyboard is fully open.

const keyboardDidShowListener = Keyboard.addListener('keyboardDidShow', (e) => {

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

Replace keyboardDidShow with keyboardWillShow in the code mentioned above. Since this change affects many places in the app, we need to test it carefully.

const keyboardWillShowListener = Keyboard.addListener('keyboardWillShow', (e) => { 
return () => {
    keyboardWillShowListener.remove();

What alternative solutions did you explore? (Optional)

Introduce a new state that listens to keyboardWillShow and use it specifically in the money request confirmation page. This approach might be safer than the main solution.

@isabelastisser isabelastisser added 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 labels Nov 22, 2024
@melvin-bot melvin-bot bot added the Overdue label Nov 22, 2024
@melvin-bot melvin-bot bot changed the title iOS Split - Submit button appears then disappears when keyboard is open [$250] iOS Split - Submit button appears then disappears when keyboard is open Nov 22, 2024
Copy link

melvin-bot bot commented Nov 22, 2024

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

Copy link

melvin-bot bot commented Nov 22, 2024

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

@s77rt
Copy link
Contributor

s77rt commented Nov 24, 2024

The offending PR has been reverted. Let's close this

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. 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 Reviewing Has a PR in review Weekly KSv2
Projects
Development

No branches or pull requests

6 participants