Merge branch 'main' into bookmarking

* main:
  fix(textarea-autoresize): add a bit of padding when setting textarea height (#1891)
  Properly send custom message in a synchronous manner (#1888)
  fix(page-main-container): Fillable container with min-height/width only when parent layout is fillable (#1885)
  Add `.latest_stream` to `ui.MarkdownStream()` and deprecate `.get_latest_stream_result()` (#1884)

Author: schloerke

CHANGELOG.md

@@ -5,6 +5,18 @@ All notable changes to Shiny for Python will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [UNRELEASED]
+
+### Changes
+
+* The `.get_latest_stream_result()` method on `ui.MarkdownStream()` was deprecated in favor of the new `.latest_stream` property. Call `.result()` on the property to get the latest result, `.status` to check the status, and `.cancel()` to cancel the stream.
+
+### Bug fixes
+
+* Fixed an issue where the `<main>` areas of `ui.page_sidebar()` and `ui.page_navbar()` (with a `sidebar`) were made to be a fillable containers even when `fillable=False`. (#1816)
+
+* Fixed an issue where the `.update_user_input()` method on `ui.Chat()` isn't working in shinylive.  (#1891)
+
 ## [1.3.0] - 2025-03-03
 
 ### New features

js/chat/chat.scss

@@ -32,7 +32,7 @@ shiny-chat-container {
     text-underline-offset: 4px;
     text-decoration-thickness: 2px;
 
-    padding: 2px;
+    padding-inline: 2px;
 
     &:hover {
       text-decoration-style: solid;

js/text-area/textarea-autoresize.ts

@@ -49,7 +49,7 @@ function update_height(target: HTMLTextAreaElement) {
   if (target.scrollHeight > 0) {
     // Automatically resize the textarea to fit its content.
     target.style.height = "auto";
-    target.style.height = target.scrollHeight + "px";
+    target.style.height = target.scrollHeight + 2 + "px";
   } else {
     // The textarea is not visible on the page, therefore it has a 0 scroll height.
 

scripts/_pkg-sources.R

@@ -1,5 +1,5 @@
 list(
-  bslib = "rstudio/bslib@main",
+  bslib = "rstudio/bslib@fix/page-main-container-fillable",
   shiny = "rstudio/shiny@main",
   sass = "sass",
   htmltools = "rstudio/htmltools@main"

shiny/ui/_chat.py

@@ -1107,16 +1107,13 @@ def update_user_input(
             }
         )
 
-        _utils.run_coro_sync(
-            self._session.send_custom_message(
-                "shinyChatMessage",
-                {
-                    "id": self.id,
-                    "handler": "shiny-chat-update-user-input",
-                    "obj": obj,
-                },
-            )
-        )
+        msg = {
+            "id": self.id,
+            "handler": "shiny-chat-update-user-input",
+            "obj": obj,
+        }
+
+        self._session._send_message_sync({"custom": {"shinyChatMessage": msg}})
 
     def set_user_message(self, value: str):
         """

shiny/ui/_markdown_stream.py

@@ -4,6 +4,7 @@
 from htmltools import css
 
 from .. import _utils, reactive
+from .._deprecated import warn_deprecated
 from .._docstring import add_example
 from .._namespaces import resolve_id
 from .._typing_extensions import TypedDict
@@ -87,9 +88,14 @@ def __init__(
         self.on_error = on_error
 
         with session_context(self._session):
-            self._latest_stream: reactive.Value[
-                Union[reactive.ExtendedTask[[], str], None]
-            ] = reactive.Value(None)
+
+            @reactive.extended_task
+            async def _mock_task() -> str:
+                return ""
+
+            self._latest_stream: reactive.Value[reactive.ExtendedTask[[], str]] = (
+                reactive.Value(_mock_task)
+            )
 
     async def stream(
         self,
@@ -151,32 +157,46 @@ async def _handle_error():
 
         return _task
 
-    def get_latest_stream_result(self) -> Union[str, None]:
+    @property
+    def latest_stream(self):
         """
-        Reactively read the latest stream result.
+        React to changes in the latest stream.
+
+        Reactively reads for the :class:`~shiny.reactive.ExtendedTask` behind the
+        latest stream.
 
-        This method reads a reactive value containing the result of the latest
-        `.stream()`. Therefore, this method must be called in a reactive context (e.g.,
-        a render function, a :func:`~shiny.reactive.calc`, or a
-        :func:`~shiny.reactive.effect`).
+        From the return value (i.e., the extended task), you can then:
+
+        1. Reactively read for the final `.result()`.
+        2. `.cancel()` the stream.
+        3. Check the `.status()` of the stream.
 
         Returns
         -------
         :
-            The result of the latest stream (a string).
+            An extended task that represents the streaming task. The `.result()` method
+            of the task can be called in a reactive context to get the final state of the
+            stream.
 
-        Raises
-        ------
-        :
-            A silent exception if no stream has completed yet.
+        Note
+        ----
+        If no stream has yet been started when this method is called, then it returns an
+        extended task with `.status()` of `"initial"` and that it status doesn't change
+        state until a message is streamed.
         """
-        stream = self._latest_stream()
-        if stream is None:
-            from .. import req
+        return self._latest_stream()
 
-            req(False)
-        else:
-            return stream.result()
+    def get_latest_stream_result(self) -> Union[str, None]:
+        """
+        Reactively read the latest stream result.
+
+        Deprecated. Use `latest_stream.result()` instead.
+        """
+        warn_deprecated(
+            "The `.get_latest_stream_result()` method is deprecated and will be removed "
+            "in a future release. Use `.latest_stream.result()` instead. "
+        )
+        self.latest_stream.result()
 
     async def clear(self):
         """

shiny/ui/_navs.py

@@ -1365,7 +1365,9 @@ def layout(self, nav: Tag, content: Tag) -> TagList:
                 # TODO-future: This could also be applied to the non-sidebar page layout above
                 from ._page import page_main_container
 
-                tab_content = page_main_container(*contents)
+                tab_content = page_main_container(
+                    *contents, fillable=self.fillable is not False
+                )
 
             content_div = div(
                 # In the fluid case, the sidebar layout should be flush (i.e.,

shiny/ui/_page.py

@@ -134,7 +134,7 @@ def page_sidebar(
         navbar_title,
         layout_sidebar(
             sidebar,
-            page_main_container(*children),
+            page_main_container(*children, fillable=fillable),
             attrs,
             fillable=fillable,
             border=False,
@@ -149,8 +149,10 @@ def page_sidebar(
     )
 
 
-def page_main_container(*args: TagChild) -> Tag:
+def page_main_container(*args: TagChild, fillable: bool = True) -> Tag:
     main = tags.main({"class": "bslib-page-main bslib-gap-spacing"}, *args)
+    if not fillable:
+        return main
     return as_fillable_container(as_fill_item(main))