-
-
Notifications
You must be signed in to change notification settings - Fork 233
feat: Add new trigger type --- 'contextmenu' #72
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
Conversation
Changes Unknown when pulling fa57faf on MrPeak:master into ** on react-component:master**. |
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.
我没问题了,补充下文档就好。
cc @afc163 |
examples/simple.js
Outdated
@@ -57,7 +57,7 @@ class Test extends React.Component { | |||
maskClosable: false, | |||
placement: 'right', | |||
trigger: { | |||
hover: 1, | |||
contextMenu: 1, |
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.
这个别改吧,新建一个 demo。
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.
现在的demo就按照嵌入跟非嵌入划分的了
默认还是hover触发,我加到后面吧,contextmenu的场景应该相当少,没必要单独搞一个html吧
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.
👌
补个用例。 |
Changes Unknown when pulling 8cb5194 on MrPeak:master into ** on react-component:master**. |
CC @afc163 |
2.1.0 |
No description provided.