From ff288c8aa2ef2c8e12a1d3b099f5f5059562a6eb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Adri=C3=A0=20Arrufat?= <adria.arrufat@gmail.com>
Date: Wed, 18 Mar 2026 12:04:53 +0900
Subject: [PATCH] browser.interactive: use for-else expression in role checks

---
 src/browser/interactive.zig | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/src/browser/interactive.zig b/src/browser/interactive.zig
index 04a0d492..94d6b799 100644
--- a/src/browser/interactive.zig
+++ b/src/browser/interactive.zig
@@ -261,10 +261,9 @@ pub fn isInteractiveRole(role: []const u8) bool {
         "combobox",         "option",        "treeitem",   "textbox",
         "listbox",          "iframe",
     };
-    for (interactive_roles) |r| {
-        if (std.ascii.eqlIgnoreCase(role, r)) return true;
-    }
-    return false;
+    return for (interactive_roles) |r| {
+        if (std.ascii.eqlIgnoreCase(role, r)) break true;
+    } else false;
 }
 
 pub fn isContentRole(role: []const u8) bool {
@@ -274,10 +273,9 @@ pub fn isContentRole(role: []const u8) bool {
         "article",      "region",    "main",
         "navigation",
     };
-    for (content_roles) |r| {
-        if (std.ascii.eqlIgnoreCase(role, r)) return true;
-    }
-    return false;
+    return for (content_roles) |r| {
+        if (std.ascii.eqlIgnoreCase(role, r)) break true;
+    } else false;
 }
 
 fn getRole(el: *Element) ?[]const u8 {