[HOLD for 42143] [$2000] [LOW] IOS - Keypad appears for a moment in details page on saving Private notes

[lanitochka17]

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:
Reproducible in staging?:
Reproducible in production?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers): [email protected]
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:

Action Performed:

Precondition: User has not saved Private notes

1. Open a workspace #announce room
2. Tap header link to go details page
3. Select Private notes
4. Enter notes & tap on Save button

Expected Result:

Keypad should not appear in details or settings page while saving Private notes

Actual Result:

Keypad appears for a moment in details/settings page while saving Private notes

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos

Add any screenshot/video evidence

Bug6366487_1707016784461.RPReplay_Final1706924534.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01d02b0565c21176af
• Upwork Job ID: 1754200247556661248
• Last Price Increase: 2024-03-25

[melvin-bot]

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

[melvin-bot]

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

[melvin-bot]

Triggered auto assignment to @bfitzexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

[lanitochka17]

We think that this bug might be related to #vip-vsp
CC @quinthar

[mkhutornyi]

Proposal

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

iOS - Keypad appears for a moment in details page on saving Private notes

What is the root cause of that problem?

App/src/pages/PrivateNotes/PrivateNotesEditPage.tsx

Lines 158 to 164 in 3ab4e6e

	ref={(el: AnimatedTextInputRef) => {
	if (!el) {
	return;
	}
	privateNotesInput.current = el;
	updateMultilineInputRange(privateNotesInput.current);
	}}

This happens because ref callback is called every time user inputs value, which triggers updateMultilineInputRange

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

As this is supposed to call only once when page is first opened, add more condition in early return here:

     if (!el || privateNotesInput.current) { 
         return; 
     } 

[getusha]

Looks like a very minor issue, @bfitzexpensify i think we can close it, wdyt?

[bfitzexpensify]

@getusha Agreed that it's very minor, but assuming there is a clear and easy solution, we should address this. What do you think of @mkhutornyi's proposal?

[getusha]

Thanks! will test @mkhutornyi's proposal.

[getusha]

@mkhutornyi are you able to reproduce this issue?

Screen.Recording.2024-02-07.at.10.54.07.PM.mov

[mkhutornyi]

@getusha yes, still reproducible on latest main

bug.mov

[getusha]

Still not able to repro, what's the your iOS version?

[mkhutornyi]

Still not able to repro, what's the your iOS version?

iOS 16.4
iPhone 14 Plus Simulator

[getusha]

asked for other c+
https://expensify.slack.com/archives/C02NK2DQWUX/p1707408881410369

[situchan]

@getusha this is reproducible not only on this page but also other pages which have multiline input

i.e. on room description page:

Screen.Recording.2024-02-08.at.10.19.21.PM.mov

[situchan]

The proposed solution doesn't fix issue completely

[situchan]

Another reproducible case: (swipe left to go back)

This even doesn't require text input change.

Screen.Recording.2024-02-08.at.10.21.44.PM.mov

[situchan]

Please let me know if it's still not reproducible for you. If still, I can take over.

[getusha]

@situchan I missed your comment there! @mananjadhav offered to take over.

I'm still unable to reproduce

Screen.Recording.2024-02-08.at.7.32.38.PM.mov

[mananjadhav]

I am able to reproduce. Will check the proposal in a while.

[bfitzexpensify]

Still held

[bfitzexpensify]

Same update, still held

[bfitzexpensify]

Remains held

[bfitzexpensify]

Still held

[bfitzexpensify]

Still held

[bfitzexpensify]

Still held on #42143.

[bfitzexpensify]

Still held

[bfitzexpensify]

Still held. Moving this to monthly for the moment

[bfitzexpensify]

Remains held

[bfitzexpensify]

Still held