Skip to content

Conversation

@LiYanan2004
Copy link
Owner

@LiYanan2004 LiYanan2004 commented Oct 19, 2025

Description

2.5.0 introduces an issue that would drop all text formatting -- This PR fixes that issue.

Comparison

Before & After

before this PR after the PR

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

https://github.com/LiYanan2004/MarkdownView/blob/495ec347770254dfd78dc29dc62f1cd8995a83e4/Sources/MarkdownView/Renderers/Node Representations/InlineMathOrText.swift#L55-L56
P1 Badge Fallback inline-math path uses removed Text initializer

When LaTeXSwiftUI is unavailable, makeBody still returns MarkdownNodeView(Text(text)). The refactor replaced MarkdownNodeView’s Text initializers with AttributedString/String versions, so this call no longer matches any initializer and the file fails to compile in configurations that omit the LaTeX dependencies (e.g. the default package without optional modules). The fallback branch should construct the node with the new string-based initializer instead.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@LiYanan2004
Copy link
Owner Author

@codex review

@chatgpt-codex-connector
Copy link

Codex Review: Didn't find any major issues. What shall we delve into next?

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@LiYanan2004 LiYanan2004 merged commit 54ac5fd into main Oct 19, 2025
5 checks passed
@LiYanan2004 LiYanan2004 deleted the fix/text-formatting branch October 19, 2025 16:58
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.

2 participants