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

NPopselect lake a lot of props from NSelect #3795

Open
M1CK431 opened this issue Sep 28, 2022 · 8 comments · May be fixed by #6460
Open

NPopselect lake a lot of props from NSelect #3795

M1CK431 opened this issue Sep 28, 2022 · 8 comments · May be fixed by #6460
Assignees
Labels
feature request New feature or request

Comments

@M1CK431
Copy link
Contributor

M1CK431 commented Sep 28, 2022

This function solves the problem (这个功能解决的问题)

For example, when data come from API, there are some very handy props like label-field or value-field which are not available.

Expected API (期望的 API)

At a first look, the following props are concerned: children-field, default-value, label-field, menu-props, render-option, reset-menu-on-options-change, value-field and there is also on-scroll event.

@github-actions github-actions bot added the feature request New feature or request label Sep 28, 2022
@07akioni
Copy link
Collaborator

07akioni commented Oct 2, 2022

We don't expect popselect to be that complex currently.

@M1CK431
Copy link
Contributor Author

M1CK431 commented Oct 2, 2022

I was thinking Popselect as a "wrapper" for regular Select, with a different behavior regarding it's trigger. By your response, I understand that I was wrong 😞

In my project I only need label-field and value-field currently. For now I just .map data from API so it's not a big deal. However, I guess that props could be useful for others and so perhaps it will be possible to add them in the futur?

@codeinfo
Copy link

codeinfo commented Oct 7, 2022

使用select tag标签,内容为object, 目前仅支持string, 以后有考虑添加对象支持么

@jahnli
Copy link
Collaborator

jahnli commented Mar 12, 2024

This issue does not have any recent activity. If you are still experiencing similar problems, open a new error, including a minimal copy of the problem

@jahnli jahnli closed this as completed Mar 12, 2024
@M1CK431
Copy link
Contributor Author

M1CK431 commented Mar 12, 2024

@jahnli reading your comment, it's obvious that you don't read this issue carefully. Or perhaps you are a bot? 🤭
It's not an error but a feature request which is not implemented for now. See here for the reason why: #3795 (comment)

@jahnli
Copy link
Collaborator

jahnli commented Mar 12, 2024

The problem is already swamped, and if you really need it, I think reopening a problem will make more people see it and want to add it

@jahnli看了你的评论,很明显你没有仔细阅读这个问题。或者也许你是一个机器人?🤭 这不是一个错误,而是一个功能请求,目前尚未实现。原因请参见此处:#3795(评论)

@M1CK431
Copy link
Contributor Author

M1CK431 commented Mar 12, 2024

I strongly disagree 😞

@jahnli jahnli reopened this Mar 13, 2024
@jahnli
Copy link
Collaborator

jahnli commented Mar 13, 2024

At present, the problem has been submerged, I personally think that only by re-opening a problem can more people see it and be willing to add it. If this problem is closed, I am very sorry and I will re-open it

@jahnli jahnli self-assigned this Oct 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants