[Hold for PR #535] [$250] iOS - Wrong cursor placement on the next line when the previous line has a block quote

[IuliiaHerets]

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.1.4-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Yes, reproducible on both
If this was caught during regression testing, add the test name, ID and link from TestRail: Exp
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team
Device used: iPhone 15 Pro Max / iOS 18.3
App Component: Chat Report View

Action Performed:

1. Launch ND or hybrid app.
2. Go to chat.
3. Type the following text

hey

4. Enter the next line.

Expected Result:

The cursor on the next line will be positioned at the beginning of the line.

Actual Result:

The cursor on the next line is positioned after a few spaces at the beginning of the line.

Workaround:

Unknown

Platforms:

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

Screenshots/Videos

Bug6750747_1740219132110.ScreenRecording_02-22-2025_18-07-50_1.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~021894347174956898347
• Upwork Job ID: 1894347174956898347
• Last Price Increase: 2025-03-04

[melvin-bot]

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

[NicMendonca]

asking here: https://expensify.slack.com/archives/C06ML6X0W9L/p1740446839396769

[melvin-bot]

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

[melvin-bot]

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

[gurus00]

@getusha I want to work on this issue

[Swor71]

Hi, I'm Marcin from Callstack - expert contributor group. I’d like to work on this issue.

[melvin-bot]

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

[melvin-bot]

@getusha 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

[getusha]

@NicMendonca we can assign @Swor71 from Callstack

[Swor71]

update: digging into the way the live markdown preview parses quote blocks and spacing

[melvin-bot]

@NicMendonca @getusha 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]

@getusha Eep! 4 days overdue now. Issues have feelings too...

[Swor71]

update: this issue should be fixed as part of Expensify/react-native-live-markdown#520 and more precisely this commit however this PR has been open for a while now. I'll check if this fix can be applied sooner.

I have also noticed that the behaviour is different when \n is already present within markdown:

cursor.placement.issue.mov

[Swor71]

update: as per the comment from the pr:

After adding a newline at the end of the blockquote, the typing attributes in the new line still contain NSParagraphStyle with non-zero firstLineHeadIndent and headIntent. This causes the cursor to be shifted to the right instead of being located at the beginning of the line.

I'm currently working on porting and adjusting this fix.

[melvin-bot]

@NicMendonca @Swor71 @getusha this issue is now 4 weeks old, please consider:

• Finding a contributor to fix the bug
• Closing the issue if BZ has been unable to add the issue to a VIP or Wave project
• If you have any questions, don't hesitate to start a discussion in #expensify-open-source

Thanks!

[melvin-bot]

@getusha Eep! 4 days overdue now. Issues have feelings too...

[NicMendonca]

Updated title

[NicMendonca]

Holding for this: Expensify/react-native-live-markdown#535

[getusha]

Not overdue

[tomekzaw]

Update: still working on this PR since it fixes vertical alignment but breaks other things.

[melvin-bot]

@NicMendonca, @Swor71, @getusha Huh... This is 4 days overdue. Who can take care of this?

[NicMendonca]

Still holding for this: Expensify/react-native-live-markdown#535

[NicMendonca]

Setting this to weekly - does that make more sense @Swor71 @getusha?

[NicMendonca]

Still holding

[NicMendonca]

@Swor71 @getusha -- are we still holding?

[NicMendonca]

bump @Swor71 @getusha - I don't see much movement on that PR. Are we still holding for that?

[NicMendonca]

Okay its been quite a long time. I am going to close and we can revisit this at a later time.