Close #142. Only attempt to set width/height on altair widget if the attributes actually exist.

Author: cpsievert

CHANGELOG.md

@@ -8,6 +8,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## [UNRELEASED]
 
 * Fixed a bug with multiple altair outputs not working inside a `@shiny.render.ui` decorator. (#140)
+* `@render_widget` no longer errors out when giving a `altair.FacetChart` class. (#142)
 
 ## [0.3.1] - 2024-03-01
 

shinywidgets/_render_widget_base.py

@@ -210,9 +210,9 @@ def set_layout_defaults(widget: Widget) -> Tuple[Widget, bool]:
                 )
             else:
                 UndefinedType = alt.utils.schemapi.UndefinedType  # type: ignore
-                if isinstance(chart.width, UndefinedType):  # type: ignore[reportMissingTypeStubs]
+                if hasattr(chart, "width") and isinstance(chart.width, UndefinedType):  # type: ignore[reportMissingTypeStubs]
                     chart = chart.properties(width="container")  # type: ignore
-                if isinstance(chart.height, UndefinedType):  # type: ignore[reportMissingTypeStubs]
+                if hasattr(chart, "height") and isinstance(chart.height, UndefinedType):  # type: ignore[reportMissingTypeStubs]
                     chart = chart.properties(height="container")  # type: ignore
             widget.chart = chart