251383 2023-01-30 12:40:40 -0800 Reflected ARIA properties should not treat setting undefined the same as null 2023-10-25 09:32:10 -0700 1 1 1 Unclassified WebKit Accessibility Safari Technology Preview Unspecified Unspecified RESOLVED INVALID InRadar P2 Normal --- 1 nlawson webkit-unassigned andresg_22 rego rniwa tyler_w webkit-bug-importer oldest_to_newest 1929504 0 464760 nlawson 2023-01-30 12:40:40 -0800 Created attachment 464760 Repro Safari Technology Preview version: Release 146 (Safari 15.4, WebKit 16614.1.14.10.6) Steps to reproduce: 1. Go to https://codepen.io/nolanlawson-the-selector/pen/OJwojEW What is the expected result? It should log "undefined" (the string). What happens instead? It logs null. Here is the minimal repro (same as in the CodePen): const div = document.createElement('div') div.ariaLabel = undefined console.log(div.ariaLabel) Per the ARIA spec [1], setting `null` on a reflected `aria*` property removes the associated `aria-` attribute (if it exists). However, it doesn't say anything about setting `undefined`. So in this case, setting `undefined` should presumably be treated the same as any other non-string value – i.e. it should be stringified: div.ariaLabel = undefined div.ariaLabel // 'undefined' div.getAttribute('aria-label') // 'undefined' Note that Chrome and Safari behave identically, and Firefox has not implemented ARIA reflection yet. More details are in the relevant ARIA bug [2]. I also filed a bug on Chromium [3]. [1]: https://w3c.github.io/aria/#enumerated-attribute-values-html [2]: https://github.com/w3c/aria/issues/1858 [3]: https://bugs.chromium.org/p/chromium/issues/detail?id=1411465 1929842 1 webkit-bug-importer 2023-01-31 13:04:18 -0800 <rdar://problem/104881298> 1987438 2 nlawson 2023-10-25 09:32:10 -0700 This issue can be closed per tentative resolution in https://github.com/w3c/aria/pull/2057. Once the spec is updated, WebKit's behavior will be correct per the spec. 464760 2023-01-30 12:40:40 -0800 2023-01-30 12:40:40 -0800 Repro repro.html text/html 145 nlawson PCFkb2N0eXBlIGh0bWw+CjxodG1sPgo8c2NyaXB0Pgpjb25zdCBkaXYgPSBkb2N1bWVudC5jcmVh dGVFbGVtZW50KCdkaXYnKQpkaXYuYXJpYUxhYmVsID0gdW5kZWZpbmVkCmNvbnNvbGUubG9nKGRp di5hcmlhTGFiZWwpCjwvc2NyaXB0Pgo8L2h0bWw+Cg==