🇯🇵Japan @tom konda

We discussed this issue as part of Drupal Contribution Day Japan 2025 →

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

tom konda → created an issue.

I removed more than half of jQuery code from FunctionalJavascript tests.
But several code are not removed because of following reasons.

1. jQuery specific pseudo selector
   Some tests include jQuery pseudo selector such as :focus, :visible and :animated.
   :visible is not removed in 📌 Refactor (if feasible) uses of the jQuery sizzle to use vanillaJS Needs work .
2. jQuery.data()
   The Drupal\FunctionalJavascriptTests\Ajax\CommandsTest includes jQuery.data().
   This function is tried to remove from core JavaScript 📌 Refactor (if feasible) uses of the jQuery data function to use Vanilla/native Active , now still go on.
3. Intentional jQuery usage
   The Drupal\FunctionalJavascriptTests\Dialog\DialogDeprecationsTest intents on to be triggered deprecated jQuery event and the Drupal\FunctionalJavascriptTests\Ajax\AjaxText checks output of JavaScript Ajax theming functions which return jQuery object.

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

The valid-jsdoc rule was removed from ESLint 9.
📌 Replace valid-jsdoc rule with eslint-plugin-jsdoc for eslint Active is blocking this issue.

I try to replace valid-jsdoc rule with eslint-plugin-jsdoc and work well.
The /core/.eslintrc.json file is enabled settings that is written in eslint-plugin-jsdoc's document.

Note: Warnings from eslint-plugin-jsdoc much increase than from valid-jsdoc.

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

tom konda → created an issue.

I don't try yet but find eslint-config-airbnb-extended which supporting ESLint flat config.
IMO, it is worth trying.

Update title of this issue.

OK. I notice to create a meta issue next time like this type of issue.

Added change record. ( https://www.drupal.org/node/3525136 → )
Please review.

tom konda → created an issue.

In this issue, there are two feature branches.
The 3515975-phpcs-issues is a target branch for currently opened MR 29.
The autosave_form-3515975-3515975-phpcs-issues has the latest commit 80bb6c26.
Which is correct branch?

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

tom konda → created an issue.

Added the reason why jQuery UI URL is removed to issue summary.
Please review.

I fixed PHPCS errors.
Please review.

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

I added configuration schema and changed max level label to Unlimited.
But, SitemapUpdateSettingsTest::testUpdateHookN() is still failing.
I think probably need to implement hook_update_N() code.

When this issue was created, the commit id 8c33fb4f was already committed but the commit id ce26a90e was not committed for the feature branch.
I rebased the feature branch and all tests ware passed.

@mparker17 Sorry for my late reply.
Thank you for your reviewing.
I'll try to add configuration schema and change unlimited value.

tom konda → created an issue.

I checked both of Intersection Observer and scrollTo options argument in MDN and "Can I use".
I think all of browsers that require to support for Drupal, work well.
If we need to checking actual browser behavior, need to further working.

On the MDN, both of features display baseline badge as "Widely available".
Both of features had been supported at least since 2020 and compatibility table shows all of browsers are supported so all of supported browsers that Drupal have can work well.
On the "Can I use", all of the latest 2 major versions browsers that Drupal need to support, are shown as green both of Intersection Observer and scrollTo options argument.

Note:

1. The latest release of Firefox ESR on 2025-04-01 works as well as Firefox 128. (Firefox 128 was released in 2024)
2. Opera Mini (except for 'extreme data savings' mode) works as well as Opera Mobile. (See #3202818: Remove untrue declaration of Opera Mini (in reverse proxy mode) support from Drupal 9.3 → )

> - make sure there are no failures in phpcs, cspell, eslint, and stylelint. Make the job fail if there are future failures

All validate stage jobs are green now.

I rebased feature branch and added a test for menu depth option.
But, need to fix SitemapUpdateSettingsTest::testUpdateHookN.

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

Issue #3176383 is removed.
IE11 had been dropped support since Drupal 10. No need to add polyfill.

In my opinion, other instances cannot be replaced.

I added type checking for a JavaScript theme function.
Please review.

tom konda → created an issue.

tom konda → created an issue.

@avpaderno Oh, sorry. I misunderstand that MR!6 is valid branch.

I fixed MR !6 and all validate stage was all green.
Please review.

tom konda → created an issue.

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

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

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

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

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

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

I think this issue is already solved in 🐛 Drupal coding standards Needs work .

I fix all of ESLint errors and all of validate stages are green.
Please review.

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

tom konda → created an issue.

Add testing code for template literal.
Please review.

I reran Functional JavaScript tests and all tests are passed.
It is probably random failure.
Please review.

Ran Functional JavaScript tests several times.
But there is an error on Drupal\Tests\ckeditor5\FunctionalJavascript\ImageUrlProviderTest.
Need to resolve this error.

tom konda → created an issue.

tom konda → created an issue.

Fix ESLint probrem.
Please review.

tom konda → created an issue.

I resolved merge conflict and stylelint is green.
Please review.

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

I rebased MR and fixed all of ESLint errors for latest release.
ESLint is green now. Please review.

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

I rebased the issue branch and fixed some of validate stage errors.

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

I rebased the issue branch and merge error is resolved.
Please review.

I rebased the issue branch and fixed no-var errors.
There are following ESLint probrems left.

• no-undef
• no-new-func
• no-restricted-syntax

Fix ESLint error and rebuild CKEditor5 plugin JavaScript file.

I fixed phpcs errors and phpcs tests are all green.

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

tom konda → created an issue.

I fixed following ESLint errors. But still there is ESLint errors, so need to work.

• ESLint fixable errors
• Camelcase errors
• Eqeqeq errors
• A no-restricted-globals error
• Errors after line 278 except no-undef error

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

Fix pipeline ESLint errors.
Please review.

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

tom konda → created an issue.