Skip to content

Commit

Permalink
undo perf to see byte impact
Browse files Browse the repository at this point in the history
  • Loading branch information
JoviDeCroock committed Mar 23, 2024
1 parent 324d27f commit 341c937
Showing 1 changed file with 15 additions and 16 deletions.
31 changes: 15 additions & 16 deletions src/diff/props.js
Expand Up @@ -13,21 +13,6 @@ function setStyle(style, key, value) {
}
}

const reserved = [
'width',
'height',
'href',
'list',
'form',
// Default value in browsers is `-1` and an empty string is
// cast to `0` instead
'tabIndex',
'download',
'rowSpan',
'colSpan',
'role'
];

/**
* Set a property value on a DOM node
* @param {PreactElement} dom The DOM node to modify
Expand Down Expand Up @@ -105,7 +90,21 @@ export function setProperty(dom, name, value, oldValue, isSvg) {
// - xlink:href / xlinkHref --> href (xlink:href was removed from SVG and isn't needed)
// - className --> class
name = name.replace(/xlink(H|:h)/, 'h').replace(/sName$/, 's');
} else if (!reserved.includes(name) && name in dom) {
} else if (
name != 'width' &&
name != 'height' &&
name != 'href' &&
name != 'list' &&
name != 'form' &&
// Default value in browsers is `-1` and an empty string is
// cast to `0` instead
name != 'tabIndex' &&
name != 'download' &&
name != 'rowSpan' &&
name != 'colSpan' &&
name != 'role' &&
name in dom
) {
try {
dom[name] = value == null ? '' : value;
// labelled break is 1b smaller here than a return statement (sorry)
Expand Down

0 comments on commit 341c937

Please sign in to comment.