Skip to content

Remove "table" from default inlineRendering - #201

Merged
cyanzhong merged 1 commit into
mainfrom
develop
Sep 27, 2026
Merged

cyanzhong merged 1 commit into
mainfrom
develop

Conversation

@cyanzhong

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

A critical hidden-syntax table test still relies on the old implicit default.

Review effort: Lite
Findings: 1 High severity

Open (1)
What changed in this PR

Removes table from default inline rendering while updating documentation, tests, and distribution bundles.

Changes:

  • Updates source defaults and settings tests.
  • Documents the new defaults.
  • Regenerates full and lite bundles.
  • A hidden-syntax table test still requires explicit table-enabled settings.
File Summary
tests/​settings.test.ts Updates default rendering expectations.
src/​support/​settings.ts Removes table from fallback defaults.
README.md Documents the new defaults.
dist/​markedit-preview.js Updates the full distribution bundle.
dist/​lite/​markedit-preview.js Updates the lite distribution bundle.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/support/settings.ts
@cyanzhong
cyanzhong merged commit 6acc418 into main Sep 27, 2026
1 check passed
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