Merge branch 'develop' into chore/bump-min-node-version

Author: marcinkrzeminski

.distignore

@@ -2,9 +2,10 @@
 .distignore
 .editorconfig
 .eslintignore
-.eslintrc.json
+.eslintrc.js
 .gitignore
 .nvmrc
+.phpunit.result.cache
 composer.lock
 docker-compose.build.yml
 docker-compose.yml
@@ -14,6 +15,7 @@ package-lock.json
 phpcs.xml.dist
 phpunit.xml
 phpunit-multisite.xml
+playwright.config.js
 renovate.json
 .git
 .github
@@ -25,3 +27,13 @@ dist
 local
 node_modules
 tests
+
+# Playwright generated files.
+/artifacts/
+/test-results/
+/playwright-report/
+/blob-report/
+/playwright/.cache/
+
+# JS and SCSS source files.
+/src/

.eslintignore

@@ -1,5 +1,3 @@
-/ui/lib/
-**/*.min.js
 /vendor/
 /node_modules/
 /build/

.eslintrc.js

@@ -0,0 +1,18 @@
+module.exports = {
+	extends: [
+		'plugin:@wordpress/eslint-plugin/recommended-with-formatting',
+	],
+	env: {
+		browser: true,
+		es6: true,
+	},
+	rules: {
+		'@wordpress/no-global-event-listener': 'off',
+		'jsdoc/check-indentation': 'error',
+		'@wordpress/dependency-group': 'error',
+		'import/order': [ 'error', { groups: [ 'builtin', [ 'external', 'unknown' ], 'internal', 'parent', 'sibling', 'index' ] } ],
+		'jsdoc/require-param': 'off',
+		'jsdoc/require-param-type': 'off',
+		'jsdoc/check-param-names': 'off',
+	},
+};

.eslintrc.json

@@ -54,11 +54,11 @@ jobs:
       - name: Lint
         run: npm run lint
 
+      - name: Build
+        run: npm run build
+
       - name: Pull custom Docker images
         run: docker compose pull wordpress
 
       - name: Test
         run: npm run test
-
-      - name: Build
-        run: npm run build

.github/workflows/lint-and-test.yml

@@ -14,10 +14,11 @@ debug.log
 package.lock
 .phpunit.result.cache
 
-# Compiled files
-ui/js/*.min.js
-ui/css/*.min.css
-alerts/js/*.min.js
-
 # Generated test data
+/artifacts/
 tests/data/tmp/*
+/test-results/
+/playwright-report/
+/blob-report/
+/playwright/.cache/
+/playwright/.auth/

.gitignore

@@ -1 +1 @@
-22
+22

.nvmrc

@@ -2,6 +2,6 @@
   "recommendations": [
     "EditorConfig.EditorConfig",
     "felixfbecker.php-debug",
-    "ikappas.phpcs"
+    "ms-playwright.playwright"
   ]
 }

.vscode/extensions.json

@@ -16,6 +16,8 @@ class Alert_Type_Highlight extends Alert_Type {
 
 	/**
 	 * Main JS file script handle.
+	 *
+	 * @deprecated 4.1.0 Constant is not used anymore and will be removed.
 	 */
 	const SCRIPT_HANDLE = 'wp-stream-alert-highlight-js';
 
@@ -307,34 +309,19 @@ public function ajax_remove_highlight() {
 	 * @param string $page WP admin page.
 	 */
 	public function enqueue_scripts( $page ) {
-		if ( 'toplevel_page_wp_stream' === $page ) {
-			$min = wp_stream_min_suffix();
-
-			wp_register_script(
-				self::SCRIPT_HANDLE,
-				$this->plugin->locations['url'] . 'alerts/js/alert-type-highlight.' . $min . 'js',
-				array(
-					'jquery',
-				),
-				$this->plugin->get_version(),
-				false
-			);
+		if ( 'toplevel_page_wp_stream' !== $page ) {
+			return;
+		}
 
-			$exports = array(
+		$this->plugin->enqueue_asset(
+			'alert-type-highlight',
+			array(),
+			array(
 				'ajaxUrl'      => admin_url( 'admin-ajax.php' ),
 				'removeAction' => self::REMOVE_ACTION,
 				'security'     => wp_create_nonce( self::REMOVE_ACTION_NONCE ),
-			);
-
-			wp_scripts()->add_data(
-				self::SCRIPT_HANDLE,
-				'data',
-				sprintf( 'var _streamAlertTypeHighlightExports = %s;', wp_json_encode( $exports ) )
-			);
-
-			wp_add_inline_script( self::SCRIPT_HANDLE, 'streamAlertTypeHighlight.init();', 'after' );
-			wp_enqueue_script( self::SCRIPT_HANDLE );
-		}
+			)
+		);
 	}
 
 	/**