Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

使用vue-clickaway处理组件外部点击事件 #1251

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

Open
HughDai wants to merge 2 commits into PanJiaChen:master
base: master
Choose a base branch
Loading
from HughDai:master

Conversation

Copy link

@HughDai HughDai commented Oct 24, 2018

tags-view组件引入vue-clickaway处理context menu外部点击事件,控制context menu的visible,让代码更简洁。

... outside of the component, and also make code neat
perf[TagsView]:use vue-clickaway to close context menu when you click...
Copy link
Owner

不觉得这样会简洁,引入新的库感觉会让新手觉得更复杂。本来实现的代码就不复杂,感觉没啥必要。

Copy link
Author

HughDai commented Oct 24, 2018

不觉得这样会简洁,引入新的库感觉会让新手觉得更复杂。本来实现的代码就不复杂,感觉没啥必要。

或许可以直接把Element UI 中的 clickoutside.js 拿过来用

Copy link
Owner

现在的实现方案有什么问题么?

Copy link
Author

HughDai commented Oct 24, 2018

@PanJiaChen 这样写当然没问题,只不过我个人感觉用click-away或者click-outside看上去优雅些

Copy link
Owner

但减少了代码可阅读性,他们看代码还需要先去了解click-away或者click-outside

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /