Refactor IconButton variants and update tests

Refactored IconButton control to support filled, filled tonal, and outlined variants with dedicated constructors. Updated Python integration tests and golden image filenames to match new variant naming. Cleaned up API usage and removed unnecessary parameters and code in both Dart and Python implementations.

Author: FeodorFitsner

packages/flet/lib/src/controls/icon_button.dart

@@ -68,7 +68,6 @@ class _IconButtonControlState extends State<IconButtonControl>
       var highlightColor = widget.control.getColor("highlight_color", context);
       var selectedIconColor =
           widget.control.getColor("selected_icon_color", context);
-      var bgcolor = widget.control.getColor("bgcolor", context);
       var disabledColor = widget.control.getColor("disabled_color", context);
       var hoverColor = widget.control.getColor("hover_color", context);
       var splashColor = widget.control.getColor("splash_color", context);
@@ -82,15 +81,13 @@ class _IconButtonControlState extends State<IconButtonControl>
       var autofocus = widget.control.getBool("autofocus", false)!;
       var enableFeedback = widget.control.getBool("enable_feedback", true)!;
       var selected = widget.control.getBool("selected", false)!;
-      var url = widget.control.getString("url");
-      var urlTarget = widget.control.getString("url_target");
       var mouseCursor = widget.control.getMouseCursor("mouse_cursor");
-      var visualDensity = widget.control.getVisualDensity("visual_density");
+      var url = widget.control.getUrl("url");
 
       Function()? onPressed = !widget.control.disabled
           ? () {
               if (url != null) {
-                openWebBrowser(url, webWindowName: urlTarget);
+                openWebBrowser(url);
               }
               widget.control.triggerEvent("click");
             }
@@ -109,7 +106,7 @@ class _IconButtonControlState extends State<IconButtonControl>
 
       var theme = Theme.of(context);
       var style = parseButtonStyle(
-          widget.control.get("style"), Theme.of(context),
+          widget.control.internals?["style"], Theme.of(context),
           defaultForegroundColor: theme.colorScheme.primary,
           defaultBackgroundColor: Colors.transparent,
           defaultOverlayColor: Colors.transparent,
@@ -126,28 +123,74 @@ class _IconButtonControlState extends State<IconButtonControl>
       if (icon is Control) {
         iconWidget = ControlWidget(control: icon);
       } else if (icon is int) {
-        // Icon values are stored as raw integers (set_id << 16 | index) in this codebase.
-        iconWidget = Icon(
-          widget.control.getIconData("icon"),
-          color: iconColor,
-        );
+        iconWidget = Icon(widget.control.getIconData("icon"), color: iconColor);
       } else if (content != null) {
         iconWidget = ControlWidget(control: content);
       }
 
       Widget? selectedIconWidget;
-
       if (selectedIcon is Control) {
         selectedIconWidget = ControlWidget(control: selectedIcon);
       } else if (selectedIcon is int) {
-        selectedIconWidget = Icon(
-          widget.control.getIconData("selected_icon"),
-          color: selectedIconColor,
-        );
+        selectedIconWidget = Icon(widget.control.getIconData("selected_icon"),
+            color: selectedIconColor);
       }
 
-      if (iconWidget != null) {
-        button = IconButton(
+      if (iconWidget == null) {
+        return const ErrorControl(
+            "IconButton must have either icon or a visible content specified.");
+      }
+
+      var variant = widget.control.type;
+
+      if (variant == "FilledIconButton") {
+        button = IconButton.filled(
+            autofocus: autofocus,
+            focusNode: _focusNode,
+            highlightColor: highlightColor,
+            disabledColor: disabledColor,
+            hoverColor: hoverColor,
+            enableFeedback: enableFeedback,
+            padding: padding,
+            alignment: alignment,
+            focusColor: focusColor,
+            splashColor: splashColor,
+            splashRadius: splashRadius,
+            icon: iconWidget,
+            iconSize: iconSize,
+            mouseCursor: mouseCursor,
+            style: style,
+            isSelected: selected,
+            constraints: sizeConstraints,
+            onLongPress: onLongPressHandler,
+            onHover: onHoverHandler,
+            selectedIcon: selectedIconWidget,
+            onPressed: onPressed);
+      } else if (variant == "FilledTonalIconButton") {
+        button = IconButton.filledTonal(
+            autofocus: autofocus,
+            focusNode: _focusNode,
+            highlightColor: highlightColor,
+            disabledColor: disabledColor,
+            hoverColor: hoverColor,
+            enableFeedback: enableFeedback,
+            padding: padding,
+            alignment: alignment,
+            focusColor: focusColor,
+            splashColor: splashColor,
+            splashRadius: splashRadius,
+            icon: iconWidget,
+            iconSize: iconSize,
+            mouseCursor: mouseCursor,
+            style: style,
+            isSelected: selected,
+            constraints: sizeConstraints,
+            onLongPress: onLongPressHandler,
+            onHover: onHoverHandler,
+            selectedIcon: selectedIconWidget,
+            onPressed: onPressed);
+      } else if (variant == "OutlinedIconButton") {
+        button = IconButton.outlined(
             autofocus: autofocus,
             focusNode: _focusNode,
             highlightColor: highlightColor,
@@ -162,7 +205,6 @@ class _IconButtonControlState extends State<IconButtonControl>
             icon: iconWidget,
             iconSize: iconSize,
             mouseCursor: mouseCursor,
-            visualDensity: visualDensity,
             style: style,
             isSelected: selected,
             constraints: sizeConstraints,
@@ -171,16 +213,28 @@ class _IconButtonControlState extends State<IconButtonControl>
             selectedIcon: selectedIconWidget,
             onPressed: onPressed);
       } else {
-        return const ErrorControl(
-            "IconButton must have either icon or a visible content specified.");
-      }
-
-      if (bgcolor != null) {
-        button = Container(
-          decoration:
-              ShapeDecoration(color: bgcolor, shape: const CircleBorder()),
-          child: button,
-        );
+        button = IconButton(
+            autofocus: autofocus,
+            focusNode: _focusNode,
+            highlightColor: highlightColor,
+            disabledColor: disabledColor,
+            hoverColor: hoverColor,
+            enableFeedback: enableFeedback,
+            padding: padding,
+            alignment: alignment,
+            focusColor: focusColor,
+            splashColor: splashColor,
+            splashRadius: splashRadius,
+            icon: iconWidget,
+            iconSize: iconSize,
+            mouseCursor: mouseCursor,
+            style: style,
+            isSelected: selected,
+            constraints: sizeConstraints,
+            onLongPress: onLongPressHandler,
+            onHover: onHoverHandler,
+            selectedIcon: selectedIconWidget,
+            onPressed: onPressed);
       }
 
       return ConstrainedControl(control: widget.control, child: button);

sdk/python/packages/flet/integration_tests/controls/golden/macos/icon_button/filled_icon_button.png renamed to sdk/python/packages/flet/integration_tests/controls/golden/macos/icon_button/filled.png

@@ -9,7 +9,7 @@ async def test_button_1(flet_app: ftt.FletTestApp, request):
     flet_app.page.theme_mode = ft.ThemeMode.LIGHT
     await flet_app.assert_control_screenshot(
         request.node.name,
-        control=ft.Button("Click me"),
+        ft.Button("Click me"),
     )
 
 
@@ -18,5 +18,5 @@ async def test_button_2(flet_app: ftt.FletTestApp, request):
     flet_app.page.theme_mode = ft.ThemeMode.LIGHT
     await flet_app.assert_control_screenshot(
         request.node.name,
-        control=ft.Button("Something else!"),
+        ft.Button("Something else!"),
     )

sdk/python/packages/flet/integration_tests/controls/golden/macos/icon_button/filled_tonal_icon_button.png renamed to sdk/python/packages/flet/integration_tests/controls/golden/macos/icon_button/filled_tonal.png

@@ -5,36 +5,36 @@
 
 
 @pytest.mark.asyncio(loop_scope="module")
-async def test_icon_button(flet_app: ftt.FletTestApp, request):
+async def test_standard(flet_app: ftt.FletTestApp, request):
     flet_app.page.theme_mode = ft.ThemeMode.LIGHT
     await flet_app.assert_control_screenshot(
         request.node.name,
-        control=ft.IconButton(ft.Icons.HOME),
+        ft.IconButton(ft.Icons.HOME),
     )
 
 
 @pytest.mark.asyncio(loop_scope="module")
-async def test_filled_icon_button(flet_app: ftt.FletTestApp, request):
+async def test_filled(flet_app: ftt.FletTestApp, request):
     flet_app.page.theme_mode = ft.ThemeMode.LIGHT
     await flet_app.assert_control_screenshot(
         request.node.name,
-        control=ft.FilledIconButton(ft.Icons.HOME),
+        ft.FilledIconButton(ft.Icons.HOME),
     )
 
 
 @pytest.mark.asyncio(loop_scope="module")
-async def test_filled_tonal_icon_button(flet_app: ftt.FletTestApp, request):
+async def test_filled_tonal(flet_app: ftt.FletTestApp, request):
     flet_app.page.theme_mode = ft.ThemeMode.LIGHT
     await flet_app.assert_control_screenshot(
         request.node.name,
-        control=ft.FilledTonalIconButton(ft.Icons.HOME),
+        ft.FilledTonalIconButton(ft.Icons.HOME),
     )
 
 
 @pytest.mark.asyncio(loop_scope="module")
-async def test_outlined_icon_button(flet_app: ftt.FletTestApp, request):
+async def test_outlined(flet_app: ftt.FletTestApp, request):
     flet_app.page.theme_mode = ft.ThemeMode.LIGHT
     await flet_app.assert_control_screenshot(
         request.node.name,
-        control=ft.OutlinedIconButton(ft.Icons.HOME),
+        ft.OutlinedIconButton(ft.Icons.HOME),
     )

sdk/python/packages/flet/integration_tests/controls/golden/macos/icon_button/icon_button_basic.png

@@ -9,7 +9,7 @@ async def test_basic_text(flet_app: ftt.FletTestApp, request):
     flet_app.page.theme_mode = ft.ThemeMode.LIGHT
     await flet_app.assert_control_screenshot(
         request.node.name,
-        control=ft.Text("Hello, world!"),
+        ft.Text("Hello, world!"),
     )
 
 
@@ -18,5 +18,5 @@ async def test_bold_text(flet_app: ftt.FletTestApp, request):
     flet_app.page.theme_mode = ft.ThemeMode.LIGHT
     await flet_app.assert_control_screenshot(
         request.node.name,
-        control=ft.Text("Hello, world!", weight=ft.FontWeight.BOLD),
+        ft.Text("Hello, world!", weight=ft.FontWeight.BOLD),
     )

sdk/python/packages/flet/integration_tests/controls/golden/macos/icon_button/outlined_icon_button.png renamed to sdk/python/packages/flet/integration_tests/controls/golden/macos/icon_button/outlined.png

@@ -9,5 +9,5 @@ async def test_basic_textfield(flet_app: ftt.FletTestApp, request):
     flet_app.page.theme_mode = ft.ThemeMode.LIGHT
     await flet_app.assert_control_screenshot(
         request.node.name,
-        control=ft.TextField(),
+        ft.TextField(),
     )