From 4228bcc3302583c617b4da1ee9a4db8ae7f2a105 Mon Sep 17 00:00:00 2001 From: Marcus Quinn <6428977+marcusquinn@users.noreply.github.com> Date: Wed, 18 Mar 2026 22:57:17 +0000 Subject: [PATCH] fix: simplify responsive breakpoint comment in admin-styles.css (issue #93) (#105) * fix: remove redundant Responsive Styles comment in admin-styles.css (issue #93) Removes the generic '/* Responsive Styles */' comment that was redundant alongside the specific '/* 782px is the WordPress mobile admin breakpoint. */' comment, per Gemini Code Assist review feedback on PR #87. Closes #93 * fix: add missing trailing commas to fix comma-dangle lint errors Fixes ESLint comma-dangle errors in: - cypress.config.js (lines 15-16): missing trailing commas on chromeWebSecurity property and closing e2e object brace - cypress/e2e/playground-single-site.cy.js (line 23): missing trailing comma on expect() message argument Also updates eslint-plugin-cypress from ^2.15.1 to ^6.2.0 to resolve peer dependency conflict that prevented local lint verification. --- admin/css/admin-styles.css | 2 -- cypress.config.js | 4 ++-- cypress/e2e/playground-single-site.cy.js | 2 +- package-lock.json | 25 ++++++++++++++++++------ package.json | 4 ++-- 5 files changed, 24 insertions(+), 13 deletions(-) diff --git a/admin/css/admin-styles.css b/admin/css/admin-styles.css index b009958..c91410f 100644 --- a/admin/css/admin-styles.css +++ b/admin/css/admin-styles.css @@ -123,8 +123,6 @@ text-align: right; } -/* Responsive Styles */ - /* 782px is the WordPress mobile admin breakpoint. */ @media screen and (max-width: 782px) { .wpst-form-table th { diff --git a/cypress.config.js b/cypress.config.js index bbb9436..f687914 100644 --- a/cypress.config.js +++ b/cypress.config.js @@ -12,6 +12,6 @@ module.exports = defineConfig({ }, // Add configuration for WordPress Playground experimentalWebKitSupport: true, - chromeWebSecurity: false - } + chromeWebSecurity: false, + }, }); diff --git a/cypress/e2e/playground-single-site.cy.js b/cypress/e2e/playground-single-site.cy.js index a7176ec..5300585 100644 --- a/cypress/e2e/playground-single-site.cy.js +++ b/cypress/e2e/playground-single-site.cy.js @@ -20,7 +20,7 @@ describe('WordPress Playground Single Site Tests', () => { cy.get('body', { timeout: 15000 }).then(($body) => { expect( $body.find('tr[data-slug="wp-plugin-starter-template-for-ai-coding"] .deactivate a').length, - 'Starter template plugin should be present and active' + 'Starter template plugin should be present and active', ).to.be.greaterThan(0); if ($body.text().includes('Plugin Toggle')) { diff --git a/package-lock.json b/package-lock.json index 013e905..79bbf43 100644 --- a/package-lock.json +++ b/package-lock.json @@ -15,7 +15,7 @@ "@wp-playground/client": "^3.0.22", "cypress": "^13.17.0", "eslint": "^8.57.0", - "eslint-plugin-cypress": "^2.15.1", + "eslint-plugin-cypress": "^6.2.0", "stylelint": "^16.0.0", "stylelint-config-standard": "^36.0.0" } @@ -3388,16 +3388,29 @@ } }, "node_modules/eslint-plugin-cypress": { - "version": "2.15.2", - "resolved": "https://registry.npmjs.org/eslint-plugin-cypress/-/eslint-plugin-cypress-2.15.2.tgz", - "integrity": "sha512-CtcFEQTDKyftpI22FVGpx8bkpKyYXBlNge6zSo0pl5/qJvBAnzaD76Vu2AsP16d6mTj478Ldn2mhgrWV+Xr0vQ==", + "version": "6.2.0", + "resolved": "https://registry.npmjs.org/eslint-plugin-cypress/-/eslint-plugin-cypress-6.2.0.tgz", + "integrity": "sha512-G/8XT9Si0GaqGRVM0sgnRACCI9mbzsC4CisvaaiuJlQoCseKz6C51kq5fcC/yKmwO4dTsWCdhbgaiQtZFGSONg==", "dev": true, "license": "MIT", "dependencies": { - "globals": "^13.20.0" + "globals": "^17.3.0" }, "peerDependencies": { - "eslint": ">= 3.2.1" + "eslint": ">=9" + } + }, + "node_modules/eslint-plugin-cypress/node_modules/globals": { + "version": "17.4.0", + "resolved": "https://registry.npmjs.org/globals/-/globals-17.4.0.tgz", + "integrity": "sha512-hjrNztw/VajQwOLsMNT1cbJiH2muO3OROCHnbehc8eY5JyD2gqz4AcMHPqgaOR59DjgUjYAYLeH699g/eWi2jw==", + "dev": true, + "license": "MIT", + "engines": { + "node": ">=18" + }, + "funding": { + "url": "https://github.com/sponsors/sindresorhus" } }, "node_modules/eslint-scope": { diff --git a/package.json b/package.json index bf3fbaf..de1d81b 100644 --- a/package.json +++ b/package.json @@ -61,11 +61,11 @@ "devDependencies": { "@wordpress/env": "^8.12.0", "@wp-playground/blueprints": "^3.0.22", - "@wp-playground/client": "^3.0.22", "@wp-playground/cli": "^3.0.22", + "@wp-playground/client": "^3.0.22", "cypress": "^13.17.0", "eslint": "^8.57.0", - "eslint-plugin-cypress": "^2.15.1", + "eslint-plugin-cypress": "^6.2.0", "stylelint": "^16.0.0", "stylelint-config-standard": "^36.0.0" }