-
Notifications
You must be signed in to change notification settings - Fork 88
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
WordPress 5.5 incompatibility #35
Comments
I'm having the same problem. After a bit of troubleshooting, I have ended up here! |
Hi guys |
Same issue |
I am also facing this issue. Any help will be greatly appreciated? We have to remove the alpha color picker because without resolving the JS error we can update the WP version against our plugins. |
@sehrish-iftikhar as a workaround, while the issue is fixed, localizing the strings that aren't localized anymore in the core seems to do fine.
Glad to learn if there's a better approach. |
…r: wpColorPickerL10n is not defined
@kallookoo Please checkout the pull request : |
@prince5ingh your solution will not work and is not a proper way. Please check @a-danae solution. |
@pzstar well i tested at my end and is working, secondly did you check the official WP commit which caused this issue ? |
Oh i see the WP commit and it looks incorrect as well because the translation value passed in javascript file like this __( 'Color value' ) can never be translated. |
lol, instead of making comments would have been better if you would have read the code & tried to understand it
|
Ok. Thanks for the explanation :) |
@prince5ingh thank you! |
@prince5ingh Tested, amazing it works like a charm 👍 |
Have just submitted a PR for this. This PR takes in account comments that were raised on a previous PR submitted by @prince5ingh including compatibility with previous WP versions. Has been tested with WordPress 4.8.14, 4.9.15, 5.4.2 and 5.5 |
@a-danae @maddisondesigns For all those who can't wait, you have this modification created by @maddisondesigns that works when it comes to translating the texts. |
Hey guys, any updates for this? |
@ArrayIterator |
Make sense, @kallookoo. Has anyone tested against jQuery 3.5? That's what will be shipped in December. |
@mweichert I just tested my PR that I submitted above, with jQuery 3.5.1 (on WP 5.5.1) and everything works fine. If you want to test it yourself, you can do it easily by installing the Test jQuery Updates plugin and enabling v3.5.1 in the settings. |
Hi, |
@kallookoo Have just tried this new version and it's not working for me unfortunately. I've just updated my script to the new 3.0.0 version and the customizer controls render as per the screenshot above. It's not showing the Alpha slider. This was tested on WP 5.5.1 |
@a-danae your suggestion worked like a charm to solve these type of warnings in console: Thanks! |
Resolved. Thank you all for the collaboration and I am sorry to wait but in my tests it already works well and if you want you have examples of how to use it in this repository |
@kallookoo Thanks again for all your work on this control. I've been updating my own Customizer code with your new script and after implementing the changes that you've made to the data attributes, I have the color picker showing as expected, in the Customizer. The problem is, whenever I select a colour, the change isn't being triggered and the Publish button is never becoming enabled. Here's a short vid: https://share.getcloudapp.com/7Ku8NJn7 Is there something else that I'm missing? When I start the script, the only option I'm passing is the palette colors. |
Hi @maddisondesigns it is fixed. |
@kallookoo This is a problem I'm having with this new version of the script. I've deactivated your test plugin, but I'm still having issues with the changes not being triggered in my own Customizer controls, using this new version of the script. |
Umm, Please open a new issue and if possible add your customize code, thank you. |
Just download WP Downgrade plugin and down grade to a more stable and compatible WP version until most plugins of that theme meets the recent WP version. |
I was just testing this with WordPress 5.5 and I'm getting the following error:
Uncaught ReferenceError: wpColorPickerL10n is not defined
Reference - WordPress/WordPress@7e7b70c
The text was updated successfully, but these errors were encountered: