Skip to content

Conversation

composable-tu
Copy link
Contributor

@composable-tu composable-tu commented Aug 25, 2025

🤔 这个 PR 的性质是?

  • 日常 bug 修复
  • 其他

🔗 相关 Issue

💡 需求背景和解决方案

  1. 此仓库的 GitHub Issue Bug 反馈模板将“重现链接”定义成了单行文本框。

    如果反馈者的复现片段很短,他可以直接粘贴代码而非重新生成完整的项目代码片段,但是这种场景下使用单行文本框会存在不便之处。我提交的更改允许反馈者在首次反馈时就可以使用富文本能力粘贴代码,从而无需二次编辑。

  2. [date-time-picker] 当 auto-close 为 false 时,打开时间选择器,拨动选项后点击遮罩层,选择器的选中时间会还原回拨动前的时间 #3876

    由于 Picker 组件未传递 close-on-overlay-click="{{autoClose}}",导致 DateTimePicker 被 Picker 及更上游的 Popup 错误传递了 triggerEvent('visible-change', false) 事件,从而导致了 DateTimePicker 即使设置了 autoClosefalse,点击遮罩层也会重置选项为拨动前的 Bug

📝 更新日志

  • fix(date-time-picker、picker): 修复了 DateTimePicker 即使设置了 autoClosefalse,点击遮罩层也会重置选项为拨动前的 Bug

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

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

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

Copy link

pkg-pr-new bot commented Aug 25, 2025

Open in StackBlitz

npm i https://pkg.pr.new/tdesign-miniprogram@3874

commit: 826c6aa

@composable-tu composable-tu changed the title feat(notice-bar): 在 Demo 中添加非通栏样式公告栏示例 修复了 DateTimePicker 即使设置了 autoClose 为 false,点击遮罩层也会重置选项为拨动前的 Bug;在 Demo 中添加非通栏样式公告栏示例 Aug 26, 2025
@anlyyao
Copy link
Collaborator

anlyyao commented Aug 26, 2025

@composable-tu noticebar 的示例另拆一个 pr,单独处理。“非通栏样式” 示例不是我们预期展示的内通,可按照视觉稿补充一个“卡片顶部”示例。

视觉稿地址:https://codesign.qq.com/s/589886479758640
访问密码:IAIL

预期示例截图:

截屏2025-08-26 20 11 47

@anlyyao
Copy link
Collaborator

anlyyao commented Aug 26, 2025

@composable-tu 感谢近期对 TDesign 小程序端组件库的关注和贡献,方便的话可以从官网底部扫码入用户群,在用户群联系一下我,提供下收件信息,可为您安排一份 TDesign 周边~~(我在用户群的昵称和 github 同名)

@composable-tu
Copy link
Contributor Author

@composable-tu noticebar 的示例另拆一个 pr,单独处理。“非通栏样式” 示例不是我们预期展示的内通,可按照视觉稿补充一个“卡片顶部”示例。

截屏2025-08-26 20 11 47

您好,视觉稿中卡片的背景是深灰色,可是您们的实机 Demo 的卡片背景灰色是浅灰色,和 TDesign 品牌色 Light 放在一起会导致视觉无障碍问题,比较不容易分辨 notice bar 的边界,如下图:

Screenshot_20250826_204242_com_tencent_mm_AppBrandUI01.jpg

@composable-tu
Copy link
Contributor Author

composable-tu commented Aug 26, 2025

此外我注意到设计稿的样式是带 notice 的圆角矩形背景板盖上圆角矩形 content 正文的样式,和您给出的设计图细节不符。这样的样式应该需要修改 card 源码使其集成 notice bar 功能吧。

Screenshot_20250826_205419_com_chrome_dev_ChromeTabbedActivity.jpg

综上看来 notice bar 卡片样式的实现细节应该是需要经过讨论后定版方案的。我稍后将 Demo 相关代码回滚到提交前状态。

@composable-tu composable-tu changed the title 修复了 DateTimePicker 即使设置了 autoClose 为 false,点击遮罩层也会重置选项为拨动前的 Bug;在 Demo 中添加非通栏样式公告栏示例 fix(DateTimePicker): 修复了 DateTimePicker 即使设置了 autoClose 为 false,点击遮罩层也会重置选项为拨动前的 Bug Aug 26, 2025
@composable-tu
Copy link
Contributor Author

composable-tu commented Aug 26, 2025

https://github.com/Tencent/tdesign-miniprogram/actions/runs/17239209785/job/48911488487?pr=3874

此 Action 出现了

npm error 403 403 Forbidden - GET https://registry.npmjs.org/performance-now/-/performance-now-2.1.0.tgz

问题,能帮忙重新运行一次吗?

已解决,感谢

@anlyyao
Copy link
Collaborator

anlyyao commented Aug 28, 2025

@composable-tu sorry、没顾上消息。我的截图是来自 figma,确实是 figma 和 codesign 两边设计稿没有同步。 我们以 figma 为准,NoticeBar 组件完整的视觉稿如下:
NoticeBar 公告栏 移动端展示

@anlyyao
Copy link
Collaborator

anlyyao commented Aug 28, 2025

@composable-tu 仅增加 NoticaBar 示例,不集成到组件。 背景色问题,按设计稿执行,maybe 设计同学有自己的考量 😂

@composable-tu
Copy link
Contributor Author

composable-tu commented Aug 28, 2025

@composable-tu 仅增加 NoticaBar 示例,不集成到组件。 背景色问题,按设计稿执行,maybe 设计同学有自己的考量 😂

可是业界应该没有哪个组件库最佳实践是单一平面内深色背景上放浅色卡片的,只有像对话框、弹出层这样的需要用户聚焦的操作才会加深背景以让用户集中直觉到弹窗上。

在用户注意力越来越有限的当下,用户界面中强对比度的色彩使用应该非常谨慎。

@novlan1
Copy link
Collaborator

novlan1 commented Aug 29, 2025

hello,有空先解决下冲突,以及后面可以不用develop分支,切feature分支

@RADWIMPS426
Copy link
Collaborator

@composable-tu 仅增加 NoticaBar 示例,不集成到组件。 背景色问题,按设计稿执行,maybe 设计同学有自己的考量 😂

可是业界应该没有哪个组件库最佳实践是单一平面内深色背景上放浅色卡片的,只有像对话框、弹出层这样的需要用户聚焦的操作才会加深背景以让用户集中直觉到弹窗上。

在用户注意力越来越有限的当下,用户界面中强对比度的色彩使用应该非常谨慎。

这里有些误解,此处的示例为交互环节中的静态展示,旨在表现组件与卡片组合的样式,实际使用中,仍然是需要通过交互触发后弹出。

@composable-tu
Copy link
Contributor Author

composable-tu commented Sep 29, 2025

这里有些误解,此处的示例为交互环节中的静态展示,旨在表现组件与卡片组合的样式,实际使用中,仍然是需要通过交互触发后弹出。

好的,也就是说卡片样式的提示条在设计中是为了在 Popup、Dialog 这些组件场景中使用吗?

@composable-tu
Copy link
Contributor Author

composable-tu commented Sep 29, 2025

@anlyyao @novlan1 抱歉 Ping🙏,可以先将该 PR 合并吗?我觉得关于 Notice Bar 的问题可能需要优化一下设计稿的表达,以避免设计同学提到的误解情况。

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