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

Ajaxify product tag selection in backend #388

Closed
muratcakir opened this issue Feb 24, 2021 · 1 comment
Closed

Ajaxify product tag selection in backend #388

muratcakir opened this issue Feb 24, 2021 · 1 comment

Comments

@muratcakir
Copy link
Contributor

No description provided.

@muratcakir muratcakir added this to the 4.5 milestone Feb 24, 2021
@muratcakir muratcakir modified the milestones: 5.0 Beta, 5.0 Apr 22, 2022
@mgesing
Copy link
Contributor

mgesing commented Jun 10, 2022

Loading via AJAX is not good here. There is no search field in a multiple tags select2 box. If you type something, the input automatically appears at the first position in the popup list and clicking on it will add it as a new tag. An "Enter" does not (!) apply the entry, but removes it again (not very intuitive). Hard to explain.

Better and easier: just load e.g. a maximum of 5000 product tags instead of all. The user won't want to search in such a long list anyway, but will simply enter missing tags directly. Product tags are stored by IProductTagService.UpdateProductTagsAsync and this method ensures that no duplicate tags are stored anyway.

@muratcakir muratcakir modified the milestones: 5.0, 5.0.1 Sep 2, 2022
@Michael-Herzog Michael-Herzog modified the milestones: 5.0.1, 5.0.2 Nov 3, 2022
@muratcakir muratcakir modified the milestones: 5.0.2, 5.1.0 Nov 18, 2022
@muratcakir muratcakir removed their assignment Mar 30, 2023
@mgesing mgesing reopened this Jul 25, 2023
@mgesing mgesing closed this as completed in 437b275 Aug 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants