Skip to content

Fix: quoted replies incorrectly render user input as part of the quote #34216

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

Merged
merged 2 commits into from
Apr 16, 2025

Conversation

khs-alt
Copy link
Contributor

@khs-alt khs-alt commented Apr 16, 2025

Problem

When clicking the "Quote reply" button, only a single line break (\n) was added after the quoted content.
As a result, the user's reply appeared to be part of the quote block when rendered in Markdown.

Solution

Updated the quote insertion logic so that it adds one more line break (\n) work like Github
This way, the cursor lands on a new line and the user's reply is no longer interpreted as part of the quote.

Before

image

After

image

References

Fixes #34177

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Apr 16, 2025
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Apr 16, 2025
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Apr 16, 2025
@lunny lunny added this to the 1.24.0 milestone Apr 16, 2025
@lunny lunny added the topic/ui Change the appearance of the Gitea UI label Apr 16, 2025
@wxiaoguang wxiaoguang merged commit dcfa42d into go-gitea:main Apr 16, 2025
26 checks passed
zjjhot added a commit to zjjhot/gitea that referenced this pull request Apr 18, 2025
* giteaofficial/main:
  Fix various misalignments and overflows (go-gitea#34227)
  Update net package (go-gitea#34228)
  Fix button alignments and remove unnecessary styles (go-gitea#34206)
  Fix project board links to related Pull Requests (go-gitea#34213)
  Fix empty repo clone panel border (go-gitea#34219)
  Fix: quoted replies incorrectly render user input as part of the quote (go-gitea#34216)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/frontend topic/ui Change the appearance of the Gitea UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The issue reply citation requires an additional line break
4 participants