reactpy.html(...) to be shorthand notation for fragments

Author: Archmonger

docs/source/about/changelog.rst

@@ -79,8 +79,8 @@ Unreleased
 - :pull:`1311` - Removed deprecated exception type ``reactpy.core.serve.Stop``.
 - :pull:`1311` - Removed deprecated component ``reactpy.widgets.hotswap``.
 - :pull:`1255` - Removed ``reactpy.sample`` module.
-- :pull:`1255` - Removed ``reactpy.svg`` module. Contents previously within ``reactpy.svg.*`` can now be accessed via ``html.svg.*``.
-- :pull:`1255` - Removed ``reactpy.html._`` function. Use ``html.fragment`` instead.
+- :pull:`1255` - Removed ``reactpy.svg`` module. Contents previously within ``reactpy.svg.*`` can now be accessed via ``reactpy.html.svg.*``.
+- :pull:`1255` - Removed ``reactpy.html._`` function. Use ``reactpy.html(...)`` or ``reactpy.html.fragment(...)`` instead.
 - :pull:`1113` - Removed ``reactpy.run``. See the documentation for the new method to run ReactPy applications.
 - :pull:`1113` - Removed ``reactpy.backend.*``. See the documentation for the new method to run ReactPy applications.
 - :pull:`1113` - Removed ``reactpy.core.types`` module. Use ``reactpy.types`` instead.

src/reactpy/_html.py

@@ -287,6 +287,7 @@ class HtmlConstructor:
         "fragment": Vdom("", custom_constructor=_fragment),
         "svg": SvgConstructor(),
     }
+    __call__ = __cache__["fragment"].__call__
 
     def __getattr__(self, value: str) -> VdomConstructor:
         value = value.rstrip("_").replace("_", "-")

src/reactpy/utils.py

@@ -242,7 +242,7 @@ def component_to_vdom(component: Component) -> VdomDict:
         return component_to_vdom(cast(Component, result))
     elif isinstance(result, str):
         return html.div(result)
-    return html.fragment()
+    return html()
 
 
 def _react_attribute_to_html(key: str, value: Any) -> tuple[str, str]:

tests/test_client.py

@@ -17,7 +17,7 @@ async def test_automatic_reconnect(
     @reactpy.component
     def SomeComponent():
         count, incr_count = use_counter(0)
-        return reactpy.html.fragment(
+        return reactpy.html(
             reactpy.html.p({"data-count": count, "id": "count"}, "count", count),
             reactpy.html.button(
                 {"onClick": lambda e: incr_count(), "id": "incr"}, "incr"

tests/test_core/test_hooks.py

@@ -981,7 +981,7 @@ async def test_context_values_are_scoped():
 
     @reactpy.component
     def Parent():
-        return html.fragment(
+        return html(
             Context(Context(Child1(), value=1), value="something-else"),
             Context(Child2(), value=2),
         )

tests/test_core/test_layout.py

@@ -1225,7 +1225,7 @@ def colorize(event):
     @component
     def App():
         items = use_state(["A", "B", "C"])
-        return html.fragment([Item(item, items, key=item) for item in items.value])
+        return html([Item(item, items, key=item) for item in items.value])
 
     async with layout_runner(Layout(App())) as runner:
         tree = await runner.render()
@@ -1266,7 +1266,7 @@ async def test_async_renders(async_rendering):
 
     @component
     def outer():
-        return html.fragment(child_1(), child_2())
+        return html(child_1(), child_2())
 
     @component
     @child_1_hook.capture

tests/test_html.py

@@ -104,13 +104,13 @@ def test_script_has_no_event_handlers():
 
 
 def test_simple_fragment():
-    assert html.fragment() == {"tagName": ""}
-    assert html.fragment(1, 2, 3) == {"tagName": "", "children": [1, 2, 3]}
-    assert html.fragment({"key": "something"}) == {
+    assert html() == {"tagName": ""}
+    assert html(1, 2, 3) == {"tagName": "", "children": [1, 2, 3]}
+    assert html({"key": "something"}) == {
         "tagName": "",
         "attributes": {"key": "something"},
     }
-    assert html.fragment({"key": "something"}, 1, 2, 3) == {
+    assert html({"key": "something"}, 1, 2, 3) == {
         "tagName": "",
         "attributes": {"key": "something"},
         "children": [1, 2, 3],
@@ -119,7 +119,7 @@ def test_simple_fragment():
 
 def test_fragment_can_have_no_attributes():
     with pytest.raises(TypeError, match=r"Fragments cannot have attributes"):
-        html.fragment({"someAttribute": 1})
+        html({"someAttribute": 1})
 
 
 async def test_svg(display: DisplayFixture):

tests/test_utils.py

@@ -325,11 +325,11 @@ def example_none_return():
             "<button></button>",
         ),
         (
-            html.fragment("hello ", html.fragment("world")),
+            html("hello ", html("world")),
             "hello world",
         ),
         (
-            html.fragment(html.div("hello"), html.fragment("world")),
+            html(html.div("hello"), html("world")),
             "<div>hello</div>world",
         ),
         (
@@ -341,15 +341,15 @@ def example_none_return():
             '<div style="background-color:blue;margin-left:10px"></div>',
         ),
         (
-            html.fragment(
+            html(
                 html.div("hello"),
                 html.a({"href": "https://example.com"}, "example"),
             ),
             '<div>hello</div><a href="https://example.com">example</a>',
         ),
         (
             html.div(
-                html.fragment(
+                html(
                     html.div("hello"),
                     html.a({"href": "https://example.com"}, "example"),
                 ),