Skip to content

Commit 753575b

Browse files
committed
[FIX] spreadsheet: fix scroll issue on chromium core 125
Since the update of chromium-based browsers to chromium 125, our `wheel` listeners declared inside a template are no longer properly registered. It turns out that registering another event listener of the same type fixes the template scroll. This revision proposes this simple hack/fix as we still struggle to find the actual reason behind this (might be owl related) to unblock users. closes #4252 Task: 0 X-original-commit: f426b43 Signed-off-by: Vincent Schippefilt (vsc) <[email protected]> Signed-off-by: Rémi Rahir (rar) <[email protected]>
1 parent c1f0c3f commit 753575b

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

src/components/spreadsheet/spreadsheet.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -231,6 +231,11 @@ export class Spreadsheet extends Component<SpreadsheetProps, SpreadsheetChildEnv
231231
useExternalListener(window as any, "resize", () => this.render(true));
232232
useExternalListener(window, "beforeunload", this.unbindModelEvents.bind(this));
233233

234+
// For some reason, the wheel event is not properly registered inside templates
235+
// in Chromium-based browsers based on chromium 125
236+
// This hack ensures the event declared in the template is properly registered/working
237+
useExternalListener(document.body, "wheel", () => {});
238+
234239
this.bindModelEvents();
235240
onMounted(() => {
236241
this.checkViewportSize();

0 commit comments

Comments
 (0)