Skip to content

fix: keep PPTX soft line breaks when formatting links - #13098

Closed
Vamsi-2581 wants to merge 1 commit into
deepset-ai:mainfrom
Vamsi-2581:fix/pptx-soft-line-breaks
Closed

Vamsi-2581 wants to merge 1 commit into
deepset-ai:mainfrom
Vamsi-2581:fix/pptx-soft-line-breaks

Conversation

@Vamsi-2581

Copy link
Copy Markdown

PPTXToDocument joined paragraph runs directly for markdown and plain links, so a Shift+Enter break between runs disappeared. Walk the paragraph's content, including line-break elements, and keep the same vertical tab the default link format already returns.

Related Issues

  • fixes #issue-number

Proposed Changes:

How did you test it?

Notes for the reviewer

Checklist

  • I have read the contributors guidelines and the code of conduct.
  • I have updated the related issue with new insights and changes.
  • I have added unit tests and updated the docstrings.
  • I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test: and added ! in case the PR includes breaking changes.
  • I have documented my code.
  • I have added a release note file, following the contributors guidelines.
  • I have run pre-commit hooks and fixed any issue.

PPTXToDocument joined paragraph runs directly for markdown and plain
links, so a Shift+Enter break between runs disappeared. Walk the
paragraph's content, including line-break elements, and keep the same
vertical tab the default link format already returns.
@Vamsi-2581
Vamsi-2581 requested a review from a team as a code owner October 3, 2026 14:49
@Vamsi-2581
Vamsi-2581 requested review from bogdankostic and removed request for a team October 3, 2026 14:49
@vercel

vercel Bot commented Oct 3, 2026

Copy link
Copy Markdown
Contributor

@Vamsi-2581 is attempting to deploy a commit to the deepset Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

github-actions Bot commented Oct 3, 2026

Copy link
Copy Markdown
Contributor

Hi @Vamsi-2581, thanks for your interest in contributing to Haystack! 🙏

⛔ First-time contributors can have at most 1 open pull request in this repository until it has been approved, so this PR was closed automatically. Your open pull request #13091 is unaffected. Once it has been approved by a maintainer, you are welcome to open more PRs. Feel free to reopen this one at that point.

See the contributing guidelines for details.

This is an automated message to help us keep the review queue healthy.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant