Frontmatter
| id | 3188 |
| title | main.mixin.DeltaUpdates: changing node attributes |
| state | Closed |
| labels | enhancement |
| assignees | tobiu |
| createdAt | Jun 23, 2022, 2:45 PM |
| updatedAt | Jun 23, 2022, 2:46 PM |
| githubUrl | https://github.com/neomjs/neo/issues/3188 |
| author | tobiu |
| commentsCount | 0 |
| parentIssue | null |
| subIssues | [] |
| subIssuesCompleted | 0 |
| subIssuesTotal | 0 |
| blockedBy | [] |
| blocking | [] |
| closedAt | Jun 23, 2022, 2:46 PM |
so far, neo was using:
node[key] = val;this works fine for many attributes, but i just ran into edge cases where it does not (only tested in chrome). minlength and maxlength did not get applied using this strategy.
node.setAttribute(key, val);works fine for these edge-cases as well, so i will change it.hopefully there are no side-effects, otherwise we will need to create follow-up tickets.