Skip to content

Conversation

yesmeck
Copy link
Member

@yesmeck yesmeck commented Aug 13, 2017

Close ant-design/ant-design#5764

single

single mov

combobox

combo mov

@yesmeck yesmeck force-pushed the backfill branch 2 times, most recently from e7bd182 to 3378d34 Compare August 24, 2017 03:23
@yesmeck yesmeck changed the title [WIP]Add backfill Add backfill Aug 24, 2017
@yesmeck yesmeck requested review from afc163 and jaredleechn August 24, 2017 03:46
@yesmeck
Copy link
Member Author

yesmeck commented Aug 24, 2017

依赖 react-component/menu#93 测试才能过。

@jaredleechn
Copy link
Member

普通模式不会回填吗?我看只是改了 placeholder

@yesmeck
Copy link
Member Author

yesmeck commented Aug 24, 2017

@jaredleechn 会的,这个 PR 依赖 react-component/menu#93 ,你可以在本地先改下 menu 的代码。

@jaredleechn
Copy link
Member

image

本地访问 examples 下的 html 返回 Internal Server Error 🤣

@yesmeck
Copy link
Member Author

yesmeck commented Aug 24, 2017

😳 12.0.0.1:8003 呢?或者换个端口?

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 3520916 on backfill into 527fc45 on master.

|value | current selected option(s) | String/Array<String>/{key:String, label:React.Node}/Array<{key, label}> | - |
|firstActiveValue | first active value when there is no value | String/Array<String> | - |
|labelInValue| whether to embed label in value, see above value type | Bool | false |
|backfill| whether backfill select option to search input (Only works in single and combobox mode) | Bool | false |
Copy link
Member

Choose a reason for hiding this comment

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

single 模式支持 backfill 感觉没啥意义,只是个展示效果?

Copy link
Member Author

Choose a reason for hiding this comment

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

只是顺便支持下

src/Select.jsx Outdated
}, 0);
}
if (value.length && value[0].key === selectedValue) {
if (lastValue && lastValue.key === selectedValue && lastValue.backfill !== true) {
Copy link
Member

Choose a reason for hiding this comment

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

lastValue.backfill !== true => !lastValue.backfill

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 0320794 on backfill into 527fc45 on master.

@yesmeck yesmeck merged commit ecefdfe into master Sep 1, 2017
@yesmeck yesmeck deleted the backfill branch September 1, 2017 02:39
@xjkms
Copy link

xjkms commented Feb 22, 2019

@yesmeck how to disable backfill feature?

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.

5 participants