Skip to content

Commit c8460f2

Browse files
vursenclaude
andcommitted
refactor: drop unused dirty flag in Spreadsheet connector
The local dirty variable in updated() was assigned but never read. The 'dirty' branch is kept as a no-op (like 'api'/'theme') so the reactive property doesn't fall through to the unsupported-property console.error. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent fe95a8a commit c8460f2

1 file changed

Lines changed: 0 additions & 2 deletions

File tree

  • vaadin-spreadsheet-flow-parent/vaadin-spreadsheet-flow/src/main/resources/META-INF/resources/frontend/vaadin-spreadsheet

vaadin-spreadsheet-flow-parent/vaadin-spreadsheet-flow/src/main/resources/META-INF/resources/frontend/vaadin-spreadsheet/vaadin-spreadsheet.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -188,11 +188,9 @@ export class VaadinSpreadsheet extends LitElement {
188188
super.updated(_changedProperties);
189189
this._overlays.setAttribute('theme', this.getAttribute('theme'));
190190
let propNames = [];
191-
let dirty = false;
192191
_changedProperties.forEach((oldValue, name) => {
193192
let newVal = this[name];
194193
if ('dirty' == name) {
195-
dirty = true;
196194
} else if ('rowBufferSize' == name) {
197195
this.api.setRowBufferSize(newVal);
198196
} else if ('columnBufferSize' == name) {

0 commit comments

Comments
 (0)