Skip to content

mobile: fix share append content merging onto same line#10042

Open
keyut-shah wants to merge 1 commit into
streetwriters:masterfrom
keyut-shah:fix/mobile-share-append-newline
Open

mobile: fix share append content merging onto same line#10042
keyut-shah wants to merge 1 commit into
streetwriters:masterfrom
keyut-shah:fix/mobile-share-append-newline

Conversation

@keyut-shah

@keyut-shah keyut-shah commented Jul 1, 2026

Copy link
Copy Markdown

Replace the bare
separator with an explicit


paragraph. The bare
was parsed into an implicit empty paragraph that inherited the live 'double spaced lines' setting; when that setting was off, the placeholder collapsed to zero height and appended content ran together on one line.

Fixes #9207

Description

Type of Change

  • Bug fix
  • Feature

Visuals

  • Attached relevant screenshots / screen recording / GIF
  • N/A (not a feature or no UI changes)

Testing

  • Ran all E2E tests
  • Ran all integration tests
  • Added/updated tests for this change (if needed)
  • N/A (tests not needed — explanation provided below)

If tests were not added, explain why

Platform

  • Web
  • Mobile
  • Desktop

Sign-off

  • QA passed
  • UI/UX passed

Replace the bare <br/> separator with an explicit
<p data-spacing=single><br></p> paragraph. The bare <br/> was parsed
into an implicit empty paragraph that inherited the live
'double spaced lines' setting; when that setting was off, the
placeholder collapsed to zero height and appended content ran
together on one line.

Fixes streetwriters#9207

Signed-off-by: keyut-shah <keyutshah1@gmail.com>
data: (rawContent?.data || "") + "<br/>" + noteContent.current,
data:
(rawContent?.data || "") +
`<p data-spacing="single"><br></p>` +

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's add data-spacing="single" only when double spaced lines are disabled by user?

@github-actions

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

On android, appending to a note through the sharing function should start with a new line, not space

2 participants