Merge pull request #26 from machow/fix-column-later-desc-sort

fix: column default_sort_order respected when resorting

Author: machow

reactable/models.py

@@ -696,6 +696,9 @@ class Column:
     footer_style: CssRules | None = None
     id: str | None = None
 
+    # props ----
+    default_sort_desc: bool = field(init=False)
+
     # internal ----
     # TODO: ideally this cannot be specified in the constructor
     # it's just passed to the widget
@@ -714,6 +717,8 @@ def __post_init__(self):
                 aggregated=self.format.to_props(),
             )
 
+        self.default_sort_desc = self.default_sort_order == "desc"
+
     def _apply_transform(self, col_data: list[Any], transform: callable):
         return [
             to_hydrate_format(transform(CellInfo(val, ii, self.id)))
@@ -790,7 +795,9 @@ def merge(self, other: Column | None):
         if other is None:
             return self
 
-        field_attrs = {field.name: getattr(self, field.name) for field in fields(self)}
+        field_attrs = {
+            field.name: getattr(self, field.name) for field in fields(self) if field.init
+        }
         return replace(other, **filter_none(field_attrs))
 
     def to_props(self) -> dict[str, Any]: