fix(dashboard): Missing filter card styles (#23256)
This commit is contained in:
parent
1f3eb56688
commit
9d0eea9600
|
|
@ -17,6 +17,7 @@
|
|||
* under the License.
|
||||
*/
|
||||
import React, { FC, useEffect, useMemo, useRef } from 'react';
|
||||
import { Global } from '@emotion/react';
|
||||
import { useHistory } from 'react-router-dom';
|
||||
import {
|
||||
CategoricalColorNamespace,
|
||||
|
|
@ -25,6 +26,7 @@ import {
|
|||
isFeatureEnabled,
|
||||
SharedLabelColorSource,
|
||||
t,
|
||||
useTheme,
|
||||
} from '@superset-ui/core';
|
||||
import pick from 'lodash/pick';
|
||||
import { useDispatch, useSelector } from 'react-redux';
|
||||
|
|
@ -57,6 +59,7 @@ import { DashboardContextForExplore } from 'src/types/DashboardContextForExplore
|
|||
import shortid from 'shortid';
|
||||
import { RootState } from '../types';
|
||||
import { getActiveFilters } from '../util/activeDashboardFilters';
|
||||
import { filterCardPopoverStyle, headerStyles } from '../styles';
|
||||
|
||||
export const DashboardPageIdContext = React.createContext('');
|
||||
|
||||
|
|
@ -140,6 +143,7 @@ const useSyncDashboardStateWithLocalStorage = () => {
|
|||
};
|
||||
|
||||
export const DashboardPage: FC<PageProps> = ({ idOrSlug }: PageProps) => {
|
||||
const theme = useTheme();
|
||||
const dispatch = useDispatch();
|
||||
const history = useHistory();
|
||||
const dashboardPageId = useSyncDashboardStateWithLocalStorage();
|
||||
|
|
@ -274,9 +278,12 @@ export const DashboardPage: FC<PageProps> = ({ idOrSlug }: PageProps) => {
|
|||
if (!readyToRender) return <Loading />;
|
||||
|
||||
return (
|
||||
<DashboardPageIdContext.Provider value={dashboardPageId}>
|
||||
<DashboardContainer />
|
||||
</DashboardPageIdContext.Provider>
|
||||
<>
|
||||
<Global styles={[filterCardPopoverStyle(theme), headerStyles(theme)]} />
|
||||
<DashboardPageIdContext.Provider value={dashboardPageId}>
|
||||
<DashboardContainer />
|
||||
</DashboardPageIdContext.Provider>
|
||||
</>
|
||||
);
|
||||
};
|
||||
|
||||
|
|
|
|||
Loading…
Reference in New Issue