ssb-dash-framework 0.2.2__tar.gz → 0.2.2a0__tar.gz

{ssb_dash_framework-0.2.2 → ssb_dash_framework-0.2.2a0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ssb-dash-framework
-Version: 0.2.2
+Version: 0.2.2a0
 Summary: SSB Dash Framework
 License-Expression: GPL-3.0
 License-File: LICENSE

{ssb_dash_framework-0.2.2 → ssb_dash_framework-0.2.2a0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "ssb-dash-framework"
-version = "0.2.2"
+version = "0.2.2a0"
 description = "SSB Dash Framework"
 authors = [{ name = "Magnus Theodor Engh", email = "dor@ssb.no" }]
 license = "GPL-3.0"

{ssb_dash_framework-0.2.2 → ssb_dash_framework-0.2.2a0}/src/ssb_dash_framework/__init__.py

@@ -60,7 +60,6 @@ from .modules import VisualizationBuilder
 from .modules import VisualizationBuilderWindow
 from .modules import apply_edits
 from .modules import export_from_parqueteditor
-from .modules import get_export_log_path
 from .modules import get_log_path
 from .setup import VariableSelector
 from .setup import VariableSelectorOption
@@ -81,7 +80,6 @@ from .utils import create_database
 from .utils import create_database_engine
 from .utils import enable_app_logging
 from .utils import hb_method
-from .utils import ibis_filter_with_dict
 from .utils import module_validator
 from .utils import sidebar_button
 
@@ -166,9 +164,7 @@ __all__ = [
     "create_database_engine",
     "enable_app_logging",
     "export_from_parqueteditor",
-    "get_export_log_path",
     "get_log_path",
-    "ibis_filter_with_dict",
     "main_layout",
     "module_validator",
     "register_control",

{ssb_dash_framework-0.2.2 → ssb_dash_framework-0.2.2a0}/src/ssb_dash_framework/assets/dashAgGridFunctions.js

@@ -155,12 +155,11 @@ window.dashAgGridFunctions.MacroModule = {
 
     // === Function displayDiffHighlight (Compare current cell's value to previous year, highlight if different) ===
     displayDiffHighlight(props) {
-
-    if (!props.data) return {};
-
     const field = props.colDef.field;
     const value = props.value;
 
+    if (!props.data) return {};
+
     const normalize = (v) =>
         v !== null && v !== undefined ? String(v).trim() : "";
 
@@ -191,6 +190,14 @@ window.dashAgGridFunctions.MacroModule = {
         return {}; // no change
     }
 
+    // all other columns
+    if (field.endsWith("_x")) {
+        return {
+            backgroundColor: '#e8e9eb',
+            color: 'black'
+        };
+    }
+
     const prevField = field + "_x";
     const prevValue = normalize(props.data[prevField]);
     const currentValue = normalize(value);
@@ -221,40 +228,6 @@ window.dashAgGridFunctions.MacroModule = {
         const prefixF = String(naringF).substring(0, 2);
 
         return prefixB !== prefixF;
-    },
-
-    // === Function displayDiffColumnHighlight (Mark tilgang & avgang on the diff-column) ===
-    displayDiffColumnHighlight(props) {
-        if (!props.data) return {};
-
-        const field = props.colDef.field;
-
-        const isDiffColumn = field.endsWith("_diff");
-        const isExitHighlightField =
-            (field === "orgnr_b" || field === "navn") &&
-            props.data.is_avgang &&
-            props.data.is_exiter;
-
-        // Guard: only apply to allowed columns
-        if (!isDiffColumn && !isExitHighlightField) return {};
-
-        // tilgang (still only for diff columns)
-        if (isDiffColumn && props.data.is_tilgang) {
-            return {
-                backgroundColor: "#c7f5c7",
-                color: "black",
-            };
-        }
-
-        // avgang (diff columns OR exiter name/orgnr)
-        if (props.data.is_avgang) {
-            return {
-                backgroundColor: "#ffc7c7",
-                color: "black",
-            };
-        }
-
-        return {};
     }
 
 };

{ssb_dash_framework-0.2.2 → ssb_dash_framework-0.2.2a0}/src/ssb_dash_framework/assets/stylesheet.css

@@ -230,19 +230,6 @@ Altinn Editor History
 }
 
 
-/*
-Altinn Editor Support Tables
-*/
-.hjelpetabeller-modal .modal-dialog {
-    max-width: 90vw !important;
-    width: 90vw !important;
-}
-
-.hjelpetabeller-modal .modal-content {
-    height: 70vh !important;
-    overflow: hidden;
-}
-
 /*
 MacroModule
 */

{ssb_dash_framework-0.2.2 → ssb_dash_framework-0.2.2a0}/src/ssb_dash_framework/control/control_framework_base.py

@@ -227,7 +227,8 @@ class ControlFrameworkBase:  # TODO: Add some common control methods here for ea
             self.conn.insert("kontroller", rows_to_register)
         elif conn_is_ibis(self.conn):
             conn = self.conn
-            conn.insert("kontroller", rows_to_register)
+            k = conn.table("kontroller")  # type: ignore[attr-defined]
+            k.insert(rows_to_register)
         else:
             raise NotImplementedError(
                 f"Connection type '{type(self.conn)}' is currently not implemented."
@@ -323,6 +324,7 @@ class ControlFrameworkBase:  # TODO: Add some common control methods here for ea
         """
         logger.info(f"Running control: {control}")
         results = getattr(self, control)()
+
         logger.debug("Starting validation of results.")
         if not isinstance(results, pd.DataFrame):
             raise TypeError(
@@ -346,14 +348,10 @@ class ControlFrameworkBase:  # TODO: Add some common control methods here for ea
             raise ValueError(
                 f"Error when running {control}. Could not find {results['kontrollid'].unique()} among registered controls. Valid options retrieved from the 'kontrollutslag' table: {self.controls}"
             )
-
-        # IMPORTANT! This code must not fix the error for the user.
-        # This check exists because it is expected that users mistakenly forget to filter on periods / form type
-        # and that forgetting that filtering WILL cause false positives/negatives. Do not change this validation check.
         for key, value in self.applies_to_subset.items():
             if len(results[key].unique()) != 1:
                 raise ValueError(
-                    f"Results from control '{control}' has too many unique values for '{key}'. Expected '{value[0]}'. Received: '{results[key].unique()}'\nYou "
+                    f"Results from control {control} has too many unique values for '{key}'. Expected '{value[0]}'. Received: '{results[key].unique()}'"
                 )
             if not value[0] == results[key].unique()[0]:
                 raise ValueError(
@@ -440,10 +438,7 @@ class ControlFrameworkBase:  # TODO: Add some common control methods here for ea
         else:
             specific_control = None
         existing_kontrollutslag = self.get_current_kontrollutslag(specific_control)
-        if existing_kontrollutslag is not None and not existing_kontrollutslag.empty:
-            logger.debug(
-                f"Merging new results with existing results.\nNew:{control_results.shape}\n{control_results.head(10)}\n\nExisting:{existing_kontrollutslag.shape}\n{existing_kontrollutslag.head(10)}"
-            )
+        if existing_kontrollutslag is not None:
             control_results = control_results.merge(
                 existing_kontrollutslag,
                 on=[*self.applies_to_subset.keys(), "kontrollid", "ident", "refnr"],
@@ -457,7 +452,7 @@ class ControlFrameworkBase:  # TODO: Add some common control methods here for ea
                         "kontrollid",
                         "ident",
                         "refnr",
-                        "verdi_x" if "verdi_x" in control_results.columns else "verdi",
+                        "verdi_x",
                         "utslag_x",
                     ]
                 ]
@@ -475,7 +470,8 @@ class ControlFrameworkBase:  # TODO: Add some common control methods here for ea
             self.conn.insert("kontrollutslag", control_results)
         elif conn_is_ibis(self.conn):
             conn = self.conn
-            conn.insert("kontrollutslag", control_results)
+            k = conn.table("kontrollutslag")  # type: ignore[attr-defined]
+            k.insert(control_results)
         else:
             raise NotImplementedError(
                 f"Connection type '{type(self.conn)}' is currently not implemented."
@@ -498,48 +494,25 @@ class ControlFrameworkBase:  # TODO: Add some common control methods here for ea
             logger.info("No existing rows found, ending here.")
             return None
         else:
-            logger.debug(
-                f"Merging new results with existing results.\nNew:{control_results.shape}\n{control_results.head(10)}\n\nExisting:{existing_kontrollutslag.shape}\n{existing_kontrollutslag.head(10)}"
-            )
             control_results = control_results.merge(
                 existing_kontrollutslag,
-                on=[*self.applies_to_subset.keys(), "kontrollid", "ident", "refnr"],
+                on=["kontrollid", "ident", "refnr"],
                 how="outer",
                 indicator=True,
-            )
-            logger.debug(f"{control_results}")
-            control_results = control_results.dropna(
-                subset=[col for col in control_results if col != "verdi"]
-            )  # TODO: Check if this causes unexpected behavior.
+            ).dropna()
         if control_results.empty:
-            logger.debug(f"Empty control_results:\n{control_results}")
             raise ValueError(
-                "Combined results from 'control_results' and 'existing_kontrollutslag' is empty when it shouldn't be."
+                "Combined results from 'control_results' and 'existing_kontrollutslag' is empty."
             )
         logger.debug(control_results)
         logger.debug(
             f"Utslag left:\n{control_results['utslag_x'].value_counts()}\nUtslag right:\n{control_results['utslag_y'].value_counts()}"
         )
-        changed = control_results[  # This might have issues.
+        changed = control_results[
             control_results["utslag_x"] != control_results["utslag_y"]
-        ][
-            [
-                *self.time_units,
-                "kontrollid",
-                "ident",
-                "refnr",
-                "verdi_x" if "verdi_x" in control_results.columns else "verdi",
-                "utslag_x",
-            ]
-        ].rename(
-            columns={"utslag_x": "utslag"}
+        ][["kontrollid", "ident", "refnr", "verdi_x", "utslag_x"]].rename(
+            columns={"utslag_x": "utslag", "verdi_x": "verdi"}
         )
-        if "verdi_x" in changed.columns:
-            changed = changed.rename(columns={"verdi_x": "verdi"})
-
-        missing = set(self._required_kontrollutslag_columns) - set(changed.columns)
-        if missing:
-            raise ValueError(f"Missing required columns: {missing}")
         if changed.empty:
             logger.info("No changed rows, ending here.")
             return None

{ssb_dash_framework-0.2.2 → ssb_dash_framework-0.2.2a0}/src/ssb_dash_framework/modules/__init__.py

@@ -52,7 +52,6 @@ from .parquet_editor import ParquetEditor
 from .parquet_editor import ParquetEditorChangelog
 from .parquet_editor import apply_edits
 from .parquet_editor import export_from_parqueteditor
-from .parquet_editor import get_export_log_path
 from .parquet_editor import get_log_path
 from .pi_memorizer import PimemorizerTab
 from .skjemapdfviewer import SkjemapdfViewer
@@ -120,6 +119,5 @@ __all__ = [
     "VisualizationBuilderWindow",
     "apply_edits",
     "export_from_parqueteditor",
-    "get_export_log_path",
     "get_log_path",
 ]

{ssb_dash_framework-0.2.2 → ssb_dash_framework-0.2.2a0}/src/ssb_dash_framework/modules/altinn_control_view.py

@@ -54,19 +54,15 @@ class ControlView(ABC):
             control_dict: A dictionary with one control class per skjema.
             conn: The eimerdb connection.
             outputs: Variable selector fields to output to. Defaults to ['ident']
-
-        Raises:
-            TypeError: if conn type is not 'EimerDBInstance' or ibis connection.
         """
         logger.warning(
             f"{self.__class__.__name__} is under development and may change in future releases."
         )
         if outputs is None:
             outputs = ["ident"]
-        if not conn_is_ibis(conn) and not isinstance(conn, EimerDBInstance):
-            raise TypeError(
-                f"Argument 'conn' must be an 'EimerDBInstance' or Ibis backend. Received: {type(conn)}"
-            )
+        assert hasattr(
+            conn, "query"
+        ), "The database connection object must have a 'query' method."
         self.module_number = ControlView._id_number
         self.module_name = self.__class__.__name__
         ControlView._id_number += 1
@@ -176,11 +172,13 @@ class ControlView(ABC):
             Input("var-altinnskjema", "value"),
             self.variableselector.get_all_inputs(),
         )
-        def kontroller_show_selected_controls(skjema: str | None, *args: Any):
+        def kontroller_show_selected_controls(skjema: str, *args: Any):
             logger.debug(f"Args:\nskjema: {skjema}\nargs: {args}")
             partition_args = dict(zip(self.time_units, args, strict=False))
             if partition_args is not None and skjema is not None:
-                valgte_vars = f"valgt partisjon: {partition_args}\nvalgt skjema: {skjema if skjema else 'Velg skjema for å starte'}"
+                valgte_vars = (
+                    f"valgt partisjon: {partition_args}\nvalgt skjema: {skjema}"
+                )
                 return valgte_vars
 
         @callback(
@@ -189,9 +187,7 @@ class ControlView(ABC):
             State("alert_store", "data"),
             prevent_initial_call=True,
         )
-        def alert_user_of_controls(
-            click: int | None, alert_store: list[dict[str, Any]]
-        ) -> list[dict[str, Any]]:
+        def alert_user_of_controls(click, alert_store):
             return [
                 create_alert(
                     "Kjører kontroller, dette kan ta litt tid, du får beskjed når den er ferdig. Ikke klikk på knappen igjen.",
@@ -213,11 +209,7 @@ class ControlView(ABC):
             prevent_initial_call=True,
         )
         def get_kontroller_overview(
-            skjema: str,
-            refresh: int | None,
-            rerun: int | None,
-            alert_store: list[dict[str, Any]],
-            *args: Any,
+            skjema: str, refresh: int | None, rerun: int | None, alert_store, *args: Any
         ):
             logger.debug(
                 f"Args:\n"
@@ -310,7 +302,7 @@ class ControlView(ABC):
             utslag = (
                 kontrollutslag.filter(kontrollutslag.utslag == True)
                 .group_by(kontrollutslag.kontrollid)
-                .aggregate(ant_utslag=ibis._.count())
+                .aggregate(ant_utslag=kontrollutslag.row_id.count())
             )
 
             subq = (
@@ -327,7 +319,7 @@ class ControlView(ABC):
                 )
                 .filter(kontrollutslag.utslag == True)
                 .group_by(kontrollutslag.kontrollid)
-                .aggregate(uediterte=ibis._.count())
+                .aggregate(uediterte=kontrollutslag.row_id.count())
                 .select("kontrollid", "uediterte")
             )
 

{ssb_dash_framework-0.2.2 → ssb_dash_framework-0.2.2a0}/src/ssb_dash_framework/modules/altinn_editor/altinn_editor_history.py

@@ -12,7 +12,6 @@ from dash.dependencies import State
 from dash.exceptions import PreventUpdate
 from eimerdb import EimerDBInstance
 from ibis import _
-import tzlocal
 
 from ssb_dash_framework.utils import conn_is_ibis
 
@@ -21,8 +20,6 @@ from ...utils.eimerdb_helpers import create_partition_select
 
 logger = logging.getLogger(__name__)
 
-local_tz = tzlocal.get_localzone()
-
 
 class AltinnEditorHistory:
     """Module for viewing the editing history for the selected observation."""
@@ -67,12 +64,10 @@ class AltinnEditorHistory:
                 dbc.ModalBody(
                     dag.AgGrid(
                         defaultColDef={
+                            "editable": True,
                             "filter": True,
                             "resizable": True,
                         },
-                        dashGridOptions={
-                            "enableCellTextSelection": True,
-                        },
                         id="skjemadata-historikkmodal-table1",
                         className="ag-theme-alpine header-style-on-filter",
                         columnSize="responsiveSizeToFit",
@@ -158,15 +153,11 @@ class AltinnEditorHistory:
                 if conn_is_ibis(self.conn):
                     conn = self.conn
                     t = conn.table("skjemadataendringshistorikk")
-                    df = t.filter(_.refnr == refnr).to_pandas()
-                    df["endret_tid"] = (
-                        pd.to_datetime(df["endret_tid"], utc=True)
-                        .dt.tz_convert(local_tz)
-                        .dt.floor("s")
-                        .dt.tz_localize(None)
-                        .dt.strftime("%Y-%m-%d %H:%M:%S")
+                    df = (
+                        t.filter(_.refnr == refnr)
+                        .order_by(_.endret_tid.cast("timestamp").desc())
+                        .to_pandas()
                     )
-                    df = df.sort_values("endret_tid", ascending=False)
                     columns = [
                         {
                             "headerName": col,

{ssb_dash_framework-0.2.2 → ssb_dash_framework-0.2.2a0}/src/ssb_dash_framework/modules/altinn_editor/altinn_editor_primary_table.py

@@ -119,41 +119,18 @@ class AltinnEditorPrimaryTable:
     def module_callbacks(self) -> None:
         """Defines the callbacks for the module."""
 
-        # check if var-bedrift exists
-        try:
-            self.variableselector.get_option("var-bedrift", search_target="id")
-            has_bedrift = True
-        except ValueError:
-            has_bedrift = False
-            logger.debug("var-bedrift not available, skipping bedrift sorting")
-
         @callback(  # type: ignore[misc]
             Output("altinnedit-table-skjemadata", "rowData", allow_duplicate=True),
             Output("altinnedit-table-skjemadata", "columnDefs", allow_duplicate=True),
             Input("altinnedit-refnr", "value"),
             Input("altinnedit-option1", "value"),
-            Input("var-ident", "value"),
             State("altinnedit-skjemaer", "value"),
-            (
-                State("var-bedrift", "value")
-                if has_bedrift
-                else State("altinnedit-refnr", "value")
-            ),  # Dummy state if no bedrift
             self.variableselector.get_all_states(),
             prevent_initial_call=True,
         )
         def hovedside_update_altinnskjema(
-            refnr: str,
-            tabell: str,
-            ident: str,
-            skjema: str,
-            bedrift_or_dummy: str,
-            *args: Any,
+            refnr: str, tabell: str, skjema: str, *args: Any
         ) -> tuple[list[dict[str, Any]] | None, list[dict[str, Any]] | None]:
-
-            # extract bedrift if it exists
-            bedrift = bedrift_or_dummy if has_bedrift else None
-
             logger.debug(
                 f"Args:\n"
                 f"refnr: {refnr}\n"
@@ -202,20 +179,12 @@ class AltinnEditorPrimaryTable:
                         f"partition_select:\n{create_partition_select(desired_partitions=self.time_units,skjema=skjema,**partition_args,)}"
                     )
 
-                    t = t.filter(_.refnr == refnr).join(d, "variabel", how="left")
+                    t = (
+                        t.filter(_.refnr == refnr)
+                        .join(d, "variabel", how="left")
+                        .order_by(_.radnr)
+                    )
                     t = t.filter(ibis_filter_with_dict(filter_dict))
-
-                    # sort by bedrift if available
-                    if bedrift and "ident" in t.columns:
-                        t = t.mutate(
-                            sort_priority=ibis.case()
-                            .when(_.ident == bedrift, 0)
-                            .else_(1)
-                            .end()
-                        ).order_by(["sort_priority", _.radnr])
-                    else:
-                        t = t.order_by(_.radnr)
-
                     df = t.drop(
                         [col for col in t.columns if col.endswith("_right")]
                         + ["datatype", "radnr", "tabell"]
@@ -256,13 +225,6 @@ class AltinnEditorPrimaryTable:
                         .to_pandas()
                     )
 
-                    # sort by bedrift if it exists
-                    if bedrift and "ident" in df.columns:
-                        df = df.sort_values(
-                            by="ident",
-                            key=lambda x: x.map(lambda v: 0 if v == bedrift else 1),
-                        )
-
                     columndefs = [
                         {
                             "headerName": col,
@@ -287,6 +249,35 @@ class AltinnEditorPrimaryTable:
                     )
                     return None, None
 
+        try:  # TODO Find better solution to sort - config file?
+            self.variableselector.get_option("var-bedrift", search_target="id")
+
+            @callback(
+                Output("altinnedit-table-skjemadata", "rowData"),
+                Output("altinnedit-table-skjemadata", "columnDefs"),
+                Input("altinnedit-table-skjemadata", "rowData"),
+                Input("altinnedit-table-skjemadata", "columnDefs"),
+                State("var-bedrift", "value"),
+                prevent_initial_call=True,
+            )
+            def sort_by_bedrift(
+                row_data: list[dict[str, Any]],
+                column_defs: list[dict[str, Any]],
+                bedrift: str,
+            ) -> tuple[list[dict[str, Any]], list[dict[str, Any]]]:
+                """Sort table to prioritize selected bedrift."""
+                if not bedrift or not row_data:
+                    return row_data, column_defs
+
+                sorted_data = sorted(
+                    row_data, key=lambda row: 0 if row.get("ident") == bedrift else 1
+                )
+
+                return sorted_data, column_defs
+
+        except ValueError:
+            logger.debug("var-bedrift not available, skipping bedrift sorting callback")
+
         @callback(  # type: ignore[misc]
             Output("var-statistikkvariabel", "value"),
             Input("altinnedit-table-skjemadata", "cellClicked"),
@@ -296,21 +287,9 @@ class AltinnEditorPrimaryTable:
             click: dict[str, Any], row_data: list[dict[str, Any]]
         ) -> str:
             logger.debug(f"Args:\nclick: {click}\nrow_data: {row_data}")
-
-            if not click or not row_data:
+            if not click:
                 raise PreventUpdate
-
-            columns = list(row_data[0].keys())
-
-            long_format = "variabel" in columns and "verdi" in columns
-            if long_format:
-                return str(row_data[click["rowIndex"]]["variabel"])
-
-            column = click.get("colId")  # wide format
-            if column in ("aar", "ident", "skjema", "refnr", "tabell"):
-                raise PreventUpdate
-
-            return str(column)
+            return str(row_data[click["rowIndex"]]["variabel"])
 
         @callback(  # type: ignore[misc]
             Output("alert_store", "data", allow_duplicate=True),

{ssb_dash_framework-0.2.2 → ssb_dash_framework-0.2.2a0}/src/ssb_dash_framework/modules/altinn_editor/altinn_editor_supporting_table.py

@@ -39,8 +39,6 @@ class AltinnSupportTable:
         get_data_func: Callable[..., pd.DataFrame],
         editor_inputs: list[str] | None = None,
         variableselector: VariableSelector | None = None,
-        pin_leftmost_column: bool = True,
-        suffix_to_colour_grey: list[str] | None = None,
     ) -> None:
         """Initializes the support table.
 
@@ -49,8 +47,6 @@ class AltinnSupportTable:
             get_data_func: Function that returns data to show in the supporting table.
             editor_inputs: List of inputs from the AltinnSkjemadataEditor to use as arguments in the get_data_func.
             variableselector: VariableSelector instance for adding arguments from the overall variableselector.
-            pin_leftmost_column: Optional. Boolean to pin the leftmost column in the AgGrid (usually the ident-col). Defaults to "True".
-            suffix_to_colour_grey: Optional. List of column-name suffixes to colour light grey in the AgGrid. Defaults to "[_x]".
 
         Note:
             The component is automatically added to the panel inside the modal.
@@ -58,8 +54,6 @@ class AltinnSupportTable:
         self.label = label
         self.inputs = editor_inputs
         self.get_data_func = get_data_func
-        self.pin_leftmost_column = pin_leftmost_column
-        self.suffix_to_colour_grey = suffix_to_colour_grey or ["_x"]
         if variableselector:
             self.variableselector = variableselector
         else:
@@ -82,10 +76,8 @@ class AltinnSupportTable:
         """The content to show in the support table."""
         return html.Div(
             dag.AgGrid(
-                defaultColDef={"editable": False, "filter": True},
-                dashGridOptions={"enableCellTextSelection": True},
+                defaultColDef={"editable": False},
                 id=f"support-table-{self.suptable_id}",
-                style={"height": "700px"},
             )
         )
 
@@ -103,17 +95,7 @@ class AltinnSupportTable:
                 f"Running get_data_func for table '{self.label}' using args: {args}"
             )
             data = self.get_data_func(*args)
-            column_defs = []
-            for col in data.columns:
-                col_def: dict[str, Any] = {"field": col, "headerName": col.lower()}
-                if self.suffix_to_colour_grey and col.endswith(
-                    tuple(self.suffix_to_colour_grey)
-                ):
-                    col_def["cellStyle"] = {"backgroundColor": "#e8e9eb"}
-                column_defs.append(col_def)
-            if self.pin_leftmost_column and column_defs:
-                column_defs[0]["pinned"] = "left"
-            return data.to_dict("records"), column_defs
+            return data.to_dict("records"), [{"field": col} for col in data.columns]
 
     def support_table_layout(self) -> dbc.Tab:
         """Creates the layout."""
@@ -197,7 +179,7 @@ class AltinnEditorSupportTables:
             ],
             id="skjemadata-hjelpetabellmodal",
             is_open=False,
-            className="hjelpetabeller-modal",
+            size="xl",
         )
         logger.debug("Created support tables modal")
         return hjelpetabellmodal