Skip to content

Conversation

@Wesley-0808
Copy link
Collaborator

@Wesley-0808 Wesley-0808 commented Sep 10, 2025

🤔 这个 PR 的性质是?

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

🔗 相关 Issue

💡 需求背景和解决方案

TODO:

📝 更新日志

  • feat(Watermark): 支持 layout 参数,生成不同布局的水印

  • fix(Watermark): 修复多行图文水印时,图片配置了灰度后,导致整个画布内容也会灰度的问题

  • 本条 PR 不需要纳入 Changelog

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

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

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

@tdesign-bot
Copy link
Collaborator

tdesign-bot commented Sep 10, 2025

TDesign Component Repositories CI Test Open

Component Lint Test Build Preview
tdesign-vue 👀
tdesign-vue-next 👀
tdesign-react 👀
tdesign-web-components 👀
tdesign-mobile-vue 👀
tdesign-mobile-react 👀

@Wesley-0808 Wesley-0808 changed the title feat(watermark): support layout [WIP] feat(watermark): support layout Sep 10, 2025
@Wesley-0808 Wesley-0808 changed the title [WIP] feat(watermark): support layout feat(watermark): support layout Sep 11, 2025
@uyarn
Copy link
Collaborator

uyarn commented Sep 22, 2025

可以在PR里说明和之前版本的diff具体在哪里吗,现在完全是个新建的文件很难review差异,按理解只是针对layout增加了相关的处理逻辑

@Wesley-0808
Copy link
Collaborator Author

Wesley-0808 commented Sep 22, 2025

可以在PR里说明和之前版本的diff具体在哪里吗,现在完全是个新建的文件很难review差异,按理解只是针对layout增加了相关的处理逻辑

🤔其实也可以在原来的文件上修改 但是我觉得原来的名字不好理解(generateBase64Url不如generateWatermark)
当然,也可以直接改原文件,那晚点我改一下吧~

主要的改动就是

  1. 画布整体旋转 -> 单个水印内容旋转
  2. 改了灰度处理(之前图文多行的灰度会导致整个水印内容灰度了)
  3. backgroundSize的值由这个函数里计算返回
  4. 抽了几个复用函数

Wesley-0808 and others added 3 commits September 23, 2025 01:15
@uyarn uyarn changed the title feat(watermark): support layout feat(watermark): support layout API Sep 23, 2025
@uyarn uyarn closed this Sep 23, 2025
@uyarn uyarn reopened this Sep 23, 2025
@uyarn uyarn closed this Sep 23, 2025
@uyarn uyarn reopened this Sep 23, 2025
@uyarn uyarn merged commit e978509 into develop Sep 23, 2025
7 checks passed
@uyarn uyarn deleted the wesley/feat/watermark/enhance branch September 23, 2025 13:56
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