Merge pull request #6955 from nextcloud-libraries/chore/stylelint-upgrade

chore(dev-deps): bump @nextcloud/stylelint-config from 3.0.1 to 3.1.0 and fix errors

Author: susnux

.stylelintignore

@@ -61,8 +61,7 @@
     "test:component:gui": "playwright test --ui -c playwright.config.ts",
     "test:coverage": "vitest run --coverage",
     "ts:check": "vue-tsc --noEmit",
-    "update:snapshots": "npm run test:component -- --grep @visual --update-snapshots",
-    "watch": "npm run dev:watch"
+    "update:snapshots": "npm run test:component -- --grep @visual --update-snapshots"
   },
   "browserslist": [
     "extends @nextcloud/browserslist-config"
@@ -121,7 +120,7 @@
     "@nextcloud/babel-config": "^1.2.0",
     "@nextcloud/browserslist-config": "^3.0.1",
     "@nextcloud/eslint-config": "^8.4.2",
-    "@nextcloud/stylelint-config": "^3.0.1",
+    "@nextcloud/stylelint-config": "^3.1.0",
     "@nextcloud/vite-config": "^2.3.2",
     "@nextcloud/webpack-vue-config": "github:nextcloud/webpack-vue-config#vue3",
     "@playwright/experimental-ct-vue": "^1.51.1",
@@ -142,7 +141,6 @@
     "resolve-url-loader": "^5.0.0",
     "sass": "^1.86.1",
     "stylelint": "^16.17.0",
-    "stylelint-use-logical": "^2.1.2",
     "ts-node": "^10.9.2",
     "typescript": "^5.8.2",
     "url-loader": "^4.1.1",

package-lock.json

@@ -2,7 +2,7 @@
  * SPDX-FileCopyrightText: 2024 Nextcloud GmbH and Nextcloud contributors
  * SPDX-License-Identifier: AGPL-3.0-or-later
  */
-@use 'variables' as variables;
+@use 'variables.scss' as variables;
 
 .header-menu {
 	position: relative;

package.json

@@ -1017,7 +1017,7 @@ export default {
 		padding-inline: var(--contenteditable-inline-start-offset) var(--contenteditable-inline-end-offset);
 		cursor: text;
 		white-space: pre-wrap;
-		word-break: break-word;
+		overflow-wrap: break-word;
 		color: var(--color-main-text);
 		border: 2px solid var(--color-border-maxcontrast);
 		border-radius: var(--border-radius-large);

src/components/NcHeaderMenu/header-menu__trigger.scss

@@ -575,7 +575,7 @@ export default {
 
 // Plain text styles
 .rich-text--wrapper {
-	word-break: break-word;
+	overflow-wrap: break-word;
 	line-height: 1.5;
 
 	.rich-text--fallback, .rich-text-component {

src/components/NcRichContenteditable/NcRichContenteditable.vue

@@ -7,7 +7,6 @@ const stylelintConfig = require('@nextcloud/stylelint-config')
 
 module.exports = {
 	extends: ['@nextcloud/stylelint-config'],
-	plugins: ['stylelint-use-logical'],
 	rules: {
 		// For CSS Modules
 		// If there will be more rules for CSS Modules - consider extending stylelint-config-css-modules
@@ -17,30 +16,6 @@ module.exports = {
 				ignorePseudoClasses: [...stylelintConfig.rules['selector-pseudo-class-no-unknown'][1].ignorePseudoClasses, 'global'],
 			},
 		],
-		'csstools/use-logical': [
-			'always',
-			{
-				severity: 'error',
-				// Only lint LTR-RTL properties for now
-				except: [
-					// Position properties
-					'top',
-					'bottom',
-					// Position properties with directional suffixes
-					/-top$/,
-					/-bottom$/,
-					// also for e.g. border-bottom-color
-					/-top-/,
-					/-bottom-/,
-					// Size properties
-					'width',
-					'max-width',
-					'min-width',
-					'height',
-					'max-height',
-					'min-height',
-				],
-			},
-		],
+		'scss/load-partial-extension': ['always'],
 	},
 }