bug(data frame filter): When the filter is reset, be sure to use a ""…

… value for the input (#1557)

CHANGELOG.md

@@ -11,6 +11,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Other changes
 
+### Bug fixes
+
+* Fixed bug where calling `.update_filter(None)` on a data frame renderer did not visually reset non-numeric column filters. (It did reset the column's filtering, just not the label). Now it resets filter's label. (#1557)
+
 * Require shinyswatch >= 0.7.0 and updated examples accordingly. (#1558)
 
 ### Bug fixes

js/data-frame/filter.tsx

@@ -94,7 +94,10 @@ export const Filter: FC<FilterProps> = ({ header, className, ...props }) => {
   return (
     <input
       {...props}
-      value={header.column.getFilterValue() as string}
+      // If there was a value and now there isn't,
+      // set the filter value to `""` and not `undefined`.
+      // `undefined` will not clear the displayed value.
+      value={(header.column.getFilterValue() as string) || ""}
       className={`form-control form-control-sm ${className}`}
       type="text"
       onChange={(e) => header.column.setFilterValue(e.target.value)}

tests/playwright/shiny/components/data_frame/filter_reset/app.py

@@ -0,0 +1,43 @@
+from typing import cast
+
+import pandas as pd
+from palmerpenguins import load_penguins  # pyright: ignore[reportMissingTypeStubs]
+
+from shiny import reactive
+from shiny.express import input, render, ui
+
+penguins = cast(pd.DataFrame, load_penguins())
+
+ui.input_action_button("update_filters", "Update filters")
+ui.input_action_button("reset_filters", "Reset filters")
+
+ui.h5("Current filters: ", {"class": "pt-2"})
+
+
+@render.code
+def penguins_code():
+    return str(penguins_df.filter())
+
+
+@render.data_frame
+def penguins_df():
+    return render.DataGrid(penguins, filters=True)
+
+
+@reactive.effect
+@reactive.event(input.update_filters)
+async def _():
+    await penguins_df.update_filter(
+        [
+            {"col": 0, "value": "Gentoo"},
+            {"col": 2, "value": (50, None)},
+            {"col": 3, "value": (None, 17)},
+            {"col": 4, "value": (220, 225)},
+        ],
+    )
+
+
+@reactive.effect
+@reactive.event(input.reset_filters)
+async def _():
+    await penguins_df.update_filter(None)  # <<

tests/playwright/shiny/components/data_frame/filter_reset/test_filter_reset.py

@@ -0,0 +1,43 @@
+from playwright.sync_api import Page, expect
+
+from shiny.playwright import controller
+from shiny.run import ShinyAppProc
+
+
+def test_filters_are_reset(page: Page, local_app: ShinyAppProc) -> None:
+    page.goto(local_app.url)
+
+    penguin_df = controller.OutputDataFrame(page, "penguins_df")
+    penguin_code = controller.OutputCode(page, "penguins_code")
+    update_filters = controller.InputActionButton(page, "update_filters")
+    reset_filters = controller.InputActionButton(page, "reset_filters")
+
+    filter_inputs = penguin_df.loc_column_filter.locator("input")
+
+    expect(filter_inputs).to_have_count(8 + 5)  # 8 columns including 5 numeric columns
+
+    penguin_code.expect_value("()")
+    for element in filter_inputs.element_handles():
+        assert element.input_value() == ""
+
+    update_filters.click()
+
+    penguin_code.expect_value(
+        "("
+        "{'col': 0, 'value': 'Gentoo'}, "
+        "{'col': 2, 'value': (50, None)}, "
+        "{'col': 3, 'value': (None, 17)}, "
+        "{'col': 4, 'value': (220, 225)}"
+        ")"
+    )
+    for value, element in zip(
+        ["Gentoo", "", "50", "", "", "17", "220", "225", "", "", "", "", ""],
+        filter_inputs.element_handles(),
+    ):
+        assert element.input_value() == value
+
+    reset_filters.click()
+
+    penguin_code.expect_value("()")
+    for element in filter_inputs.element_handles():
+        assert element.input_value() == ""