Skip to content

Fix CRs for paragraphs with empty first line#5340

Merged
bastianjoel merged 47 commits intoOpenSlides:mainfrom
bastianjoel:5165-fix-empty-br
Apr 27, 2026
Merged

Fix CRs for paragraphs with empty first line#5340
bastianjoel merged 47 commits intoOpenSlides:mainfrom
bastianjoel:5165-fix-empty-br

Conversation

@bastianjoel
Copy link
Copy Markdown
Member

@bastianjoel bastianjoel commented Aug 22, 2025

resolves #5164
resolves #5165
resolves #5355
reoslves potentionally OpenSlides/OpenSlides#6927

Needs:
autoupdate in main
backend in main

@bastianjoel bastianjoel changed the title Fix CRs for paragraphs with empty br Fix CRs for paragraphs with empty first line Aug 26, 2025
@Elblinator Elblinator added the waiting Waiting for some other PR/feature; more details in comments label Aug 27, 2025
@Elblinator
Copy link
Copy Markdown
Member

Do not merge yet

@Elblinator
Copy link
Copy Markdown
Member

Please reassign me if approved

@Elblinator Elblinator removed their assignment Aug 27, 2025
@bastianjoel bastianjoel marked this pull request as draft September 30, 2025 13:46
@bastianjoel bastianjoel force-pushed the 5165-fix-empty-br branch 4 times, most recently from ed0db4d to 22867e2 Compare September 30, 2025 16:05
@luisa-beerboom luisa-beerboom removed their assignment Mar 26, 2026
@m-schieder m-schieder force-pushed the 5165-fix-empty-br branch 2 times, most recently from 70587a7 to 29ed38d Compare April 1, 2026 12:36
Copy link
Copy Markdown

@MSoeb MSoeb left a comment

Choose a reason for hiding this comment

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

Existing motions that contain blank lines without line numbers will be assigned line numbers when the PR is applied. This results in incorrect line numbers in existing motions.

This should not happen.

I’ve discussed this with Bastian and Raimund.

@MSoeb MSoeb assigned bastianjoel and unassigned MSoeb Apr 23, 2026
@bastianjoel bastianjoel requested a review from MSoeb April 23, 2026 16:27
@bastianjoel bastianjoel assigned MSoeb and unassigned bastianjoel Apr 23, 2026
@Elblinator Elblinator removed their request for review April 24, 2026 06:48
@Elblinator Elblinator removed their assignment Apr 24, 2026
Copy link
Copy Markdown

@MSoeb MSoeb left a comment

Choose a reason for hiding this comment

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

Missing line value is fixed. No new line numbering will be added.

Info: The height gap between two paragraphs in this case is equal to the one which will be added in new motions by hitting enter two times. If this will be relevant, we have the option to change it at a later point. For now the problems are fixed and the line numbers are consinstent. So we can go on.

@MSoeb MSoeb assigned bastianjoel and unassigned MSoeb Apr 27, 2026
@bastianjoel bastianjoel merged commit 9642535 into OpenSlides:main Apr 27, 2026
5 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Motion diff compatibility improvement Fix CRs for paragraphs with empty br Fix motions with empty br at the beginning of p

6 participants