Skip to content

Conversation

@betavs
Copy link
Contributor

@betavs betavs commented Dec 19, 2025

🤔 这个 PR 的性质是?

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

🔗 相关 Issue

#6276

💡 需求背景和解决方案

📝 更新日志

  • 本条 PR 不需要纳入 Changelog

tdesign-vue-next

  • fix(ImageViewer): 修复无法使用插槽自定义预览标题的问题

@tdesign-vue-next/chat

@tdesign-vue-next/auto-import-resolver

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

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

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

@tdesign-bot
Copy link
Collaborator

tdesign-bot commented Dec 19, 2025

TDesign Component Site Preview Open

Component Preview
tdesign-vue-next 完成
@tdesign-vue-next/chat 完成

@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 19, 2025

  • tdesign-vue-next-demo

    npm i https://pkg.pr.new/Tencent/tdesign-vue-next/@tdesign-vue-next/auto-import-resolver@6277
    
    npm i https://pkg.pr.new/Tencent/tdesign-vue-next@6277
    
    npm i https://pkg.pr.new/Tencent/tdesign-vue-next/@tdesign-vue-next/chat@6277
    

commit: d496e87

@Wesley-0808
Copy link
Collaborator

Wesley-0808 commented Dec 20, 2025

@betavs 确认一下此处只传props.title会不会有问题:


image

@betavs
Copy link
Contributor Author

betavs commented Dec 20, 2025

@Wesley-0808 目前调整只针对模态预览情况下的修改并不影响非模态预览

@Wesley-0808
Copy link
Collaborator

Wesley-0808 commented Dec 20, 2025

@Wesley-0808 目前调整只针对模态预览情况下的修改并不影响非模态预览

emm,我的意思是非模态可能也有这个bug
我刚看了,确实也有,demo:https://stackblitz.com/edit/vitejs-vite-pgzsyman?file=src%2FApp.vue

@betavs
Copy link
Contributor Author

betavs commented Dec 20, 2025

@Wesley-0808modeless 情况下 .t-image-viewer__dialog .t-dialog__header 样式中固定了高度 height: 40px;
也许设计之初就是基于该情况下并不支持 title 插槽呢 🤔

@uyarn
Copy link
Collaborator

uyarn commented Dec 21, 2025

感觉也传一下比较好,虽然是自定义的且有个固定高度,但是文档上也没写什么情况支持什么类型;这个就交给用户使用决定吧

@betavs
Copy link
Contributor Author

betavs commented Dec 21, 2025

感觉也传一下比较好,虽然是自定义的且有个固定高度,但是文档上也没写什么情况支持什么类型;这个就交给用户使用决定吧

done

@uyarn uyarn merged commit 6015cb6 into Tencent:develop Dec 21, 2025
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.

4 participants