This is an automated email from the ASF dual-hosted git repository.
thiagohp pushed a commit to branch feature/coffeescript-to-typescript
in repository https://gitbox.apache.org/repos/asf/tapestry-5.git
The following commit(s) were added to
refs/heads/feature/coffeescript-to-typescript by this push:
new 03230ed0c TAP5-2804: fixed ElementWrapper.attr() in jQuery.
03230ed0c is described below
commit 03230ed0ce8131fdbaef28130b6c6f21bfdc3ee0
Author: Thiago H. de Paula Figueiredo <[email protected]>
AuthorDate: Tue Jun 3 10:33:23 2025 -0300
TAP5-2804: fixed ElementWrapper.attr() in jQuery.
---
tapestry-core/src/main/typescript/src/t5/core/t5-core-dom-jquery.ts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/tapestry-core/src/main/typescript/src/t5/core/t5-core-dom-jquery.ts
b/tapestry-core/src/main/typescript/src/t5/core/t5-core-dom-jquery.ts
index 073a170fa..379a79809 100644
--- a/tapestry-core/src/main/typescript/src/t5/core/t5-core-dom-jquery.ts
+++ b/tapestry-core/src/main/typescript/src/t5/core/t5-core-dom-jquery.ts
@@ -133,7 +133,7 @@ class JQueryElementWrapper implements ElementWrapper {
}
current = this.$.attr(name);
if (arguments.length > 1) {
- if (value === null) {
+ if (value === null || value === undefined) {
this.$.removeAttr(name);
} else {
this.$.attr(name, _.isString(value) ? value : String(value));