Skip to content

feat(reader): copy tables as Markdown - #41

Open
oldwinter wants to merge 1 commit into
v5from
happy-hour/copy-table-markdown
Open

feat(reader): copy tables as Markdown#41
oldwinter wants to merge 1 commit into
v5from
happy-hour/copy-table-markdown

Conversation

@oldwinter

@oldwinter oldwinter commented Aug 31, 2026

Copy link
Copy Markdown
Owner

这次为正文中的 Markdown 表格构建了一个本地化的一键复制按钮:读者可以把网页表格直接带回 Obsidian、Markdown 编辑器或 issue,而不必手动选中和重排列。它只在结构可安全转换的原生表格上出现,输出会归一化空白、转义竖线与反斜杠并补齐短行;我觉得它值得合并,因为它把一个高频但繁琐的复用动作变成了清晰、可撤销、完全本地的 40px 操作,同时不改变任何原始内容。

Validation

  • npx tsx --test quartz/components/scripts/tableMarkdown.test.ts quartz/plugins/emitters/componentResources.test.ts - 14/14 passed
  • npx tsc --noEmit - passed
  • scoped Prettier check - passed
  • git diff --check - passed
  • npx quartz build - passed (314 Markdown inputs, 1,913 emitted files)
  • npm test - 247/248 passed; the sole failure is the known baseline jsdom package absence in local-plugins/theme-switcher/test/themeSwitcherScript.test.ts
  • Real Chromium QA passed on desktop and mobile for keyboard copy, success/failure/retry feedback, focus retention, dark theme, reduced motion, no page overflow, and idempotent SPA navigation/render lifecycle

Impact

  • Adds a removable reader enhancement for supported authored tables only.
  • Excludes transcluded and structurally unsupported tables.
  • Adds English, Simplified Chinese, and Traditional Chinese labels with the existing English fallback pattern.
  • No dependency, lockfile, content, storage, analytics, network, configuration, CI, or deployment changes.

Rollback

Revert this commit. The feature writes no persistent state and requires no data or content cleanup.

Hosted status

Netlify deploy preview 6a95e87254d01300081b844d failed all four contexts. GitHub exposed zero annotations and Netlify's public API returned an unavailable summary with no messages, publish time, or deploy time. The exact-SHA local build and browser QA above pass; no deployment, authentication, configuration, or infrastructure change was made to bypass this opaque hosted-preview blocker.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@oldwinter oldwinter added the enhancement New feature or request label Aug 31, 2026
@netlify

netlify Bot commented Aug 31, 2026

Copy link
Copy Markdown

Deploy Preview for aquamarine-salamander-b041fe failed. Why did it fail? →

Name Link
🔨 Latest commit 4408ad8
🔍 Latest deploy log https://app.netlify.com/projects/aquamarine-salamander-b041fe/deploys/6a95e87254d01300081b844d

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant