taro icon indicating copy to clipboard operation
taro copied to clipboard

feat(taro-h5): 路由切换时保留toast

Open KkZsc opened this issue 3 years ago • 4 comments

这个 PR 做了什么? (简要描述所做更改) 目前H5端路由切换时会主动关闭toast,相关的关闭toast代码是在修复modal在跳转后不隐藏的问题时添加的。 但是个人认为跨页面的显示toast是很常见使用率很高的场景,且toast不同于loading和modal,是有自动关闭逻辑的,不应随着路由变化关闭toast。

这个 PR 是什么类型? (至少选择一个)

  • [ ] 错误修复(Bugfix) issue: fix #
  • [x] 新功能(Feature)
  • [ ] 代码重构(Refactor)
  • [ ] TypeScript 类型定义修改(Typings)
  • [ ] 文档修改(Docs)
  • [ ] 代码风格更新(Code style update)
  • [ ] 其他,请描述(Other, please describe):

这个 PR 涉及以下平台:

  • [ ] 所有小程序
  • [ ] 微信小程序
  • [ ] 支付宝小程序
  • [ ] 百度小程序
  • [ ] 字节跳动小程序
  • [ ] QQ 轻应用
  • [ ] 京东小程序
  • [ ] 快应用平台(QuickApp)
  • [x] Web 平台(H5)
  • [ ] 移动端(React-Native)

KkZsc avatar Sep 17 '22 04:09 KkZsc

欢迎提交 PR~ Taro 非常感谢您对开源事业做出的贡献!🌷🌷🌷

一般 PR 会在一到两周内进行 review,成功合入后会随下一个版本进行发布。

Review 需要耗费大量时间,所以请遵循以下规范,协助我们提高 review 效率🙏🙏🙏

  1. 详细介绍 PR 的背景(非常重要,例如解决了什么问题,该问题如何复现等)
  2. 确保 CI 顺利运行。
  3. 最好能提供对应的测试用例。

为了更好地进行沟通,请加入 Taro 开发者微信群:

taro-bot2[bot] avatar Sep 17 '22 04:09 taro-bot2[bot]

欢迎提交 PR~ Taro 非常感谢您对开源事业做出的贡献!🌷🌷🌷

一般 PR 会在一到两周内进行 review,成功合入后会随下一个版本进行发布。

Review 需要耗费大量时间,所以请遵循以下规范,协助我们提高 review 效率🙏🙏🙏

  1. 详细介绍 PR 的背景(非常重要,例如解决了什么问题,该问题如何复现等)
  2. 确保 CI 顺利运行。
  3. 最好能提供对应的测试用例。

为了更好地进行沟通,请加入 Taro 开发者微信群:

taro-bot2[bot] avatar Sep 17 '22 04:09 taro-bot2[bot]

关联 issue #11168

ZakaryCode avatar Sep 19 '22 05:09 ZakaryCode

Modal函数内已关联调用时的页面path,而且__taroRouterChange事件触发的太早在其回调里不能正确的取到router change之后的页面path。因此在taro-router里路由切换之后增加了__afterTaroRouterChange事件的trigger。

KkZsc avatar Sep 21 '22 10:09 KkZsc

路由切换时保留toastModal可以仍然关闭吧?

yoyo837 avatar Nov 11 '22 06:11 yoyo837

是的,这个pr也关联修改了#11168

KkZsc avatar Nov 11 '22 10:11 KkZsc

冲突了, 来解决一下

yoyo837 avatar Dec 02 '22 14:12 yoyo837

push -f

yoyo837 avatar Dec 05 '22 03:12 yoyo837

可以增加一个和 next 分支的 merge 节点,差异文件太多了没法儿看 😂

ZakaryCode avatar Dec 05 '22 03:12 ZakaryCode

请问这要怎么操作呢,上一次提交我已经rebase nextpush --force了😂

KkZsc avatar Dec 05 '22 04:12 KkZsc

请问这要怎么操作呢,上一次提交我已经rebase nextpush --force了😂

merge next 然后提交

ZakaryCode avatar Dec 05 '22 05:12 ZakaryCode

现在merge next已经没有更改了 image

KkZsc avatar Dec 05 '22 06:12 KkZsc

现在merge next已经没有更改了 image

参考这里

ZakaryCode avatar Dec 06 '22 06:12 ZakaryCode