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

itable添加禁用某行选中的功能 #286

Merged
merged 12 commits into from
Feb 28, 2017
Merged

Conversation

GITleonine1989
Copy link
Contributor

#265
针对第二个问题,禁用某行的选中而提的pr
第一个问题在pr #275 已提交

Copy link
Contributor Author

@GITleonine1989 GITleonine1989 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IE下不支持es6字符串模板

@@ -13,7 +13,7 @@
</button>
<ul :class="dotsClasses">
<template v-for="n in slides.length">
<li :class="{ [`${prefixCls}-active`]: n === currentIndex }"
<li :class="[n === currentIndex ? prefixCls+'-active':'' ]"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1.如果某行禁用_disabled=true,则这行的_checked,无论如何都是false,因为禁用的目的就是不能选择
2.修复某行设置_disabled=true时,全选状态判断有误的bug
@icarusion icarusion merged commit 183ab50 into iview:master Feb 28, 2017
@icarusion
Copy link
Contributor

Thx.

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

2 participants