-
Notifications
You must be signed in to change notification settings - Fork 8
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
feat(conditions): add search by input #150
Conversation
This PR is deployed at https://happy-dune-06c336600-150.eastasia.2.azurestaticapps.net |
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.
src/views/Tools/Conditions.vue
Outdated
if (!find) { | ||
this.$message.error('未找到对应韵母!韵母是必选项!') | ||
return |
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.
为啥韵母是必选项
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.
如果是因为双向转化里需要一个韵母的位置可以考虑用?还是什么代替一下?
src/views/Tools/Conditions.vue
Outdated
<a-row style="margin-top: 5px"> | ||
<a-input-search | ||
v-model="searchContent" | ||
enter-button | ||
placeholder="请输入拼音" | ||
size="large" | ||
@search="search(searchContent)" | ||
/> | ||
</a-row> |
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.
我理解为,这个输入框的目的是和上面的选择框双向转换?
那我认为说,其实上面的单选框如果点点点发生变化了,这个地方应该会同步?
另外,#146说的其实不是这个地方添加,我更新了一下描述。不过这个功能还是可以修改而后更新的 |
修改需求 |
This PR is deployed at https://happy-dune-06c336600-150.eastasia.2.azurestaticapps.net |
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.
这个先通过了,不过搜索框里搜索的时候应该不需要额外添加“pinyin”吧?直接搜索就行
No description provided.