prosemirror-model - npm Package Compare versions

Comparing version 1.21.2 to 1.21.3

CHANGELOG.md

@@ -0,1 +1,7 @@
+## 1.21.3 (2024-06-26)
+
+### Bug fixes
+
+Fix an issue where parse rules for CSS properties that were shorthands for a number of more detailed properties weren't matching properly.
+
 ## 1.21.2 (2024-06-25)
@@ -2,0 +8,0 @@

package.json

 {
   "name": "prosemirror-model",
-  "version": "1.21.2",
+  "version": "1.21.3",
   "description": "ProseMirror's document model",
@@ -5,0 +5,0 @@ "type": "module",

src/from_dom.ts

@@ -184,2 +184,4 @@ import {Fragment} from "./fragment"
   /// @internal
+  matchedStyles: readonly string[]
+  /// @internal
   normalizeLists: boolean
@@ -196,5 +198,11 @@
   ) {
+    let matchedStyles: string[] = this.matchedStyles = []
     rules.forEach(rule => {
-      if (isTagRule(rule)) this.tags.push(rule)
-      else if (isStyleRule(rule)) this.styles.push(rule)
+      if (isTagRule(rule)) {
+        this.tags.push(rule)
+      } else if (isStyleRule(rule)) {
+        let prop = /[^=]*/.exec(rule.style)![0]
+        if (matchedStyles.indexOf(prop) < 0) matchedStyles.push(prop)
+        this.styles.push(rule)
+      }
     })
@@ -547,6 +555,11 @@
     let add = Mark.none, remove = Mark.none
-    for (let i = 0, l = styles.length; i < l; i++) {
-      let name = styles.item(i)
-      for (let after: StyleParseRule | undefined = undefined;;) {
-        let rule = this.parser.matchStyle(name, styles.getPropertyValue(name), this, after)
+    // Because many properties will only show up in 'normalized' form
+    // in `style.item` (i.e. text-decoration becomes
+    // text-decoration-line, text-decoration-color, etc), we directly
+    // query the styles mentioned in our rules instead of iterating
+    // over the items.
+    if (styles.length) for (let i = 0; i < this.parser.matchedStyles.length; i++) {
+      let name = this.parser.matchedStyles[i], value = styles.getPropertyValue(name)
+      if (value) for (let after: StyleParseRule | undefined = undefined;;) {
+        let rule = this.parser.matchStyle(name, value, this, after)
         if (!rule) break
@@ -553,0 +566,0 @@ if (rule.ignore) return null

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

515191

increased by0.38%

Lines of code

11016

increased by0.33%

No dependency changes