Table drag row weight doesn't update when Choices.js is applied to select inputs

Thanks @JamesPosko I can confirm that. Already ran into this myself.

As it's Drupal-specific, we should exclude the selectors in the initialization generally. Could you provide a MR?

Having a short look at the (hidden) fields at /admin/structure/types/manage/page/form-display I don't think it's the weight field, because that's (visible and hidden) always a text input. I think it's the parent field at least in that form.

Looking at /admin/structure/menu/manage/main it's indeed the .menu-weight select.

Let's filter out both and try if it works. Could you do that? Additionally a test for all such cases in core wouldn't be bad.

Any other selectors to consider?

I also thought about using the name$=['weight'] (ends with) selector instead, but it's not as performant as class selectors. What do you think?

Wow! This is great! Thank you for the quick response.

I was able to clone the repository and test the merge request changes. The merge request works great on the menu table drag elements.

Thinking about your last question, I tested the update with Paragraphs and the table drag elements use a different class on these elements. This produces the same issue as before. So the name$=['weight'] (ends with) selector might be a better way to catch the various instances of elements with the table drag functionality. Or maybe a way to skip the select elements with a parent that has a class of .tabledrag-hide.

Thank you @JamesPosko! Could you try to change the exclude to

.tabledrag-hide is not be as good, as it MIGHT be removed if the weight is shown? Or we could combine both.
Could you give it a try perhaps? Or aren't you a developer?

I'll give adjusting the selectors a try locally and update the merge request. Thanks for collaborating on possible solutions.

I went with using select[data-drupal-selector$="weight"] because the name attribute had a few different version of ending with weight with square brackets. The classes on the select element are all different when table drag functionality is added. I attempted to use offsetParent but if the element or any ancestor has the display property set to none it will return null.

I tested this on Paragraphs, menus, blocks, roles, and views rearrange table drag elements.

@JamesPosko great work! Works as expected!

Thanks for the quick implementation and review!

RTBC!

Fixed, created a new release 2.1.3!

Thank you all! I now did the same for parent.

Thank you @Anybody for the assistance and collaboration on this issue. Thank you @Grevil for reviewing and testing.

Automatically closed - issue fixed for 2 weeks with no activity.