-
Notifications
You must be signed in to change notification settings - Fork 29
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
[dev] All terms get saved instead of just selected terms #53
Comments
Great, I was going to open this based on your WP.org reply. And just noticed it myself. I think I have to dig into the API as everything in Gutenberg is saved via the API. |
Either need to find a way to modify the results of |
I actually think this may be more oriented towards the JS side of things. Specifically, may need to mimic how the Radio controls set state. https://github.com/WordPress/gutenberg/tree/master/packages/components/src/radio-control |
@leymannx Hi Norman. I just pushed a few changes. I think it solves the saving issue. I'd appreciate it if you could take a look when you have a chance. |
It works super smooth now. Select and updates precisely. Well done! Only one minor thing with the styles of the focused radio that underlaps the meta box wrapper. It's not bad thing, but it may look broken to the enduser. I'll drop a new issue for that. |
I've seen the style thing too. Not sure what to do about it. Glad it's working! Gotta give credit to @5ally and her work at https://github.com/5ally/my-term-selector/ she's been amazingly helpful on this. I am a little nervous about just how many changes were made to the term selector script. I fear it coud be hard to maintain, but we'll cross that bridge when we get to it. I'll give a quick thought to the style issue and try to get this packaged up for release! |
I'm a little bit concerned regarding backwards compatibility. But you didn't remove any of the old code? So the plugin should still work on older WordPress versions you think? |
For back compat, the old scripts should still load when not using
Gutenberg.... So anywhere tractional metaboxs are leading. But I only
tested quickly with product post type.
…On Wed, Apr 24, 2019, 7:57 PM Norman Kämper-Leymann < ***@***.***> wrote:
I'm a little bit concerned regarding backwards compatibility. But you
didn't remove any of the old code? So the plugin should still work on older
WordPress versions you think?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#53 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAD3ZEJTH62X2VYG6CT2CILPSA4LHANCNFSM4HEZ37CQ>
.
|
I checked out the dev branch for WP5 compatibility and radios appear nicely for the chosen taxonomies.
Unfortunately the term saving doesn't seem to work properly. After I selected a term, saved the post and watched it in the frontend, all terms appear.
I'd expect only the chosen term to be applied.
That's what I selected
That's what has been saved
The text was updated successfully, but these errors were encountered: