Refactor (if feasible) uses of the jQuery parent function to use vanillaJS

Made remaining changes and Fixed tests for MR 4337.

few things

Made all the changes. Please Review.

smustgrave → changed the visibility of the branch 3238867-refactor-if-feasible to hidden.

Believe feedback has been addressed but would like if it could get the thumbs up from @nod_

some feedback in the MR

Will leave for sub-maintainer but personally I find addition of [0] for selecting makes the UX go down.

The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

shubh_ → made their first commit to this issue’s fork.

Fixed failing tests.

Believe all feedback has been addressed on this one.

we don't need to replace the customevent dispatch with jquery trigger, it's on purpose that it's a dom event now.

Feedback appears to have been addressed.

Thanks for sticking with it, found a way to simplify a number of selectors thanks to :has()

Thanks, I realized that 6 months ago I found the CSS only solution but browser support was not there yet, now it's been a while and all the supported browsers caught up.

Appears all feedback has been addressed

Sorry, a few more things.

Seems latest round of feedback has been addressed.

saving credit

Doing a last check, there are instances of .parent() left that are not detected by the jquery rule:

        drupal
            core/misc  (4 usages found)
                tabledrag.js  (1 usage found)
                    337 columnIndex = cell.parent().find('> td').index(cell.get(0)) + 1;
                vertical-tabs.js  (2 usages found)
                    237 .parent()
                    263 .parent()
            core/misc/dialog  (1 usage found)
                dialog.position.js  (1 usage found)
                    99 Math.round(event.data.$element.parent().outerHeight()) <
            core/modules/views_ui/js  (2 usages found)
                views-admin.js  (2 usages found)
                    392 .parent()
                    1220 this.$parent = this.$button.parent('div.views-expose, div.views-grouped');

For some reason I added an ignore to one of the instances identified in #39. I have removed that one. Unfortunately, I don't know how to change any of the remaining instances to vanillaJS.

If the rule is not detecting some instances how will future usages be avoided?