Skip to content

Use safe blockquote prefix on legacy Windows#4094

Open
lwhelton wants to merge 1 commit intoTextualize:masterfrom
lwhelton:fix-markdown-blockquote-windows-3281
Open

Use safe blockquote prefix on legacy Windows#4094
lwhelton wants to merge 1 commit intoTextualize:masterfrom
lwhelton:fix-markdown-blockquote-windows-3281

Conversation

@lwhelton
Copy link
Copy Markdown

Type of changes

  • [X ] Bug fix
  • New feature
  • Documentation / docstrings
  • [X ] Tests
  • Other

AI?

  • AI was used to generate this PR

AI generated PRs may be accepted, but only if @willmcgugan has responded on an issue or discussion.

Checklist

  • [X ] I've run the latest black with default args on new code.
  • [X ] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate (see note about typos above).
  • [X ] I've added tests for new code.
  • [ X] I accept that @willmcgugan may be pedantic in the code review.

Description

Please describe your changes here.
Fixes Markdown blockquote rendering on legacy Windows / restricted encodings by falling back from ▌ to | when the output environment is not safe for the Unicode block character. Normal environments still use the original blockquote marker. A regression test was added to verify the safe fallback behavior.

Important: Link to an issue or discussion regarding these changes.

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.

1 participant