Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

improve menuitem route by vue3.4'defineModel #701

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

w6et
Copy link

@w6et w6et commented Mar 30, 2024

🤔 这个 PR 的性质是?

  • 新特性提交
    涉及到同步修改的,貌似还有个Header.vue,暂时没动,只是提供一种新的方式供参考

🔗 相关 Issue

💡 需求背景和解决方案

📝 更新日志

  • fix(组件名称): 处理问题或特性描述 ...

  • 本条 PR 不需要纳入 Changelog

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

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

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

@timi137137
Copy link
Collaborator

image
根据官方文档,该宏目的是为了更便利的进行双向绑定,具体getActive有没有必要双向绑定,个人觉得没有需要。
@uyarn 你看怎么说,我这边本地测试功能上没区别

@uyarn
Copy link
Collaborator

uyarn commented Mar 31, 2024

image 根据官方文档,该宏目的是为了更便利的进行双向绑定,具体getActive有没有必要双向绑定,个人觉得没有需要。 @uyarn 你看怎么说,我这边本地测试功能上没区别

只是另一种写法吗?

Copy link

失败

@timi137137
Copy link
Collaborator

image 根据官方文档,该宏目的是为了更便利的进行双向绑定,具体getActive有没有必要双向绑定,个人觉得没有需要。 @uyarn 你看怎么说,我这边本地测试功能上没区别

只是另一种写法吗?

是的,原理是设定两个ref,一个负责监控另一个负责同步

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.

None yet

3 participants