Skip to content

Commit 7dae890

Browse files
authored
fix: DH-20953: AG Grid styling only worked when deephaven.ui installed (#1271)
- Adding a generic widget-container class that can be used by any widget - See deephaven/web-client-ui#2581 for the web-client-ui updates - Not removing the old class so it doesn't break in the interim - Set up deephaven.ui to handle `.widget-container` correctly
1 parent ffad984 commit 7dae890

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

plugins/ag-grid/src/js/src/components/AgGridWidget.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ export function AgGridWidget(
4646
const table = useWidgetFetch(dh, fetch);
4747

4848
return table != null ? (
49-
<div className="ui-table-container">
49+
<div className="ui-table-container widget-container">
5050
<AgGridView table={table} settings={settings} agGridProps={agGridProps} />
5151
</div>
5252
) : (

plugins/ui/src/js/src/styles.scss

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,7 @@
6767
> .ui-table-container:only-child
6868
> .iris-grid:only-child
6969
),
70+
&:has(.dh-inner-react-panel > .widget-container:only-child),
7071
&:has(.dh-inner-react-panel > .chart-wrapper:only-child) {
7172
// remove the default panel padding when grid or chart is the only child
7273
padding: 0 !important; // important required to override inline spectrum style

0 commit comments

Comments
 (0)