- Merge request !1239Issue #3238868: Refactor (if feasible) uses of the jQuery parents function to use vanillaJS → (Open) created by hooroomoo
- Issue was unassigned.
- Status changed to Needs review
12 months ago 6:53am 1 February 2024 - Status changed to Needs work
12 months ago 8:48am 1 February 2024 - 🇩🇪Germany Feuerwagen Bonn 🇩🇪🇪🇺
I'm pretty sure the issue title doesn't mean „replace uses of the jQuery parents function with another jQuery function“ – these issues are targeted to replace jQuery with vanillaJS.
- 🇮🇳India ahsannazir
Yes got it. I have used closest() with the intention that it works without jquery.
- Status changed to Needs review
12 months ago 11:49am 1 February 2024 - 🇩🇪Germany Feuerwagen Bonn 🇩🇪🇪🇺
Sorry about the noise than and apologies – I wasn't aware that this method exists in vanillaJs with identical functionality than the one in jQuery.
- Status changed to Needs work
12 months ago 12:47pm 2 February 2024 - 🇮🇳India ahsannazir
ahsannazir → changed the visibility of the branch 11.x to hidden.
- Merge request !6608Issue #3238868: Refactor jquery parents function. → (Closed) created by Unnamed author
- 🇮🇳India ahsannazir
ahsannazir → changed the visibility of the branch 3238868-refactor-parents to hidden.
- Status changed to Needs review
12 months ago 1:45pm 15 February 2024 - Status changed to Needs work
12 months ago 3:22pm 20 February 2024 - 🇺🇸United States smustgrave
Left some small comments.
Will want sub maintainer sign off before merging.
- Status changed to Needs review
11 months ago 9:42am 21 February 2024 - Status changed to Needs work
11 months ago 2:51am 1 March 2024 - 🇫🇷France nod_ Lille
If we need a 10/20 lines method to replace a jquery call, it's not worth the effort. Let's keep them, and add an eslint ignore.
- 🇫🇷France nod_ Lille
Actually we might be able to replicate the parents() functionality with css :has() in some cases:
document.querySelectorAll('*:has(#edit-description)');
would give all the parents of#edit-description
- Status changed to Needs review
11 months ago 11:44am 4 March 2024 - 🇺🇸United States smustgrave
smustgrave → changed the visibility of the branch 3238868-refactor-if-feasible to hidden.
- Status changed to RTBC
11 months ago 10:24pm 4 March 2024 - 🇺🇸United States smustgrave
Feedback appears to be addressed
@nod_ can we remove the submaintainer tag?
Pushed up a super nitpicky change to remove a space.
- Status changed to Needs work
11 months ago 9:20pm 5 March 2024 - 🇫🇷France nod_ Lille
There are 4 exceptions and 3 legitimate replacements so let's not call this removing the use of parents.
I'll take the 3 replacements but let's keep the jquery rule as-is and remove the eslint ignore comments
- Status changed to Needs review
11 months ago 6:44am 6 March 2024 - Status changed to Needs work
11 months ago 10:03am 6 March 2024 - 🇫🇷France nod_ Lille
Thanks! a few tiny changes and we should be good to go :)
- 🇫🇷France nod_ Lille
I want to see what testbot says about a change in ajax.js
- Status changed to Needs review
11 months ago 9:16am 7 March 2024 - Status changed to RTBC
9 months ago 3:16pm 22 April 2024 - 🇺🇸United States smustgrave
Going to lean on @nod_ numerous reviews on this one. Tried to fix an out of scope change for removing a space that's what that commit was.
- 🇫🇷France nod_ Lille
Retitle for commit, opened 📌 Refactor (if feasible) uses of the jQuery parents function to use vanillaJS Active to address the other uses of parents and change the eslint config
- Status changed to Fixed
9 months ago 9:08pm 22 April 2024 Automatically closed - issue fixed for 2 weeks with no activity.
- 🇯🇴Jordan Ahmad Khader
This issue appears to break the settings tray, causing contextual links to disappear. please review
https://www.drupal.org/project/drupal/issues/3458067 🐛 Contextual links disappear intermittently leading to console errors Active Unable to find closestSettingsTray;