tksheet 7.4.4__py3-none-any.whl → 7.4.5__py3-none-any.whl

tksheet/sheet.py

@@ -4,6 +4,7 @@ import tkinter as tk
 from bisect import bisect_left
 from collections import deque
 from collections.abc import Callable, Generator, Hashable, Iterator, Sequence
+from contextlib import suppress
 from itertools import accumulate, chain, filterfalse, islice, product, repeat
 from operator import attrgetter
 from timeit import default_timer
@@ -123,14 +124,14 @@ class Sheet(tk.Frame):
         header_align: str = "n",
         row_index_align: str | None = None,
         index_align: str = "n",
-        displayed_columns: list[int] = [],
+        displayed_columns: list[int] | None = None,
         all_columns_displayed: bool = True,
-        displayed_rows: list[int] = [],
+        displayed_rows: list[int] | None = None,
         all_rows_displayed: bool = True,
         to_clipboard_delimiter: str = "\t",
         to_clipboard_quotechar: str = '"',
         to_clipboard_lineterminator: str = "\n",
-        from_clipboard_delimiters: list[str] | str = ["\t"],
+        from_clipboard_delimiters: list[str] | str = "\t",
         show_default_header_for_empty: bool = True,
         show_default_index_for_empty: bool = True,
         page_up_down_select_row: bool = True,
@@ -734,10 +735,14 @@ class Sheet(tk.Frame):
         - "rc_delete_row", "end_rc_delete_row", "end_delete_rows", "delete_rows"
         - "begin_rc_delete_column", "begin_delete_columns"
         - "rc_delete_column", "end_rc_delete_column","end_delete_columns", "delete_columns"
-        - "begin_rc_insert_column", "begin_insert_column", "begin_insert_columns", "begin_add_column","begin_rc_add_column", "begin_add_columns"
-        - "rc_insert_column", "end_rc_insert_column", "end_insert_column", "end_insert_columns", "rc_add_column", "end_rc_add_column", "end_add_column", "end_add_columns", "add_columns"
-        - "begin_rc_insert_row", "begin_insert_row", "begin_insert_rows", "begin_rc_add_row", "begin_add_row", "begin_add_rows"
-        - "rc_insert_row", "end_rc_insert_row", "end_insert_row", "end_insert_rows", "rc_add_row", "end_rc_add_row", "end_add_row", "end_add_rows", "add_rows"
+        - "begin_rc_insert_column", "begin_insert_column", "begin_insert_columns", "begin_add_column",
+          "begin_rc_add_column", "begin_add_columns"
+        - "rc_insert_column", "end_rc_insert_column", "end_insert_column", "end_insert_columns", "rc_add_column",
+          "end_rc_add_column", "end_add_column", "end_add_columns", "add_columns"
+        - "begin_rc_insert_row", "begin_insert_row", "begin_insert_rows", "begin_rc_add_row", "begin_add_row",
+          "begin_add_rows"
+        - "rc_insert_row", "end_rc_insert_row", "end_insert_row", "end_insert_rows", "rc_add_row", "end_rc_add_row",
+          "end_add_row", "end_add_rows", "add_rows"
         - "row_height_resize"
         - "column_width_resize"
         - "cell_select"
@@ -1613,10 +1618,7 @@ class Sheet(tk.Frame):
                 res = list(chain.from_iterable(res))
         elif span.ndim == 1:
             # flatten sublists
-            if len(res) == 1 and len(res[0]) == 1:
-                res = res[0]
-            else:
-                res = list(chain.from_iterable(res))
+            res = res[0] if len(res) == 1 and len(res[0]) == 1 else list(chain.from_iterable(res))
         # if span.ndim == 2 res keeps its current
         # dimensions as a list of lists
         if span.convert is not None:
@@ -2208,10 +2210,7 @@ class Sheet(tk.Frame):
                         c_ops=False,
                     )
         numrows = len(data)
-        if self.MT.all_rows_displayed:
-            displayed_ins_idx = idx
-        else:
-            displayed_ins_idx = bisect_left(self.MT.displayed_rows, idx)
+        displayed_ins_idx = idx if self.MT.all_rows_displayed else bisect_left(self.MT.displayed_rows, idx)
         event_data = self.MT.add_rows(
             *self.MT.get_args_for_add_rows(
                 data_ins_row=idx,
@@ -2599,7 +2598,8 @@ class Sheet(tk.Frame):
         """
         Sort the data within specified box regions in the table.
 
-        This method sorts the data within one or multiple box regions defined by their coordinates. Each box's columns are sorted independently.
+        This method sorts the data within one or multiple box regions defined by their coordinates.
+        Each box's columns are sorted independently.
 
         Args:
             boxes (CreateSpanTypes): A type that can create a Span.
@@ -2617,7 +2617,8 @@ class Sheet(tk.Frame):
                 Note: Performance might be slow with the natural sort for very large datasets.
 
         Returns:
-            EventDataDict: A dictionary containing information about the sorting event, including changes made to the table.
+            EventDataDict: A dictionary containing information about the sorting event,
+            including changes made to the table.
 
         Raises:
             ValueError: If the input boxes are not in the expected format or if the data cannot be sorted.
@@ -2758,9 +2759,9 @@ class Sheet(tk.Frame):
     def dropdown(
         self,
         *key: CreateSpanTypes,
-        values: list = [],
+        values: list[object] | None = None,
         edit_data: bool = True,
-        set_values: dict[tuple[int, int], object] = {},
+        set_values: dict[tuple[int, int], object] | None = None,
         set_value: object = None,
         state: Literal["normal", "readonly", "disabled"] = "normal",
         redraw: bool = True,
@@ -2770,6 +2771,10 @@ class Sheet(tk.Frame):
         validate_input: bool = True,
         text: None | str = None,
     ) -> Span:
+        if values is None:
+            values = []
+        if set_values is None:
+            set_values = {}
         if not search_function:
             search_function = dropdown_search_function
         v = set_value if set_value is not None else values[0] if values else ""
@@ -2794,13 +2799,13 @@ class Sheet(tk.Frame):
                 self.del_index_cell_options_dropdown_and_checkbox(r)
                 add_to_options(self.RI.cell_options, r, "dropdown", d)
                 if edit_data:
-                    set_idata(r, value=set_values[r] if r in set_values else v)
+                    set_idata(r, value=set_values.get(r, v))
         if header:
             for c in cols:
                 self.del_header_cell_options_dropdown_and_checkbox(c)
                 add_to_options(self.CH.cell_options, c, "dropdown", d)
                 if edit_data:
-                    set_hdata(c, value=set_values[c] if c in set_values else v)
+                    set_hdata(c, value=set_values.get(c, v))
         if table:
             if span.kind == "cell":
                 for r in rows:
@@ -2808,21 +2813,21 @@ class Sheet(tk.Frame):
                         self.del_cell_options_dropdown_and_checkbox(r, c)
                         add_to_options(self.MT.cell_options, (r, c), "dropdown", d)
                         if edit_data:
-                            set_tdata(r, c, value=set_values[(r, c)] if (r, c) in set_values else v)
+                            set_tdata(r, c, value=set_values.get((r, c), v))
             elif span.kind == "row":
                 for r in rows:
                     self.del_row_options_dropdown_and_checkbox(r)
                     add_to_options(self.MT.row_options, r, "dropdown", d)
                     if edit_data:
                         for c in cols:
-                            set_tdata(r, c, value=set_values[(r, c)] if (r, c) in set_values else v)
+                            set_tdata(r, c, value=set_values.get((r, c), v))
             elif span.kind == "column":
                 for c in cols:
                     self.del_column_options_dropdown_and_checkbox(c)
                     add_to_options(self.MT.col_options, c, "dropdown", d)
                     if edit_data:
                         for r in rows:
-                            set_tdata(r, c, value=set_values[(r, c)] if (r, c) in set_values else v)
+                            set_tdata(r, c, value=set_values.get((r, c), v))
         self.set_refresh_timer(redraw)
         return span
 
@@ -3002,11 +3007,13 @@ class Sheet(tk.Frame):
     def format(
         self,
         *key: CreateSpanTypes,
-        formatter_options: dict = {},
+        formatter_options: dict | None = None,
         formatter_class: object = None,
         redraw: bool = True,
         **kwargs,
     ) -> Span:
+        if formatter_options is None:
+            formatter_options = {}
         span = self.span_from_key(*key)
         rows, cols = self.ranges_from_span(span)
         kwargs = fix_format_kwargs({"formatter": formatter_class, **formatter_options, **kwargs})
@@ -4034,7 +4041,7 @@ class Sheet(tk.Frame):
         column: int = 0,
         keep_yscroll: bool = False,
         keep_xscroll: bool = False,
-        bottom_right_corner: bool = False,
+        bottom_right_corner: bool | None = None,
         check_cell_visibility: bool = True,
         redraw: bool = True,
     ) -> Sheet:
@@ -4053,7 +4060,11 @@ class Sheet(tk.Frame):
         return self.MT.cell_visible(r, c)
 
     def cell_completely_visible(self, r: int, c: int, seperate_axes: bool = False) -> bool:
-        return self.MT.cell_completely_visible(r, c, seperate_axes)
+        if seperate_axes:
+            d = self.MT.cell_visibility_info(r, c)
+            return d["yvis"], d["xvis"]
+        else:
+            return self.MT.cell_completely_visible(r, c)
 
     def set_xview(self, position: None | float = None, option: str = "moveto") -> Sheet | tuple[float, float]:
         if position is not None:
@@ -4507,18 +4518,14 @@ class Sheet(tk.Frame):
     def unbind_key_text_editor(self, key: str) -> Sheet:
         if key == "all":
             for key in self.MT.text_editor_user_bound_keys:
-                try:
+                with suppress(Exception):
                     self.MT.text_editor.tktext.unbind(key)
-                except Exception:
-                    pass
             self.MT.text_editor_user_bound_keys = {}
         else:
             if key in self.MT.text_editor_user_bound_keys:
                 del self.MT.text_editor_user_bound_keys[key]
-            try:
+            with suppress(Exception):
                 self.MT.text_editor.tktext.unbind(key)
-            except Exception:
-                pass
         return self
 
     def get_text_editor_value(self) -> str | None:
@@ -4977,9 +4984,9 @@ class Sheet(tk.Frame):
             columns=set(),
         )
         for tag in unpack(tags):
-            res.cells.update(self.MT.tagged_cells[tag] if tag in self.MT.tagged_cells else set())
-            res.rows.update(self.MT.tagged_rows[tag] if tag in self.MT.tagged_rows else set())
-            res.columns.update(self.MT.tagged_columns[tag] if tag in self.MT.tagged_columns else set())
+            res.cells.update(self.MT.tagged_cells.get(tag, set()))
+            res.rows.update(self.MT.tagged_rows.get(tag, set()))
+            res.columns.update(self.MT.tagged_columns.get(tag, set()))
         return res
 
     # Treeview Mode
@@ -5073,10 +5080,7 @@ class Sheet(tk.Frame):
         """
         If used without args all items are opened
         """
-        if items := set(unpack(items)):
-            to_open = self._tree_open(items)
-        else:
-            to_open = self._tree_open(set(self.get_children()))
+        to_open = self._tree_open(items) if (items := set(unpack(items))) else self._tree_open(set(self.get_children()))
         return self.show_rows(
             rows=to_open,
             redraw=redraw,
@@ -5103,10 +5107,7 @@ class Sheet(tk.Frame):
         """
         If used without args all items are closed
         """
-        if items:
-            to_close = self._tree_close(unpack(items))
-        else:
-            to_close = self._tree_close(self.get_children())
+        to_close = self._tree_close(unpack(items)) if items else self._tree_close(self.get_children())
         return self.hide_rows(
             rows=to_close,
             redraw=redraw,
@@ -5164,13 +5165,10 @@ class Sheet(tk.Frame):
                     self.RI.tree_rns[parent]
                     + index
                     + 1
-                    + sum(
-                        sum(1 for _ in self.RI.get_iid_descendants(cid))
-                        for cid in islice(self.get_children(parent), index)
-                    )
+                    + sum(self.RI.num_descendants(cid) for cid in islice(self.get_children(parent), index))
                 )
             else:
-                datarn = self.RI.tree_rns[parent] + sum(1 for _ in self.RI.get_iid_descendants(parent)) + 1
+                datarn = self.RI.tree_rns[parent] + self.RI.num_descendants(parent) + 1
         else:
             if isinstance(index, int):
                 datarn = index
@@ -5206,10 +5204,7 @@ class Sheet(tk.Frame):
         datarn = self._get_id_insert_row(index=index, parent=parent)
         rns_to_add = {}
         for rn, r in enumerate(data, start=datarn):
-            if iid_column is None:
-                iid = self.RI.new_iid()
-            else:
-                iid = r[iid_column]
+            iid = self.RI.new_iid() if iid_column is None else r[iid_column]
             new_node = Node(
                 r[text_column] if isinstance(text_column, int) else text_column if isinstance(text_column, str) else "",
                 iid,
@@ -5302,8 +5297,8 @@ class Sheet(tk.Frame):
     def itemrow(self, item: str) -> int:
         try:
             return self.RI.tree_rns[item]
-        except Exception:
-            raise ValueError(f"item '{item}' does not exist.")
+        except ValueError as error:
+            raise ValueError(f"item '{item}' does not exist.") from error
 
     def rowitem(self, row: int, data_index: bool = False) -> str | None:
         try:
@@ -5524,7 +5519,7 @@ class Sheet(tk.Frame):
         return self.set_refresh_timer(redraw)
 
     def selection_toggle(self, *items, redraw: bool = True) -> Sheet:
-        selected = set(self.MT._row_index[self.displayed_row_to_data(rn)].iid for rn in self.get_selected_rows())
+        selected = {self.MT._row_index[self.displayed_row_to_data(rn)].iid for rn in self.get_selected_rows()}
         add = []
         remove = []
         for item in unpack(items):
@@ -5892,14 +5887,12 @@ class Sheet(tk.Frame):
 
     def readonly_rows(
         self,
-        rows: list | int = [],
+        rows: list[int] | int,
         readonly: bool = True,
         redraw: bool = False,
     ) -> Sheet:
         if isinstance(rows, int):
             rows = [rows]
-        else:
-            rows = rows
         if not readonly:
             for r in rows:
                 if r in self.MT.row_options and "readonly" in self.MT.row_options[r]:
@@ -5913,14 +5906,12 @@ class Sheet(tk.Frame):
 
     def readonly_columns(
         self,
-        columns: list | int = [],
+        columns: list[int] | int,
         readonly: bool = True,
         redraw: bool = False,
     ) -> Sheet:
         if isinstance(columns, int):
             columns = [columns]
-        else:
-            columns = columns
         if not readonly:
             for c in columns:
                 if c in self.MT.col_options and "readonly" in self.MT.col_options[c]:
@@ -5936,7 +5927,7 @@ class Sheet(tk.Frame):
         self,
         row: int = 0,
         column: int = 0,
-        cells: list = [],
+        cells: list[tuple[int, int]] | None = None,
         readonly: bool = True,
         redraw: bool = False,
     ) -> Sheet:
@@ -5965,7 +5956,7 @@ class Sheet(tk.Frame):
 
     def readonly_header(
         self,
-        columns: list = [],
+        columns: list[int],
         readonly: bool = True,
         redraw: bool = False,
     ) -> Sheet:
@@ -5974,7 +5965,7 @@ class Sheet(tk.Frame):
 
     def readonly_index(
         self,
-        rows: list = [],
+        rows: list[int],
         readonly: bool = True,
         redraw: bool = False,
     ) -> Sheet:
@@ -5983,7 +5974,7 @@ class Sheet(tk.Frame):
 
     def dehighlight_rows(
         self,
-        rows: list[int] | Literal["all"] = [],
+        rows: list[int] | Literal["all"],
         redraw: bool = True,
     ) -> Sheet:
         if isinstance(rows, int):
@@ -5998,19 +5989,15 @@ class Sheet(tk.Frame):
                     del self.RI.cell_options[r]["highlight"]
         else:
             for r in rows:
-                try:
+                with suppress(Exception):
                     del self.MT.row_options[r]["highlight"]
-                except Exception:
-                    pass
-                try:
+                with suppress(Exception):
                     del self.RI.cell_options[r]["highlight"]
-                except Exception:
-                    pass
         return self.set_refresh_timer(redraw)
 
     def dehighlight_columns(
         self,
-        columns: list[int] | Literal["all"] = [],
+        columns: list[int] | Literal["all"],
         redraw: bool = True,
     ) -> Sheet:
         if isinstance(columns, int):
@@ -6025,14 +6012,10 @@ class Sheet(tk.Frame):
                     del self.CH.cell_options[c]["highlight"]
         else:
             for c in columns:
-                try:
+                with suppress(Exception):
                     del self.MT.col_options[c]["highlight"]
-                except Exception:
-                    pass
-                try:
+                with suppress(Exception):
                     del self.CH.cell_options[c]["highlight"]
-                except Exception:
-                    pass
         return self.set_refresh_timer(redraw)
 
     def highlight_rows(
@@ -6074,7 +6057,7 @@ class Sheet(tk.Frame):
         self,
         row: int | Literal["all"] = 0,
         column: int | Literal["all"] = 0,
-        cells: list[tuple[int, int]] = [],
+        cells: list[tuple[int, int]] | None = None,
         canvas: Literal["table", "index", "header"] = "table",
         bg: bool | None | str = False,
         fg: bool | None | str = False,
@@ -6128,7 +6111,7 @@ class Sheet(tk.Frame):
         self,
         row: int | Literal["all"] = 0,
         column: int = 0,
-        cells: list[tuple[int, int]] = [],
+        cells: list[tuple[int, int]] | None = None,
         canvas: Literal["table", "row_index", "index", "header"] = "table",
         all_: bool = False,
         redraw: bool = True,
@@ -6207,7 +6190,7 @@ class Sheet(tk.Frame):
         self,
         row: int = 0,
         column: int = 0,
-        cells: list = [],
+        cells: list[tuple[int, int]] | dict[tuple[int, int], str] | None = None,
         align: str | None = "global",
         redraw: bool = True,
     ) -> Sheet:
@@ -6224,7 +6207,7 @@ class Sheet(tk.Frame):
 
     def align_rows(
         self,
-        rows: list | dict | int = [],
+        rows: list[int] | dict[int, str] | int,
         align: str | None = "global",
         align_index: bool = False,
         redraw: bool = True,
@@ -6249,7 +6232,7 @@ class Sheet(tk.Frame):
 
     def align_columns(
         self,
-        columns: list | dict | int = [],
+        columns: list[int] | dict[int, str] | int,
         align: str | None = "global",
         align_header: bool = False,
         redraw: bool = True,
@@ -6274,7 +6257,7 @@ class Sheet(tk.Frame):
 
     def align_header(
         self,
-        columns: list | dict | int = [],
+        columns: list[int] | dict[int, str] | int,
         align: str | None = "global",
         redraw: bool = True,
     ) -> Sheet:
@@ -6291,7 +6274,7 @@ class Sheet(tk.Frame):
 
     def align_index(
         self,
-        rows: list | dict | int = [],
+        rows: list[int] | dict[int, str] | int,
         align: str | None = "global",
         redraw: bool = True,
     ) -> Sheet:
@@ -6457,11 +6440,11 @@ class Sheet(tk.Frame):
         c: int | Literal["all"] = 0,
     ) -> None:
         if isinstance(r, str) and r.lower() == "all" and isinstance(c, int):
-            for r_, c_ in self.MT.cell_options:
+            for r_, _ in self.MT.cell_options:
                 if "checkbox" in self.MT.cell_options[(r_, c)]:
                     self.del_cell_options_checkbox(r_, c)
         elif isinstance(c, str) and c.lower() == "all" and isinstance(r, int):
-            for r_, c_ in self.MT.cell_options:
+            for _, c_ in self.MT.cell_options:
                 if "checkbox" in self.MT.cell_options[(r, c_)]:
                     self.del_cell_options_checkbox(r, c_)
         elif isinstance(r, str) and r.lower() == "all" and isinstance(c, str) and c.lower() == "all":
@@ -6712,11 +6695,11 @@ class Sheet(tk.Frame):
         c: int | Literal["all"] = 0,
     ) -> None:
         if isinstance(r, str) and r.lower() == "all" and isinstance(c, int):
-            for r_, c_ in self.MT.cell_options:
+            for r_, _ in self.MT.cell_options:
                 if "dropdown" in self.MT.cell_options[(r_, c)]:
                     self.del_cell_options_dropdown(r_, c)
         elif isinstance(c, str) and c.lower() == "all" and isinstance(r, int):
-            for r_, c_ in self.MT.cell_options:
+            for _, c_ in self.MT.cell_options:
                 if "dropdown" in self.MT.cell_options[(r, c_)]:
                     self.del_cell_options_dropdown(r, c_)
         elif isinstance(r, str) and r.lower() == "all" and isinstance(c, str) and c.lower() == "all":
@@ -6810,9 +6793,11 @@ class Sheet(tk.Frame):
         r: int = 0,
         c: int = 0,
         set_existing_dropdown: bool = False,
-        values: list[object] = [],
+        values: list[object] | None = None,
         set_value: object = None,
     ) -> Sheet:
+        if values is None:
+            values = []
         if set_existing_dropdown:
             if self.MT.dropdown.open:
                 r_, c_ = self.MT.dropdown.get_coords()
@@ -6835,9 +6820,11 @@ class Sheet(tk.Frame):
         self,
         c: int = 0,
         set_existing_dropdown: bool = False,
-        values: list[object] = [],
+        values: list[object] | None = None,
         set_value: object = None,
     ) -> Sheet:
+        if values is None:
+            values = []
         if set_existing_dropdown:
             if self.CH.dropdown.open:
                 c_ = self.CH.dropdown.get_coords()
@@ -6859,9 +6846,11 @@ class Sheet(tk.Frame):
         self,
         r: int = 0,
         set_existing_dropdown: bool = False,
-        values: list[object] = [],
+        values: list[object] | None = None,
         set_value: object = None,
     ) -> Sheet:
+        if values is None:
+            values = []
         if set_existing_dropdown:
             if self.RI.current_dropdown_window is not None:
                 r_ = self.RI.current_dropdown_window.r
@@ -6950,11 +6939,13 @@ class Sheet(tk.Frame):
         self,
         r: int | Literal["all"],
         c: int | Literal["all"],
-        formatter_options: dict = {},
+        formatter_options: dict | None = None,
         formatter_class: object = None,
         redraw: bool = True,
         **kwargs,
     ) -> Sheet:
+        if formatter_options is None:
+            formatter_options = {}
         kwargs = fix_format_kwargs({"formatter": formatter_class, **formatter_options, **kwargs})
         if isinstance(r, str) and r.lower() == "all" and isinstance(c, int):
             for r_ in range(self.MT.total_data_rows()):
@@ -6983,11 +6974,11 @@ class Sheet(tk.Frame):
         clear_values: bool = False,
     ) -> Sheet:
         if isinstance(r, str) and r.lower() == "all" and isinstance(c, int):
-            for r_, c_ in self.MT.cell_options:
+            for r_, _ in self.MT.cell_options:
                 if "format" in self.MT.cell_options[(r_, c)]:
                     self.MT.delete_cell_format(r_, c, clear_values=clear_values)
         elif isinstance(c, str) and c.lower() == "all" and isinstance(r, int):
-            for r_, c_ in self.MT.cell_options:
+            for _, c_ in self.MT.cell_options:
                 if "format" in self.MT.cell_options[(r, c_)]:
                     self.MT.delete_cell_format(r, c_, clear_values=clear_values)
         elif isinstance(r, str) and r.lower() == "all" and isinstance(c, str) and c.lower() == "all":
@@ -7001,11 +6992,13 @@ class Sheet(tk.Frame):
     def format_row(
         self,
         r: AnyIter[int] | int | Literal["all"],
-        formatter_options: dict = {},
+        formatter_options: dict | None = None,
         formatter_class: object = None,
         redraw: bool = True,
         **kwargs,
     ) -> Sheet:
+        if formatter_options is None:
+            formatter_options = {}
         kwargs = fix_format_kwargs({"formatter": formatter_class, **formatter_options, **kwargs})
         if isinstance(r, str) and r.lower() == "all":
             for r_ in range(len(self.MT.data)):
@@ -7043,11 +7036,13 @@ class Sheet(tk.Frame):
     def format_column(
         self,
         c: AnyIter[int] | int | Literal["all"],
-        formatter_options: dict = {},
+        formatter_options: dict | None = None,
         formatter_class: object = None,
         redraw: bool = True,
         **kwargs,
     ) -> Sheet:
+        if formatter_options is None:
+            formatter_options = {}
         kwargs = fix_format_kwargs({"formatter": formatter_class, **formatter_options, **kwargs})
         if isinstance(c, str) and c.lower() == "all":
             for c_ in range(self.MT.total_data_cols()):
@@ -7099,7 +7094,7 @@ class Dropdown(Sheet):
         height: int | None = None,
         font: None | tuple[str, int, str] = None,
         outline_thickness: int = 2,
-        values: list[object] = [],
+        values: list[object] | None = None,
         close_dropdown_window: Callable | None = None,
         search_function: Callable = dropdown_search_function,
         modified_function: None | Callable = None,
@@ -7157,7 +7152,7 @@ class Dropdown(Sheet):
             ops=ops,
             outline_color=outline_color,
             align=align,
-            values=values,
+            values=[] if values is None else values,
             search_function=search_function,
             modified_function=modified_function,
         )
@@ -7245,10 +7240,7 @@ class Dropdown(Sheet):
             row = None
         elif event.keysym == "Return":
             row = self.get_selected_rows()
-            if not row:
-                row = None
-            else:
-                row = next(iter(row))
+            row = None if not row else next(iter(row))
         else:
             row = self.identify_row(event, exclude_index=True, allow_end=False)
         if self.single_index:
@@ -7274,10 +7266,12 @@ class Dropdown(Sheet):
 
     def values(
         self,
-        values: list = [],
+        values: list[object] | None = None,
         redraw: bool = True,
         width: int | None = None,
     ) -> None:
+        if values is None:
+            values = []
         self.set_sheet_data(
             [[v] for v in values],
             reset_col_positions=False,

tksheet/sorting.py

@@ -67,7 +67,7 @@ def _string_fallback(item: str) -> tuple[int, ...]:
             5,
             len(components),
             tuple(int(e) if e.isdigit() else e.lower() for comp in components[:-1] for e in split(r"(\d+)", comp) if e),
-            tuple(),
+            (),
         )
 
 
@@ -117,7 +117,7 @@ def version_sort_key(item: object) -> tuple[int, ...]:
                         for e in split(r"(\d+)", comp)
                         if e
                     ),
-                    tuple(),
+                    (),
                 )
         else:
             return (1, item)
@@ -195,7 +195,7 @@ def natural_sort_key(item: object) -> tuple[int, ...]:
                             for e in split(r"(\d+)", comp)
                             if e
                         ),
-                        tuple(),
+                        (),
                     )
         else:
             return (1, item)
@@ -393,7 +393,7 @@ def sort_columns_by_row(
         unsorted_part = [elem for idx, elem in enumerate(row_data) if idx not in sort_indices_set]
         new_data.append(sorted_part + unsorted_part)
 
-    return sort_indices, {old: new for old, new in zip(range(len(data[row])), sort_indices)}
+    return sort_indices, dict(zip(range(len(data[row])), sort_indices))
 
 
 def _sort_node_children(

tksheet/text_editor.py

@@ -111,9 +111,15 @@ class TextEditorTkText(tk.Text):
         ):
             self.tag_add("align", 1.0, "end")
             self.event_generate("<<TextModified>>")
-            if args and len(args) > 1 and args[1] != "\n" and args != ("1.0", "end"):
-                if self.yview() != (0.0, 1.0) and self.newline_bindng is not None:
-                    self.newline_bindng(check_lines=False)
+            if (
+                args
+                and len(args) > 1
+                and args[1] != "\n"
+                and args != ("1.0", "end")
+                and self.yview() != (0.0, 1.0)
+                and self.newline_bindng is not None
+            ):
+                self.newline_bindng(check_lines=False)
         return result
 
     def rc(self, event: object) -> None: