Fix filter number validation with trailing zeroes

rust/perspective-viewer/src/rust/components/column_selector/filter_column.rs

@@ -53,9 +53,8 @@ pub struct FilterColumnProps {
 }
 
 impl PartialEq for FilterColumnProps {
-    fn eq(&self, _rhs: &Self) -> bool {
-        false
-        // self.idx == other.idx && self.filter == other.filter
+    fn eq(&self, rhs: &Self) -> bool {
+        self.idx == rhs.idx && self.filter == rhs.filter && self.on_keydown == rhs.on_keydown
     }
 }
 
@@ -219,13 +218,15 @@ impl FilterColumnProps {
         };
 
         if let Some(input) = filter_input {
-            filter_column.2 = input;
-            let update = ViewConfigUpdate {
-                filter: Some(filter),
-                ..ViewConfigUpdate::default()
-            };
+            if input != filter_column.2 {
+                filter_column.2 = input;
+                let update = ViewConfigUpdate {
+                    filter: Some(filter),
+                    ..ViewConfigUpdate::default()
+                };
 
-            ApiFuture::spawn(self.update_and_render(update));
+                ApiFuture::spawn(self.update_and_render(update));
+            }
         }
     }
 }

rust/perspective-viewer/test/js/regressions.spec.js

@@ -10,8 +10,11 @@
 // ┃ of the [Apache License 2.0](https://www.apache.org/licenses/LICENSE-2.0). ┃
 // ┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛
 
-import { test } from "@playwright/test";
-import { compareContentsToSnapshot } from "@finos/perspective-test";
+import { test, expect } from "@playwright/test";
+import {
+    compareContentsToSnapshot,
+    shadow_type,
+} from "@finos/perspective-test";
 
 async function get_contents(page) {
     return await page.evaluate(async () => {
@@ -57,4 +60,37 @@ test.describe("Regression tests", () => {
             "regressions-not_in-filter-works-correctly.txt",
         ]);
     });
+
+    test("Numeric filter input does not trigger render on trailing zeroes", async ({
+        page,
+    }) => {
+        await page.evaluate(async () => {
+            const viewer = document.querySelector("perspective-viewer");
+            await viewer.restore({
+                filter: [["Sales", ">", 1.1]],
+                settings: true,
+            });
+        });
+
+        // await new Promise((x) => setTimeout(x, 10000));
+
+        await shadow_type(
+            page,
+            "0001",
+            true,
+            "perspective-viewer",
+            "input.num-filter"
+        );
+
+        const value = await page.evaluate(async () => {
+            const viewer = document.querySelector("perspective-viewer");
+            return viewer.shadowRoot.querySelector("input.num-filter").value;
+        });
+
+        expect(value).toEqual("1.10001");
+        const contents = await get_contents(page);
+        await compareContentsToSnapshot(contents, [
+            "numeric-filter-input-does-not-trigger-render-on-trailing-zeroes.txt",
+        ]);
+    });
 });