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

perf: 优化 dilog 下组件 #66

Merged
merged 1 commit into from
Apr 29, 2024
Merged

perf: 优化 dilog 下组件 #66

merged 1 commit into from
Apr 29, 2024

Conversation

Chanzhaoyu
Copy link
Contributor

@Chanzhaoyu Chanzhaoyu commented Apr 29, 2024

  • dialogs 文件夹下组件优化为最新 vue3 + setup + typescript 写法
  • 重命名事件函数名称

其它文件夹下组件正在调整

Copy link

vercel bot commented Apr 29, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
douyin ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 29, 2024 6:03am

Copy link

netlify bot commented Apr 29, 2024

Deploy Preview for douyins ready!

Name Link
🔨 Latest commit 6a6554a
🔍 Latest deploy log https://app.netlify.com/sites/douyins/deploys/662f37f2ebc7170008dffae9
😎 Deploy Preview https://deploy-preview-66--douyins.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@zyronon zyronon merged commit 9da5f78 into zyronon:master Apr 29, 2024
6 checks passed
@zyronon
Copy link
Owner

zyronon commented Apr 29, 2024

谢谢,已合并!我现在已经把 jQuery 去掉了,装了 unplugin-vue-macros 这个依赖,可以使用 $ref 啦

@Chanzhaoyu
Copy link
Contributor Author

谢谢,已合并!我现在已经把 jQuery 去掉了,装了 unplugin-vue-macros 这个依赖,可以使用 $ref 啦

老实说我不喜欢用 macros 😂

@zyronon
Copy link
Owner

zyronon commented Apr 29, 2024

谢谢,已合并!我现在已经把 jQuery 去掉了,装了 unplugin-vue-macros 这个依赖,可以使用 $ref 啦

老实说我不喜欢用 macros 😂

好吧,各有所爱,不强求😂

@zyronon
Copy link
Owner

zyronon commented Apr 29, 2024

大哥你没测试嘛 - -,master我回滚了,你的代码现在合到dev上了

@Chanzhaoyu
Copy link
Contributor Author

大哥你没测试嘛 - -,master我回滚了,你的代码现在合到dev上了

我本地开发环境看是正常,我打包后试试

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.

2 participants