Skip to content

fix(site-components): 修复 td-doc-demo 示例代码高度被置 0 - #953

Merged
uyarn merged 1 commit into
mainfrom
fix/site-components-doc-demo-show-code
Oct 26, 2026
Merged

uyarn merged 1 commit into
mainfrom
fix/site-components-doc-demo-show-code

Conversation

@liweijie0812

@liweijie0812 liweijie0812 commented Oct 26, 2026 •

Copy link
Copy Markdown
Collaborator

td-doc-demo 在 mode="open" 下示例代码无法展示,原因是 show-code
布尔属性传入空字符串时被当作 false,导致 maxHeight 恒为 0。

  • parseBoolean 将空字符串(HTML 布尔属性)解析为 true
  • td-doc-demo 的 showCode 改为经 parseBoolean 解析,且 mode="open" 时始终展开
  • td-doc-usage 的 showCode 同步修复
  • 补充 showCode 解析单元测试

🤔 这个 PR 的性质是?

  • 日常 bug 修复
  • 新特性提交
  • 文档改进
  • 演示代码改进
  • 组件样式/交互改进
  • CI/CD 改进
  • 重构
  • 代码风格优化
  • 测试用例
  • 分支合并
  • 其他

🔗 相关 Issue

💡 需求背景和解决方案

image

📝 更新日志

  • fix(site-components): 修复 td-doc-demo 示例代码高度被置 0

  • 本条 PR 不需要纳入 Changelog

☑️ 请求合并前的自查清单

⚠️ 请自检并全部勾选全部选项。⚠️

  • 文档已补充或无须补充
  • 代码演示已提供或无须提供
  • TypeScript 定义已补充或无须补充
  • Changelog 已提供或无须提供

td-doc-demo 在 mode="open" 下示例代码无法展示,原因是 show-code
布尔属性传入空字符串时被当作 false,导致 maxHeight 恒为 0。

- parseBoolean 将空字符串(HTML 布尔属性)解析为 true
- td-doc-demo 的 showCode 改为经 parseBoolean 解析,且 mode="open" 时始终展开
- td-doc-usage 的 showCode 同步修复
- 补充 showCode 解析单元测试
@pkg-pr-new

pkg-pr-new Bot commented Oct 26, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@tdesign/site-components@953
npm i https://pkg.pr.new/@tdesign/theme-generator@953

commit: aa7b1f3

@github-actions

Copy link
Copy Markdown

TDesign Component Site Preview CI Open

Sites (9)
Component Preview
tdesign-site 完成
tdesign-vue 完成
tdesign-react 完成
tdesign-vue-next 完成
@tdesign-vue-next/chat 完成
tdesign-mobile-vue 完成
tdesign-mobile-react 完成
tdesign-miniprogram 完成
tdesign-uniapp 完成

@uyarn
uyarn merged commit f2a0b10 into main Oct 26, 2026
20 checks passed
@uyarn
uyarn deleted the fix/site-components-doc-demo-show-code branch October 26, 2026 06:43
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.

3 participants