Merge pull request #671 from trheyi/main

fix the bug in handling the replaceNodeUse attribute
YaoApp · Jul 8, 2024 · 1b808f9 · 1b808f9
2 parents c0f9105 + 9bae50b
commit 1b808f9
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/sui/core/data.go b/sui/core/data.go
@@ -135,8 +135,8 @@ func (data Data) replaceNodeUse(re *regexp.Regexp, node *html.Node) bool {
 
 	case html.ElementNode:
 		for i := range node.Attr {
-			// Keepwords
-			if strings.HasPrefix(node.Attr[i].Key, "s:") || node.Attr[i].Key == "is" {
+
+			if (strings.HasPrefix(node.Attr[i].Key, "s:") || node.Attr[i].Key == "is") && !allowUsePropAttrs[node.Attr[i].Key] {
 				continue
 			}
 

diff --git a/sui/core/parser.go b/sui/core/parser.go
@@ -65,6 +65,13 @@ var keepWords = map[string]bool{
 	"s:bind":      true,
 }
 
+var allowUsePropAttrs = map[string]bool{
+	"s:if":    true,
+	"s:elif":  true,
+	"s:for":   true,
+	"s:click": true,
+}
+
 var keepAttrs = map[string]bool{
 	"s:ns":    true,
 	"s:cn":    true,