-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
Link: add link component #15052
Link: add link component #15052
Conversation
Deploy preview for element ready! Built with commit 1b5c236 |
原生属性我觉得还是应该传递一下。 click 事件要不要 preventDefault ? isHover 纯 css 就可以。 如果作为统一风格 / 替代原生 |
test/unit/specs/link.spec.js
Outdated
}, true); | ||
vm.$el.click(); | ||
|
||
setTimeout(_ => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
使用 async 函数。
packages/link/src/main.vue
Outdated
<template v-if="$slots.icon"><slot v-if="$slots.icon" name="icon"></slot></template> | ||
|
||
<!-- underline --> | ||
<div |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
prefer ::after
review 都已经修改, functional component 讨论下来暂时先不做 看看用户反馈 |
预览地址 https://deploy-preview-15052--element.netlify.com/#/zh-CN/component/link
等中文文档 review 之后 再翻译英文文档
Please make sure these boxes are checked before submitting your PR, thank you!
dev
branch.