style string can have a final semicolon

Author: mwouts

docs/changelog.md

@@ -13,6 +13,7 @@ ITables ChangeLog
 
 **Fixed**
 - We have added type hints to `itable.options` even for the options that don't have a default value ([#224](https://github.com/mwouts/itables/issues/224))
+- The optional final semicolon in `style` argument is now supported again ([#386](https://github.com/mwouts/itables/issues/386))
 
 
 2.4.0 (2025-05-17)

src/itables/javascript.py

@@ -117,7 +117,7 @@ def get_expanded_style(style: Union[str, Mapping[str, str]]) -> dict[str, str]:
     elif isinstance(style, str):
         return {
             k.strip(): v.strip()
-            for k, v in (item.split(":") for item in style.split(";"))
+            for k, v in (item.split(":") for item in style.split(";") if item.strip())
         }
     else:
         raise TypeError(f"style must be a string or a dict, not {type(style)}")

tests/test_javascript.py

@@ -12,6 +12,7 @@
     check_table_id,
     get_compact_classes,
     get_compact_style,
+    get_expanded_style,
     get_itable_arguments,
     replace_value,
     to_html_datatable,
@@ -153,3 +154,16 @@ def test_unpkg_urls_are_up_to_date():
         UNPKG_DT_BUNDLE_CSS
         == f"https://www.unpkg.com/dt_for_itables@{bundle_version}/dt_bundle.css"
     )
+
+
+@pytest.mark.parametrize(
+    "style",
+    [
+        "width: fit-content; float: left;",
+        "width: fit-content; float: left; overflow: auto; max-height: 500px",
+    ],
+)
+def test_get_expanded_style(style: str):
+    exp_style = get_expanded_style(style)
+    compact_style = get_compact_style(exp_style)
+    assert compact_style == style.replace(" ", "").removesuffix(";")