Tabulator: sorters are no longer automatically applied on cell edits

examples/reference/widgets/Tabulator.ipynb

@@ -78,7 +78,7 @@
     "\n",
     "##### Properties\n",
     "\n",
-    "* **``current_view``** (``DataFrame``): The current view of the table that is displayed, i.e. after sorting and filtering are applied\n",
+    "* **``current_view``** (``DataFrame``): The current view of the table that is displayed, i.e. after sorting and filtering are applied. `current_view` isn't guaranteed to be in sync with the displayed current view when sorters are applied and values are edited, in which case `current_view` is sorted while the displayed table isn't.\n",
     "* **``selected_dataframe``** (``DataFrame``): A DataFrame reflecting the currently selected rows.\n",
     "\n",
     "##### Callbacks\n",

panel/models/tabulator.ts

@@ -1106,7 +1106,6 @@ export class DataTabulatorView extends PanelHTMLBoxView {
       this._tabulator_cell_updating = false
     }
     this.model.trigger_event(new TableEditEvent(field, index))
-    this.setSorters()
     this.tabulator.scrollToRow(index, "top", false)
   }
 }

panel/tests/ui/widgets/test_tabulator.py

@@ -1,3 +1,5 @@
+from __future__ import annotations
+
 import datetime as dt
 import time
 
@@ -117,6 +119,16 @@ def count_per_page(count: int, page_size: int):
     return count_per_page
 
 
+def tabulator_column_values(page, col_name: str) -> list[str]:
+    """Get the values of a column.
+
+    >>> tabulator_column_values(page, 'color')
+    ['blue', 'red']
+    """
+    cells = page.locator(f'[tabulator-field={col_name}][role=gridcell]')
+    return cells.all_inner_texts()
+
+
 def test_tabulator_default(page, port, df_mixed, df_mixed_as_string):
     nrows, ncols = df_mixed.shape
     widget = Tabulator(df_mixed)
@@ -2503,6 +2515,35 @@ def test_tabulator_sorters_pagination(page, port, df_mixed, pagination):
     wait_until(page, lambda: widget.current_view.equals(expected_sorted_df))
 
 
+def test_tabulator_edit_event_sorters_not_automatically_applied(page, port, df_mixed):
+    widget = Tabulator(df_mixed, sorters=[{'field': 'str', 'dir': 'desc'}])
+
+    values = []
+    widget.on_edit(lambda e: values.append((e.column, e.row, e.old, e.value)))
+
+    serve(widget, port=port, threaded=True, show=False)
+
+    time.sleep(0.2)
+
+    page.goto(f"http://localhost:{port}")
+
+    expected_vals = list(df_mixed['str'].sort_values(ascending=False))
+
+    wait_until(page, lambda: tabulator_column_values(page, 'str') == expected_vals)
+
+    # Chankge the cell that contains B to BB
+    cell = page.locator('text="B"')
+    cell.click()
+    editable_cell = page.locator('input[type="text"]')
+    editable_cell.fill("Z")
+    editable_cell.press('Enter')
+
+    wait_until(page, lambda: len(values) == 1)
+
+    expected_vals = [item if item != 'B' else 'Z' for item in expected_vals]
+    wait_until(page, lambda: tabulator_column_values(page, 'str') == expected_vals)
+
+
 @pytest.mark.xfail(reason='See https://github.com/holoviz/panel/issues/3660')
 def test_tabulator_edit_event_and_header_filters(page, port):
     df = pd.DataFrame({