Skip to content

Commit 4d32d0f

Browse files
committed
Update status about fixed duplicate attributes
1 parent 04b4c81 commit 4d32d0f

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

STATUS.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -93,8 +93,8 @@ Normalization runs twice: once in `compile()` for text detection, once in `rende
9393
**5. Error Handling Philosophy Unclear**
9494
`render()` returns `false` on errors, but `bind()` uses `_doing_it_wrong()` warnings and still returns a template. Mixed signals—should errors be fatal or recoverable? The test gaps document lists many untested failure cases.
9595

96-
**6. Duplicate Attributes Bug with false/null Removal**
97-
When using `false` or `null` to remove an attribute, only the first occurrence is removed. If the HTML contains duplicate attributes (e.g., `<input disabled="</%d>" disabled>`), the first `disabled` will be removed but the second will remain in the output. Duplicate attributes should likely be stripped during the normalization pass to prevent this inconsistency.
96+
**6. ~~Duplicate Attributes Bug with false/null Removal~~ FIXED**
97+
~~When using `false` or `null` to remove an attribute, only the first occurrence is removed. If the HTML contains duplicate attributes (e.g., `<input disabled="</%d>" disabled>`), the first `disabled` will be removed but the second will remain in the output.~~ Fixed by emitting removal edits for duplicate attributes during `compile()`. All duplicate attributes are now stripped as part of template compilation.
9898

9999
---
100100

@@ -148,6 +148,7 @@ The ticket's philosophy is "prefer trust and safety over features"—valid, but
148148
| Funky comment placeholders || Clean implementation |
149149
| Nested HTML via Templates || Text context only |
150150
| Boolean attributes || true/false/null for whole-attr |
151+
| Duplicate attribute removal|| Stripped during compile() |
151152
| URL escaping || Only generic escaping |
152153
| Attribute spread || Not implemented |
153154
| Output format methods || Not implemented |

0 commit comments

Comments
 (0)