diff --git a/src/browser/actions.zig b/src/browser/actions.zig
index 60b03f1e..36b96bc5 100644
--- a/src/browser/actions.zig
+++ b/src/browser/actions.zig
@@ -63,21 +63,21 @@ pub fn fill(node: *DOMNode, text: []const u8, page: *Page) !void {
     _ = page._event_manager.dispatch(el.asEventTarget(), change_evt) catch {};
 }
 
-pub fn scroll(node: ?*DOMNode, x: i32, y: i32, page: *Page) !void {
+pub fn scroll(node: ?*DOMNode, x: ?i32, y: ?i32, page: *Page) !void {
     if (node) |n| {
         const el = n.is(Element) orelse return error.InvalidNodeType;
 
-        if (x != 0) {
-            el.setScrollLeft(x, page) catch {};
+        if (x) |val| {
+            el.setScrollLeft(val, page) catch {};
         }
-        if (y != 0) {
-            el.setScrollTop(y, page) catch {};
+        if (y) |val| {
+            el.setScrollTop(val, page) catch {};
         }
 
         const scroll_evt = try Event.initTrusted(comptime lp.String.wrap("scroll"), .{ .bubbles = true }, page);
         _ = page._event_manager.dispatch(el.asEventTarget(), scroll_evt) catch {};
     } else {
-        page.window.scrollTo(.{ .x = x }, y, page) catch |err| {
+        page.window.scrollTo(.{ .x = x orelse 0 }, y, page) catch |err| {
             lp.log.err(.app, "scroll failed", .{ .err = err });
             return error.ActionFailed;
         };
diff --git a/src/cdp/domains/lp.zig b/src/cdp/domains/lp.zig
index ed6c6994..fedd71e6 100644
--- a/src/cdp/domains/lp.zig
+++ b/src/cdp/domains/lp.zig
@@ -207,9 +207,6 @@ fn scrollNode(cmd: anytype) !void {
     const bc = cmd.browser_context orelse return error.NoBrowserContext;
     const page = bc.session.currentPage() orelse return error.PageNotLoaded;
 
-    const x = params.x orelse 0;
-    const y = params.y orelse 0;
-
     const input_node_id = params.nodeId orelse params.backendNodeId;
 
     var target_node: ?*DOMNode = null;
@@ -218,7 +215,7 @@ fn scrollNode(cmd: anytype) !void {
         target_node = node.dom;
     }
 
-    lp.actions.scroll(target_node, x, y, page) catch |err| {
+    lp.actions.scroll(target_node, params.x, params.y, page) catch |err| {
         if (err == error.InvalidNodeType) return error.InvalidParam;
         return error.InternalError;
     };
diff --git a/src/mcp/tools.zig b/src/mcp/tools.zig
index 6f256dca..d8fd4ead 100644
--- a/src/mcp/tools.zig
+++ b/src/mcp/tools.zig
@@ -494,9 +494,6 @@ fn handleScroll(server: *Server, arena: std.mem.Allocator, id: std.json.Value, a
         return server.sendError(id, .PageNotLoaded, "Page not loaded");
     };
 
-    const x = args.x orelse 0;
-    const y = args.y orelse 0;
-
     var target_node: ?*DOMNode = null;
     if (args.backendNodeId) |node_id| {
         const node = server.node_registry.lookup_by_id.get(node_id) orelse {
@@ -505,7 +502,7 @@ fn handleScroll(server: *Server, arena: std.mem.Allocator, id: std.json.Value, a
         target_node = node.dom;
     }
 
-    lp.actions.scroll(target_node, x, y, page) catch |err| {
+    lp.actions.scroll(target_node, args.x, args.y, page) catch |err| {
         if (err == error.InvalidNodeType) {
             return server.sendError(id, .InvalidParams, "Node is not an element");
         }