rusackas commented on code in PR #43638:
URL: https://github.com/apache/superset/pull/43638#discussion_r3878261983


##########
docs/admin_docs/configuration/theming.mdx:
##########
@@ -240,6 +240,40 @@ Font URLs are validated against a configurable allowlist. 
By default, fonts from
 
 This feature works with the stock Docker image - no custom build required!
 
+## Results Grid Configuration Overrides
+
+Superset exposes a handful of opt-in tokens that customize the appearance of
+AG Grid-backed result tables, such as the results grid in SQL Lab. These
+tokens have no effect unless explicitly set, since the results grid otherwise
+falls back to its built-in defaults.
+
+```python
+THEME_DEFAULT = {
+    "token": {
+        "colorPrimary": "#2893B3",
+        # ... other Ant Design tokens
+
+        # Results grid overrides
+        "resultsGridRowHeight": 32,
+        "resultsGridHeaderFontSize": 13,
+        "resultsGridHeaderFontWeight": 600,
+        "resultsGridBorderRadius": 4,
+        "resultsGridNoStriping": True,
+    }
+}
+```
+
+| Token | Type | Description |
+| --- | --- | --- |
+| `resultsGridRowHeight` | `number` | Row and header height, in pixels. |
+| `resultsGridHeaderFontSize` | `number` | Header cell font size, in pixels. |
+| `resultsGridHeaderFontWeight` | `number` | Header cell font weight. |
+| `resultsGridBorderRadius` | `number` | Border radius applied to the grid and 
its wrapper, in pixels. |

Review Comment:
   wrapperBorderRadius is consumed — it flows through GridThemeOverrides into 
ThemedAgGridReact's `withParams`, which is ag-grid's own theming API param for 
the wrapper's corner radius (ag-grid 36 Theming API). Not a doc bug.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to