-
-
Notifications
You must be signed in to change notification settings - Fork 911
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
Translate multiple select drop down #3756
base: master
Are you sure you want to change the base?
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
Do you want to test this code? Here you have an automated build: |
@HThuren alternatively, you could take a look at an ms-select fork that I created mutiple-select-vanilla. The locales are working fine with imports. I created the fork mainly because I wanted to drop jQuery and also added a few more options. However this would require some code refactoring from your part, it might still be worth it though. Cheers |
@ghiscoding thank you, I'll will look into your solution :-) |
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.
A
Hi @RadAda81, not sure what your comment address ? |
Use locale setup in multiple-select addon
#3583 will have same need for solution like this PR
BUT giving an "Uncaught TypeError: Cannot read properties of undefined (reading 'locales')"
Made an issue wenzhixin/multiple-select#618 to get help
NOT READY TO MERGE